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

Update sites.json with CHIME and Galt telescope locations #108

Merged
merged 4 commits into from
Oct 11, 2021

Conversation

aelanman
Copy link
Contributor

Description

This pull request is to address the lack of positions of the CHIME telescope and the Dominion Radio Astronomical Observatory (marked by the position of the John Galt telescope) in known sites of astropy.

Fixes #

Copy link
Member

@hamogu hamogu left a comment

Choose a reason for hiding this comment

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

Because we merged another pull request which also appended to the bottom on the file first, git had a merge conflict. I've taken the freedom to rebase this, so that you don't have to bother - it's not your fault that this has been sitting for so long.

In [1]: from astropy.coordinates.tests import test_sites 
   ...: test_sites.check_builtin_matches_remote('coordinates/sites.json')

pass and I verified the coordinates in the 2018 paper and compared to an image on Google maps.

Thanks a lot for helping us to make astropy better!

@hamogu hamogu merged commit c61db80 into astropy:gh-pages Oct 11, 2021
@aelanman
Copy link
Contributor Author

Thank you @hamogu !

@aelanman aelanman deleted the patch-1 branch October 11, 2021 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants