Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #119 +/- ##
===========================================
- Coverage 72.53% 26.54% -45.99%
===========================================
Files 31 32 +1
Lines 1857 1891 +34
===========================================
- Hits 1347 502 -845
- Misses 510 1389 +879 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
|
||
| combine_coord = time_coord & energy_coord | ||
|
|
||
| # I'm not sure what should live in the meta vs extra_coords |
There was a problem hiding this comment.
In ndcube-land, a coordinate is a strictly monotonic sampling of a coordinate space, i.e., the values must be numerical, ordered and strictly increasing or decreasing. If your property doesn't satisfy this, then it's axis-aligned metadata. Once ndcube #455 is merged and ndcube 2.3 released, NDCube will support axis-aligned metadata. This will be the appropriate place for detector labels, as they will be sliced when the cube is sliced. And we can add functionality to a class STIXMeta(ndcube.Meta) so that detector labels are properly combined when the data is rebinned.
| meta = Meta({"detector": ["a", "b", "c"]}, data_shape=pd_shape) | ||
|
|
||
| scube = NDCube(data=pd, wcs=combine_coord.wcs, meta=meta) | ||
| scube.extra_coords.add("integration_time", 0, [2, 3] * u.s, physical_types="time.duration") |
There was a problem hiding this comment.
Following from the above comment, "integration time" should be considered metadata, not a coordinate.
| from stixpy.extern.meta import Meta | ||
|
|
||
|
|
||
| class PossionUncertainty(NDUncertainty): |
There was a problem hiding this comment.
This is very exciting!
2 participants
No description provided.