TOMToolkit / tom_base

The base Django project for a Target and Observation Manager
https://tom-toolkit.readthedocs.io
GNU General Public License v3.0
26 stars 46 forks source link

Data harvested from brokers doesn't have assigned Data Product #1122

Closed KKruszynska closed 3 days ago

KKruszynska commented 6 days ago

Currently, if a harvester downloads data into a TOM Toolkit-based system, the data points are stored as a ReducedDatum, but they don't have a linked Data Product. This is not the case if the user uploads the data. It is necessary for data sharing with the collaborators (for example, if there is Gaia data in my TOM, I cannot share it easily with the others through the Target page) and for efficiently fitting microlensing events tracked by MOP.

jchate6 commented 5 days ago

Hi @KKruszynska, we have hesitated to automatically create data products associated with the results of a broker query because as data comes in, the same query can generate different results each time. This could result in "overlapping" data products that contain mostly the same information with a few extra data points each time.

The preferred way to share data is to use the sharing options at the bottom of the photometry tab to upload data to Hermes, download data to a csv, or directly via the API of another TOM. image

Alternatively you can share the entire target and any of its data from the "share" button under the target name on the target detail page: image

I understand that this can be clunky with the current tables. We are working on adding filtering, pagination and sorting to these tables to make them more useful. see

If filtering out specific data points because they have already been shared is too cumbersome, it should be safe to share all the photometry data with another TOM and it should only ingest new data.

If you are sharing the data with a collaborator who doesn't have a TOMToolkit TOM, then hopefully the Download option should be what you need.

If you are trying to share spectroscopy, or any of the above features are not present in MOP, or are giving you trouble, let me know and I can help sort them out.

rachel3834 commented 3 days ago

Many thanks for explaining @jchate6 - I agree that reasoning makes sense for data that routinely updates.
The data sharing tools are a good way of manually downloading the data.
I'll close this thread because what we now think we actually need is an API, but that's a different issue....