-
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
zhang19 gaia names #438
zhang19 gaia names #438
Conversation
There is no gaia_j0452-36a json file found to modify. |
Then I guess we should ingest it. Here's the original paper. |
How can I run/check that the json file edit was implemented in the db? |
Ingest using a new file or is there an existing file to add to? |
I'm not sure I understand the question. The JSON file modification is changing the database. If you want to see the change in the change in the DB browser, you reload the database. That will generate a new SQLite file using the JSON files. |
That's what I thought too and reloaded the db. Unless I misunderstood what you meant by changing the names. I added 'Gaia EDR3 4818823808553134592' as another name under 'Gaia J0452-36B'. Or does it mean to delete the 'Gaia J0452-36B.json' file and create a new one with the name 'Gaia EDR3 4818823808553134592'? |
This paper is already ingested. Is gaia_j0452-36a and gaia_j0452-36b supposed to be two sources from the same paper? 36b is there but no json for 36a. |
Yes, two sources from the same paper. |
It does not pass the test when changing the source because it says there is no Gaia EDR3 4818823808553134592 in the Source table, just the first name. |
I attempted making a json file for gaia 36a, but the ra, dec, equinox, and spectral type code are wrong. I only got the spectral type string and changed the source name. In the paper it seems like I have to convert the coordinates data into decimal before I can ingest them. |
|
Might as well also do the photometry. The relevant filters names from the Photometry Filters table are:
|
data/gaia_j0452-36b.json
Outdated
"value": -1.4, | ||
"value_error": 0.2, | ||
"unit": "Fe/H", | ||
"comments": null, |
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.
"comments": null, | |
"comments": "[Fe/H]", |
Tests for data and integrity are failing completely at the moment |
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.
Looks great!
Short description: Changing two Gaia names
Gaia J0452-36A should be Gaia EDR3 4818823636756117504
Gaia J0452-36B should be Gaia EDR3 4818823808553134592
The Gaia J names should be removed.
Link to relevant issue: Closes #245
For data ingests: