-
Notifications
You must be signed in to change notification settings - Fork 25
ingest CATWISE #650
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
ingest CATWISE #650
Conversation
|
|
Please include the output of the counters in the script as comments. |
|
|
I added comments to each of my ingests, edited pytest case checking propermotion references for maro 21, edited versions table |
|
I also made a new pytest that checks for adopted propermotion refs |
kelle
left a comment
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.
I think this is good to go. Great work!
Versions table looks slightly different than the other pr because this versions table had one more update than the previous
Link to relevant issue: Closes #614
For data ingests:
[ ] includes script used for ingest
[ ] includes modified JSON files
[ ] Add new tests
[ ] Update the Versions table