wrote script for radial velocities

SIMPLE-AstroDB / SIMPLE-db

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

11 stars 22 forks source link

wrote script for radial velocities #505

Closed Exu-112 closed 3 months ago

Exu-112 commented 3 months ago

Created csv with only sources with rv and rv_ref. Wrote script ingesting them into the RadialVelocities table.

kelle commented 3 months ago

Just curious, how did you create the csv files with only rv data? And I assume we only need the file that includes references, right?

kelle commented 3 months ago

Also just noticed that uncertainty is missing. Please also ingest the uncertainty on the measurement.

Exu-112 commented 3 months ago

I created the csv files manually using a bit of google sheets automation - figured it would be quicker than writing code for it. It doesn't seem that the new method for ingestion works on my side, though.

dr-rodriguez commented 3 months ago

I created the csv files manually using a bit of google sheets automation - figured it would be quicker than writing code for it. It doesn't seem that the new method for ingestion works on my side, though.

What do you mean that "the new method for ingestion" doesn't work on your side? Can you make the comment in the lines of code that you are referring to? I see over a hundred json files modified with radial velocities so it looks like it works. Is this just an old comment and it's working now?