-
Notifications
You must be signed in to change notification settings - Fork 36
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
Tag flexibility #801
Tag flexibility #801
Conversation
please do not merge yet |
This would be it for the moment, please leave your 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.
I compiled the following tables from the data access test for Tag while reviewing and thought it might be useful to have associated with the PR.
Data dimensionality and sampling interval:
1D | 2D | 3D | |
---|---|---|---|
Dimensionality | 100 | 100, 10 | 100, 10, 5 |
Sampling (first dim only) | 0.1 | 1.0 | 1.0 |
Tag data retrieval
Position | Extent | Data 1D | Data 2D | Data 3D |
---|---|---|---|---|
25, 0 | 50, 5 | 501 | 51, 6 | 51, 6, 5 |
25, 0, 0 | None | 1 | 1, 1 | 1, 1, 1 |
25, 0, 0 | 0, 0, 0 | 1 | 1, 1 | 1, 1, 1 |
25, 0, 0 | 0, 0 | Error | Error | Error |
So as discussed, the data access rules now are:
- Length of Position and Extent arrays should still match.
- Extra dimensions in Pos and Ext wrt DataArray are ignored (e.g., 3 pos on 2D array).
- Missing dimensions in Pos and Ext wrt DataArray are considered "full slice" (e.g., 2 pos on 3D array).
- Zeroes in Ext are equivalent to no Ext.
auto extSize = extents.size(); | ||
auto it = refs.begin(); | ||
while (!mismatch && (it != refs.end())) { | ||
auto arrayExtent = (*it).dataExtent(); | ||
mismatch = extSize != arrayExtent.size(); | ||
++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.
Delete?
addresses #800 but only for 1.4