Skip to content
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

Merged
merged 13 commits into from
Dec 27, 2023
Merged

zhang19 gaia names #438

merged 13 commits into from
Dec 27, 2023

Conversation

LishaRamon
Copy link
Collaborator

@LishaRamon LishaRamon commented Dec 22, 2023

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:

  • includes modified JSON files
  • Update the Versions table

@LishaRamon LishaRamon self-assigned this Dec 22, 2023
@LishaRamon
Copy link
Collaborator Author

There is no gaia_j0452-36a json file found to modify.

@LishaRamon LishaRamon requested a review from kelle December 22, 2023 19:20
@kelle
Copy link
Collaborator

kelle commented Dec 22, 2023

Then I guess we should ingest it.

Here's the original paper.
https://academic.oup.com/mnras/article/489/1/1423/5545604

@LishaRamon
Copy link
Collaborator Author

How can I run/check that the json file edit was implemented in the db?

@LishaRamon
Copy link
Collaborator Author

Then I guess we should ingest it.

Here's the original paper. https://academic.oup.com/mnras/article/489/1/1423/5545604

Ingest using a new file or is there an existing file to add to?

@kelle
Copy link
Collaborator

kelle commented Dec 22, 2023

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.

@LishaRamon
Copy link
Collaborator Author

LishaRamon commented Dec 22, 2023

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'?

@LishaRamon
Copy link
Collaborator Author

Then I guess we should ingest it.

Here's the original paper. https://academic.oup.com/mnras/article/489/1/1423/5545604

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.

@kelle
Copy link
Collaborator

kelle commented Dec 22, 2023

It might have gotten changed back when you merged. The way you had it originally was correct image

@kelle
Copy link
Collaborator

kelle commented Dec 22, 2023

Yes, two sources from the same paper.

@LishaRamon
Copy link
Collaborator Author

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.

@LishaRamon
Copy link
Collaborator Author

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.

@LishaRamon
Copy link
Collaborator Author

Table 1 Data

Table 1 Screenshot for Parallax + ProperMotion Data

@kelle
Copy link
Collaborator

kelle commented Dec 26, 2023

  • The separation in arcseconds (115.3) goes in the Companion Relationships table.
  • you could also add the T_eff, metallicty (Fe/H), gravity (logg), and mass to the Modeled Parameters table.

@kelle
Copy link
Collaborator

kelle commented Dec 26, 2023

Might as well also do the photometry. The relevant filters names from the Photometry Filters table are:

  • Gaia2.G
  • Gaia2.Gbp
  • Gaia2.Grp
  • VHS J = VISTA.J
  • VHS K = VISTA.K_s
  • WISE.W1
  • WISE.W2

data/gaia_j0452-36b.json Outdated Show resolved Hide resolved
"value": -1.4,
"value_error": 0.2,
"unit": "Fe/H",
"comments": null,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"comments": null,
"comments": "[Fe/H]",

data/gaia_j0452-36b.json Outdated Show resolved Hide resolved
data/gaia_j0452-36b.json Outdated Show resolved Hide resolved
data/gaia_j0452-36b.json Outdated Show resolved Hide resolved
data/gaia_j0452-36b.json Outdated Show resolved Hide resolved
data/gaia_j0452-36b.json Outdated Show resolved Hide resolved
@LishaRamon
Copy link
Collaborator Author

Tests for data and integrity are failing completely at the moment

@LishaRamon LishaRamon marked this pull request as ready for review December 26, 2023 22:19
Copy link
Collaborator

@kelle kelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@kelle kelle merged commit 49a6443 into SIMPLE-AstroDB:main Dec 27, 2023
1 check passed
@LishaRamon LishaRamon deleted the zhang19-gaia branch December 27, 2023 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix Zhang19 source names
2 participants