-
Notifications
You must be signed in to change notification settings - Fork 2
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
ensure ogip lightcurve is produced by oda api #267
Comments
It's about oda_api/oda_api/data_products.py Line 754 in 7dcb229
Of course, these parts should not be completely separated (as well as image, spectrum etc) Even worse, we have some logic (mainly visualisation) in dispatcher plugin https://github.com/oda-hub/dispatcher-plugin-nb2workflow/blob/9ea4d3c8202ca500e3cc0bf9f7a2ffb9d4755d4f/dispatcher_plugin_nb2workflow/products.py#L241 which refers to the main code https://github.com/oda-hub/dispatcher-app/blob/e8f8082f68008510a7c9e865ff883c34852890a4/cdci_data_analysis/analysis/products.py#L406 completely separate from what's in oda_api. Not saying that we will also have a js visualisation library... |
need to replace this with the coming ogip functionality when reading from file, there should be product-specific validation and transformation (i.e. loose reading and strict writing) |
as reminded by esg-epfl-apc/tools-astro#115 (comment) and it's good to follow the standard.
@burnout87 I think you dealt with this?
The text was updated successfully, but these errors were encountered: