-
-
Notifications
You must be signed in to change notification settings - Fork 404
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding a general interface for N-dimensional gridded data #542
Merged
Merged
Changes from all commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
6236444
Added validation method to DictColumns interface
philippjfr f00c306
Fix for scalar columns in DictColumns
philippjfr 9589707
Added initial dense Columns interface
philippjfr 0645535
Consistently added expanded keyword to dimension_values method
philippjfr ead988f
Renamed NdArrayColumns to GridColumns
philippjfr f26d95f
Cleanup and minor fixes in core.data module
philippjfr 03c1e22
Added validation to detect dense formats in existing interfaces
philippjfr 81bb78f
Added default arguments to interface values method
philippjfr c5f4f61
Updated Column interface unit test
philippjfr 3b5cada
Reverted change to DictColumns interface
philippjfr 631e91c
Small fixes for NdElement interface
philippjfr 49e729a
Changed GridColumns format to expand vdims
philippjfr 7d5dc26
Improved GridColumns validation
philippjfr 232b372
Fixed scalar return values from GridColumn slicing
philippjfr d6e3440
Ensured GridColumns aggregate returns at least 1D array
philippjfr e30bc53
Implemented GridColumns add_dimension and sort methods
philippjfr 46c5e8a
Added unit tests for GridColumns interface
philippjfr 034e659
Added missing import in core.util
philippjfr 84a3667
Renamed Columns interface reshape method to init
philippjfr 5780d1d
Removed stray GridColumns.add_dimension method
philippjfr 4d0e547
Renamed check_dense to expanded_format and improved validation
philippjfr a38409b
Renamed GridColumns coord_mask to key_select_mask
philippjfr 487e667
Added comment for Image dimension_values method
philippjfr 5468271
Enforced samples have uniform length on GridColumns
philippjfr 626dd07
Allowed returning non-flat key dimensions from gridded Elements
philippjfr b664343
Allowed dropping constant dimensions via GridColumns.reindex
philippjfr fb8a2f2
Improved error message on GridColumns.sort
philippjfr 0af0d1e
Disabled support for expanding vdims in GridColumns
philippjfr 4ff50a4
Small fixes for NdColumns and DFColumns constructors
philippjfr 675dd4c
Updated GridColumns unit test
philippjfr b853d12
Updated GridColumns value slicing exception
philippjfr 0ffcbf2
Fixed ArrayColumns init bug
philippjfr c509a07
Fixed inverted Image.dimension_values
philippjfr 2a640c6
Renamed expanded_format method to expanded
jlstevens c109675
Updated the class docstring for GridColumns
jlstevens 1b8d27a
Added 'grid' interface to default datatype list
jlstevens File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
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
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
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
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
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
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
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
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
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
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
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The values are the flattened, transposed
flipud
of the data array? This may well be correct but we will need to know why this is necessary and document it.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will add some comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.