Use annotation instead of field for tagged union #244
Merged
Conversation
This file contains 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
also removed non-standard axis
|
@edyoshikun this should fix CI failures in #242. |
edyoshikun
approved these changes
Sep 7, 2024
edyoshikun
pushed a commit
that referenced
this pull request
Sep 9, 2024
also removed non-standard axis
edyoshikun
pushed a commit
that referenced
this pull request
Sep 17, 2024
also removed non-standard axis
ieivanov
added a commit
that referenced
this pull request
Sep 27, 2024
* initial commit with added docs prior to refactoring to simplify the naming and functions. * considering varying t_idx_in and out * template for hypothesis * Refactor NGFF module and migrate to Pydantic v2 (#233) * unify comment format * fix typing and docstring * create ngff sub-package and refactor display util file * refactor ngff meta file * refactor ngff * export transformation model * fix type hint * bump ome-zarr target in docstring * migrate to pydantic v2 * isort * fix validators * remove union type * fix dependency * update docstring * typing improvements * Update iohub/ngff/models.py Co-authored-by: Jordão Bragantini <[email protected]> * fix style * update module docstring to specify their content --------- Co-authored-by: Jordão Bragantini <[email protected]> * improving docstring for functions and renaming input and output path arguments to be more intuitive. * Fixing pyramid scaling factor (#238) * Fixing pyramid scaling factor * fix pyramid test * fix test again * Use annotation instead of field for tagged union (#244) also removed non-standard axis * Export `ImageArray` from the `ngff` module (#245) * adding the ImageArray * precommit * black * renaming method arguments to have consistent naming structure * flake8 * refactor _calculate_zyx_chunk_size * use input_store_path and output_store_path throughout * style * rename and clean up time indices * update time_indices documentation * add processing for channel indices * fix syntax and move ngff_utils.py to ngff/utils.py * update import * typing * docs typos * fix process_single_position iterator @talonchandler @edyoshikun * update apply_transform... docstring * compatibility with minimal deskew w/ @edyoshikun * pretty flat_iterable * adding new tests * create_empty test without testing channel names * fixing the create_empty_zarr extra indentation * -attempt to fix apply_transform_test. @ieivanov revert if needed * fixed apply_transform_czyx * debug pytest * fixing the test for create_empty_plate pytest * synchronize log messages * docs improvements * improved docs and typing for Callable func * remove commented slurmkit fix * delete unused function --------- Co-authored-by: Ziwen Liu <[email protected]> Co-authored-by: Jordão Bragantini <[email protected]> Co-authored-by: Ivan Ivanov <[email protected]> Co-authored-by: Talon Chandler <[email protected]>
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.
Use annotation instead of field for tagged union to allow
list[AxisMeta]to work with pydantic 2.9.0.Also removed non-standard axis which was added in anticipation of RFC-3 to make the fix easier.