Update to rail v2.0: Parameter changes, ephemeral DataStore and interactive package#244
Merged
Conversation
…rage instead of a permanent storage
…_do_chunk_output` in CatEstimator into two separate functions
…tor and minor fixes for _update_ancil and _handle_chunk_output
Future work to come for specific tooling - usage will look something like `numpydoc validate rail.estimation.algos.random_gauss.RandomGaussEstimator.estimate` where this points to a specific entrypoint function
…mode = return in non-parallel usage
…minor spelling fixes
Stubs paths aren't quite right I think
Stub files are copy/pasted and then have additional formatting and typing is imported
…g if file is in memory so chunk size is effectively the whole file
…into sidrat-staging
… had moved inside an if statement
…into sidrat-staging
…that it collects multiple outputs in return mode
18 tasks
eacharles
approved these changes
Feb 5, 2026
Collaborator
eacharles
left a comment
There was a problem hiding this comment.
I'm wondering a bit about the process you envisage for keeping the .pyi files up to data.
Thanks for all the very nice work.
Contributor
Author
That's a good question -- we can go over it a bit in the meeting on Monday, but our current thought is to add a test to the smoke tests that checks if the .pyi files need to be updated, and then will output the appropriate error message so it will be obvious that what needs to happen is a quick PR where you just run the script to create the .pyi files again. |
- removes the `input` kwarg that was sometimes a dict, all params are now called by their original names - reads the ceci input handles of stages, and ensures those are captured in docstrings and required kwargs of interactive functions - many knock-on effects of those two things
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem & Solution Description (including issue #)
This PR involves the following changes:
Code Quality
#pragma: no cover; in the case of a bugfix, a new test that breaks as a result of the bug has been added