Skip to content

Modernize molplotly: pyproject.toml, drop JupyterDash - #36

Merged
wjm41 merged 2 commits into
mainfrom
claude/refactor-molecular-viz-Gmdoa
Sep 18, 2026
Merged

wjm41 merged 2 commits into
mainfrom
claude/refactor-molecular-viz-Gmdoa

Conversation

@wjm41

@wjm41 wjm41 commented Mar 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • Migrate from setup.py to pyproject.toml for modern Python packaging (PEP 621)
  • Replace deprecated JupyterDash with standard Dash, which now natively supports Jupyter
  • Expand test coverage with comprehensive tests for add_molecules including edge cases, error handling, and SVG rendering
  • Update CI workflow to use the modernized package setup
  • Add PLAN.md documenting cleanup and modernization roadmap

Test plan

  • Verify pip install . works with the new pyproject.toml
  • Run pytest tests/ to confirm all tests pass
  • Test add_molecules in a Jupyter notebook to verify Dash integration
  • Check that the GitHub Actions CI workflow passes

https://claude.ai/code/session_0117V87rftVNjK2yCo3xZK63

@wjm41
wjm41 merged commit 60e3772 into main Sep 18, 2026
3 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.

2 participants