Skip to content

Make interactive visualization example optional-dependency-safe for examples verification - #1338

Merged
SkBlaz merged 4 commits into
masterfrom
copilot/verify-examples-work
Apr 9, 2026
Merged

Make interactive visualization example optional-dependency-safe for examples verification#1338
SkBlaz merged 4 commits into
masterfrom
copilot/verify-examples-work

Conversation

Copilot AI commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

The examples verification issue was caused by an interactive visualization example hard-failing when plotly is not installed. This made examples checks brittle in environments that intentionally omit optional visualization deps.

  • Example behavior update (optional dependency handling)

    • Updated examples/visualization/example_interactive_multilayer.py to treat missing plotly as a graceful skip (exit code 0) instead of an error exit.
    • This keeps optional tooling from blocking broad example validation runs.
  • Regression coverage for the specific failure mode

    • Added tests/test_example_interactive_multilayer.py to execute the example in a subprocess and assert non-failing behavior under standard example-test environment settings.
try:
    import plotly.graph_objects as go
except ImportError:
    print(" Plotly not found. Install with: pip install plotly")
    print(" Skipping interactive visualization example")
    exit(0)

Copilot AI linked an issue Apr 8, 2026 that may be closed by this pull request
Copilot AI and others added 3 commits April 8, 2026 21:08
Agent-Logs-Url: https://github.com/SkBlaz/py3plex/sessions/520d56eb-e9a7-4001-a6a9-42cad607deb2

Co-authored-by: SkBlaz <10035780+SkBlaz@users.noreply.github.com>
This reverts commit ca1540d.

Co-authored-by: SkBlaz <10035780+SkBlaz@users.noreply.github.com>
Copilot AI changed the title [WIP] Verify that all examples work properly Make interactive visualization example optional-dependency-safe for examples verification Apr 8, 2026
Copilot AI requested a review from SkBlaz April 8, 2026 21:12
@SkBlaz
SkBlaz marked this pull request as ready for review April 9, 2026 15:46
@SkBlaz
SkBlaz merged commit 3380cf8 into master Apr 9, 2026
34 checks passed
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.

Examples

2 participants