Skip to content

Expose missing plotting exports and align version metadata#1

Merged
b-rodrigues merged 4 commits intomainfrom
copilot/improve-submodule-imports
Jan 26, 2026
Merged

Expose missing plotting exports and align version metadata#1
b-rodrigues merged 4 commits intomainfrom
copilot/improve-submodule-imports

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 26, 2026

Improve top-level imports so plotting helpers like rxp_phart are accessible directly from ryxpress, and align package version metadata across pyproject.toml and __init__.py.

  • Exports
    • Add lazy export for rxp_phart (plotting)
    • Align module mapping documentation with actual filenames
  • Versioning
    • Bump package version to 0.1.3 in pyproject.toml and __init__.py
  • Tests
    • Add a lightweight test to confirm new exports resolve from ryxpress

Example:

from ryxpress import rxp_phart
Original prompt

please complete init.py to make submodule importing better, for examalpe rxp_phart is not listed there so I need to do from ryxpress.plotting import rxp_phart but I don’t want to have to write all that, so double check that everything is there, also make sure to bump version in pyproject toml and in init.py once you’re done


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits January 26, 2026 16:42
Co-authored-by: b-rodrigues <2998834+b-rodrigues@users.noreply.github.com>
Co-authored-by: b-rodrigues <2998834+b-rodrigues@users.noreply.github.com>
Copilot AI changed the title [WIP] Improve submodule imports in __init__.py Expose missing plotting exports and align version metadata Jan 26, 2026
Copilot AI requested a review from b-rodrigues January 26, 2026 16:45
@b-rodrigues
Copy link
Copy Markdown
Owner

@copilot do we need to have RRunResult importable?

Co-authored-by: b-rodrigues <2998834+b-rodrigues@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jan 26, 2026

@copilot do we need to have RRunResult importable?

Removed the top-level RRunResult export and updated the import test accordingly in commit e0548d6.

@b-rodrigues b-rodrigues marked this pull request as ready for review January 26, 2026 16:51
@b-rodrigues b-rodrigues merged commit ab240dc into main Jan 26, 2026
1 check 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