SNflows / flows

Flows Pipeline
https://flows.phys.au.dk
GNU General Public License v3.0
3 stars 4 forks source link

Remove API scripts, add pointer to SNflows/tendrils, continue re-organizing code. #49

Closed emirkmo closed 2 years ago

emirkmo commented 2 years ago

https://github.com/SNflows/tendrils is the new API endpoint.

Now a discussion should be had about which API endpoints should actually be in there (e.g., should photometry be its own separate module?) Should query TNS/Alerce/etc. get some functionality there? (I think so).

After we decide, then the ones from here should be removed, and a pointer should be added to README and onto the wiki/website.

Finally, there are a bunch of scripts here that are not really part of the flows photometry pipeline, but are more helper scripts for e.g. populating the marshal. I propose that they should be moved to either flows-tools, or a separate module. The end goal is to isolate only the photometry and related pipeline scripts here, make it a version LOCKED module, and then release it as stable V1.0. Until then we should go to v0.10, etc.

emirkmo commented 2 years ago

The linked PR moves the API wrapper to Tendrils.

To make even less dependencies, some of the Runners should really be moved to a separate repo, probably flows-tools for now. (Such as run-querytns, run-ztf, etc.)

Eventually, they would make sense in a flows Webserver/Database repo. This repo can also have the python based API (ReST) that interacts with the database.