Skip to content

Cache gdot script execution results in Sphinx environment#49

Merged
sdpython merged 9 commits intomainfrom
copilot/speedup-runpython-execution
Mar 6, 2026
Merged

Cache gdot script execution results in Sphinx environment#49
sdpython merged 9 commits intomainfrom
copilot/speedup-runpython-execution

Conversation

Copy link
Contributor

Copilot AI commented Mar 3, 2026

  • Explore codebase to understand the issue
  • Add script execution caching to GDotDirective using the Sphinx environment
  • Prefix cache key with docname.split("/")[-1] so entries are identifiable by source document
  • Add test_gdot_script_cache test verifying identical scripts produce correct output via cache
  • Run existing tests to ensure no regression
  • Code review and security scan
Original prompt

This section details on the original issue you should resolve

<issue_title>speedup runpython execution for graphviz</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI changed the title [WIP] Speed up runpython execution for graphviz Cache gdot script execution results in Sphinx environment Mar 3, 2026
Copy link
Contributor Author

Copilot AI commented Mar 6, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • cdnjs.cloudflare.com
    • Triggering command: /usr/bin/python3 /usr/bin/python3 -m sphinx -b rst /tmp/tmpe6erkhnq /tmp/tmpe6erkhnq/output (dns block)
    • Triggering command: /usr/bin/python3 /usr/bin/python3 -m sphinx -b rst /tmp/tmp7r_8ciy_ /tmp/tmp7r_8ciy_/output (dns block)
    • Triggering command: /usr/bin/python3 /usr/bin/python3 -m sphinx -b rst /tmp/tmpovcj_nv1 /tmp/tmpovcj_nv1/output (dns block)
  • requirejs.org
    • Triggering command: /usr/bin/python3 /usr/bin/python3 -m sphinx -b rst /tmp/tmpe6erkhnq /tmp/tmpe6erkhnq/output (dns block)
    • Triggering command: /usr/bin/python3 /usr/bin/python3 -m sphinx -b rst /tmp/tmp7r_8ciy_ /tmp/tmp7r_8ciy_/output (dns block)
    • Triggering command: /usr/bin/python3 /usr/bin/python3 -m sphinx -b rst /tmp/tmpovcj_nv1 /tmp/tmpovcj_nv1/output (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@xadupre xadupre marked this pull request as ready for review March 6, 2026 14:44
@sdpython sdpython merged commit f3fadce into main Mar 6, 2026
22 checks passed
@sdpython sdpython deleted the copilot/speedup-runpython-execution branch March 6, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

speedup runpython execution for graphviz

3 participants