Ingest uc photometry - Githubissues

SIMPLE-AstroDB / SIMPLE-db

BSD 3-Clause "New" or "Revised" License

11 stars 21 forks source link

Ingest uc photometry #516

Closed Exu-112 closed 1 week ago

Exu-112 commented 2 weeks ago

Short description: Ingesting Spitzer photometry from the Ultracool sheet

Link to relevant issue: Closes #515

For data ingests:

[x] includes script used for ingest
[ ] includes modified JSON files
[x] Add new tests
[ ] Update the Versions table

kelle commented 2 weeks ago

I'd prefer to the script before the JSON files.

Exu-112 commented 2 weeks ago

I only added the script for photometry, the other commits are because I branched off of the radial velocity branch

kelle commented 1 week ago

Now that the rv branch has been merged, this one is a little messed up. My advice is to 1) make a copy of your new photometry script and tests. Save someplace like your Desktop. 2) make a new branch from upstream main. 3) move your new script and tests into the new branch. add, commit, push. 4) open a new pull request, close this one. Let me know if you have any trouble!