Details
-
Type:
Story
-
Status: Won't Fix
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: afw
-
Labels:
-
Story Points:2
-
Epic Link:
-
Team:Data Release Production
Description
This include file was damaged somewhat by the addition of slot routines to work with the flattened field definitions. It would be nice to put the Measurement abstraction back in place – or get rid of it. We need to decide whether the old slot and compound key mechanisms from SourceTable version 0 are going to be continued for doing this.
Attachments
Issue Links
Activity
Perry Gee
created issue -
Perry Gee
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Epic Link | DM-15 [ 11263 ] |
I thought that I would also need to define some new FunctorKeys (for flux = double) and define the methods of Point2DKey under FunctorKey< Point<double> >, but perhaps you have a way to define a meas and err FunctorKey for each Measuremnt instantiation. I didn't see how to do that.
Anyway, it seemed better to leave this for later, since the lack of slots is holding up other work.
Perry Gee
added a comment - I thought that I would also need to define some new FunctorKeys (for flux = double) and define the methods of Point2DKey under FunctorKey< Point<double> >, but perhaps you have a way to define a meas and err FunctorKey for each Measuremnt instantiation. I didn't see how to do that.
Anyway, it seemed better to leave this for later, since the lack of slots is holding up other work. | Epic Link | DM-15 [ 11263 ] |
| Rank | Ranked higher |
| Labels | Measurement | AppsPlanning Measurement |
| Labels | AppsPlanning Measurement | SciencePipelines |
| Team | Princeton [ 10301 ] |
This is essentially obsoleted by the work planned for DM-1764. Will close as Won't Fix.
| Resolution | Done [ 10000 ] | |
| Status | To Do [ 10001 ] | Won't Fix [ 10405 ] |
I think the right time to deal with this is after we've added table aliases in
DM-417andDM-419; I think that's what will allow us to clean this up.