astrodbtoolkit / astrodb-template-db

A template for astronomical databases.
https://astrodb-template-db.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
1 stars 2 forks source link

Add Radial_Velocity table #69

Open kelle opened 4 months ago

kelle commented 4 months ago

We have an ingest_radial_velocity function ready to go. Let's add the table!

arjunsavel commented 3 months ago

@kelle Don't we already have some example quantity tables? My gut feeling is that not everyone would have radial velocities in their own databse.

kelle commented 3 months ago

Hmm, this raises a good topic which I don't think we've discussed explicitly recently -- how do we want to decide what tables to include in the template. At the moment, I'm inclined to include more tables than less...it's easier for a user to remove/modify tables than it is to make up their own from scratch. The more tables we have defined, the more likely databases will be compatible with each other. For example, I think it would make sense for the template to have both a radial velocity AND redshift table.