Open guignonv opened 5 years ago
By the way, I don't need an "end time" on capture date but it may be useful for others. Should we add something like "timecapturedend TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,"?
What is the status on this? I mentioned to @guignonv that it would be nice if the PR https://github.com/GMOD/Chado/pull/108 could be reworked to be in migrations rather than changing the sql file directly, but I can do that since it's been awhile since this has been touched. Does anybody have an opinion about the need for a "timecapturedend" field?
I think it's a good idea to add the timecapturedend
column as we often store min, max, median temperatures. These could be stored with the timecaptured
being the start of the range, the timecapturedend
being the end of the range, the type_id pointing to a temperature minimum
cvterm and the value being the minimum temperature recorded.
PS. I approve this table addition 👍 and we would work it into our environmental data storage plans!
I added a commit that adds the timecapturedend
field, making it nullable and with a description: This optional value can be used to mark the end of the time that the catured fact data refers to, that is to provide a time span rather than a time point; can be null
Since this is a medium on the edge of large change (that is, I don't think there is a similar "fact" table anywhere else in Chado, is there?), I'd like to see a little more feedback about this change before I make the PR. My plan then would be to close @guignonv 's PR and create a new one based on this commit. Thoughts?
We're moving away from Chado for non-technical reasons, but we often thought we wanted to store "interventions" (e.g. insecticide spray campaigns, bednet distribution etc) in Chado and this would have been great.
For people doing plant characterization or breeding, it may be useful to store environmental data (climatic data and other). Those data are related to a specific geo-location and are taken at a given time. Therefore I would tend to use the nd_geolcation table for geo-locations and add a nd_fact table: