-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GAIA DR3 radial velocities ingest #253
Changes from all commits
cd894c2
44735bd
f31ada2
677d207
cf2a8a6
5446353
6f66805
01208a7
44a42a9
f0ec117
24bf584
022142e
7d49fbe
eec09a6
104f5ce
b2eee82
6dd68d2
166a07f
208525c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
{ | ||
"Sources": [ | ||
{ | ||
"source": "2MASS J16060391–2056443", | ||
"source": "2MASS J16060391\u20132056443", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. unicode. |
||
"ra": 241.516, | ||
"dec": -20.9457, | ||
"epoch": 2000.0, | ||
|
@@ -14,7 +14,7 @@ | |
], | ||
"Names": [ | ||
{ | ||
"other_name": "2MASS J16060391–2056443" | ||
"other_name": "2MASS J16060391\u20132056443" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. more unicode. \u2013 instead of - |
||
} | ||
], | ||
"Spectra": [ | ||
|
@@ -34,4 +34,4 @@ | |
"other_references": null | ||
} | ||
] | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6852,13 +6852,13 @@ | |
"description": "ISOCAM observations of the Chamaeleon I dark cloud" | ||
}, | ||
{ | ||
"publication": "Lópe04", | ||
"publication": "L\u00f3pe04", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We've got some unicode in here and I'm not sure what to do. @dr-rodriguez , thoughts? I feel like we've dealt with this before but I don't remember the discussion. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should be fixed now, but we will need to merge the latest changes from main into this branch and re-ingest the RVs |
||
"bibcode": "2004A&A...416..555L", | ||
"doi": "10.1051/0004-6361:20031720", | ||
"description": "The brown dwarf population in the Chamaeleon I cloud" | ||
}, | ||
{ | ||
"publication": "Góme01", | ||
"publication": "G\u00f3me01", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. more unicode. Not sure if we want \u00f3 or ó. @dr-rodriguez ? |
||
"bibcode": "2001AJ....121..974G", | ||
"doi": "10.1086/318776", | ||
"description": "A Near-Infrared Imaging Survey of the Chamaeleon I Dark Cloud" | ||
|
@@ -6874,5 +6874,11 @@ | |
"bibcode": "2020MNRAS.491.5925M", | ||
"doi": "10.1093/mnras/stz3441", | ||
"description": "Spectral library of age-benchmark low-mass stars and brown dwmarfs" | ||
}, | ||
{ | ||
"publication": "GaiaDR3", | ||
"bibcode": null, | ||
"doi": "10.1051/0004-6361/202243940", | ||
"description": "Gaia Data Release 3.Summary of the content and survey properties" | ||
} | ||
] | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to ingest RVs that have a very large uncertainty? @Will-Cooper , thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's a bigger discussion @kelle; I wouldn't trust any RVs for UCDs from Gaia to do any science but is the idea of the database to record all forms of data or just what we are saying is trustworthy? I see the Gaia G mag is 13.7 and the limit for RV processing is 14, so it's borderline snuck in there at the most uncertain bound.