SIMPLE-AstroDB / SIMPLE-db

Curated collection of data for low mass stars, brown dwarfs, and directly imaged exoplanets.
https://simple-bd-archive.org/
BSD 3-Clause "New" or "Revised" License
11 stars 22 forks source link

Spectral type string variety #100

Closed Will-Cooper closed 1 year ago

Will-Cooper commented 3 years ago

Do we want to require a specific format to the spectral types in the schema? e.g. first char should be in M,L,T,Y (or any letter perhaps); second char should be inclusively between 0-9.
To stop certain structures like "sdM0" or ">M10V".

kelle commented 3 years ago

I am ok with spectral type string being whatever it is, with whatever prefix's and suffix's the author's so choose. In PR #94 , I am modifying the schema so there's a spectral type code which can be used for querying. E.g., 0 = M0, 10=L0, etc.

kelle commented 3 years ago

@Will-Cooper , is your concern addressed by the Spectral Type tables doc? Maybe we should add more text there to say that spectral_type_string can be whatever, but spectral_type_code is standardized?

Will-Cooper commented 3 years ago

Sure, do we also want spectral_type_code to be searchable on? Would be nice to search spectral_type_string but understand that could get complex? I personally use 60=M0, 70=L0 but as long as the standardisation is listed as it is, don't think anyone can complain!

kelle commented 3 years ago

I think this is worth discussing...60 = M0 vs 0 = M0. Can you say a little bit about why you chose to start with 60?

Will-Cooper commented 3 years ago

It's more inclusive of the MS stars, useful if you're dealing with multiple systems and dumping both primaries and companions in the same table and want an automated system to create the numerical types. i.e. OBAFGKMLTY = 0,10,20,30,40,50,60,70,80,90

kelle commented 3 years ago

Any other thoughts on this? @NiallWhiteford ? @cfontanive ? @aburgasser ? (We need to decide before we can merge #112.)

cfontanive commented 3 years ago

If we're not including any objects earlier than ~mid-M, I would vote for MLTY=0,10,20,30 which is typically used in substellar contexts. We may have some comments about binarity for companion objects somewhere (?), but unless the primaries are themselves substellar, they won't have their own entry in the database, right? So this should not be a problem. I also agree with the spectral type string being allowed to be whatever ('sd', '>', a range...) and for queries to be made based on the spectral type code (M6, sdM6, >M6, M6-7 would all be 6 for example).

kelle commented 3 years ago

One thing I just thought of: if we go with the 0=O, 60=M0, it's higher probability that our database will be more compatible with other databases. @dr-rodriguez , is there some VO standard for this. (I doubt it, but thought I would ask. Maybe we should propose one!)

Will-Cooper commented 3 years ago

The only thing I can find online is the UCD keyword of src.spType, which is the MK spectral type as a string -- I don't think there's a vigorous standard anywhere... Perhaps a twitter poll? Eric Mamajek as well might be a good person to ask?

kelle commented 3 years ago

In interest of the future possibility of ingesting data on primary stars, we've decided to go with OBAFGKMLTY = 0,10,20,30,40,50,60,70,80,90. So M0=60. (And the docs need to be updated.)