Skip to content

Adding Devasthal Optical Telescope - #158

Merged
hamogu merged 3 commits into
astropy:gh-pagesfrom
varghesereji:gh-pages
Jun 2, 2026
Merged

Adding Devasthal Optical Telescope#158
hamogu merged 3 commits into
astropy:gh-pagesfrom
varghesereji:gh-pages

Conversation

@varghesereji

Copy link
Copy Markdown

Description

This pull request is to address ...

Fixes #

@hamogu

hamogu commented Jun 2, 2026

Copy link
Copy Markdown
Member

Thanks for this pull request to add more data!
I do have a question about the coordinates though. When I input the coordinates into google maps (https://www.google.com/maps/@29.3620343,79.6859866,301m/data=!3m1!1e3!5m1!1e2?entry=ttu&g_ep=EgoyMDI2MDUyNy4wIKXMDSoASAFQAw%3D%3D) then it seems the coordinates are more centered on the ILMT than the DOT. That's probably just that the precision of the DOT is given only to full seconds in the website you referenced.
We can use that, but if it happens to fall on the ILMT, does it makes sense to set that as an alias?
If someone later comes with greater precision data, we could always split the two entries later. What do you think?

@hamogu

hamogu commented Jun 2, 2026

Copy link
Copy Markdown
Member

@varghesereji And I apologize that it took me so long to get to this. I usually reply to PRs within a week, but it seems that the notification email from GitHub ended up in my spam folder and so I didn't see your PR. I'm sorry!

…is near to 3.6 m DOT. Now, the exact coordinate of 3.6 m DOT, which is given in google maps is used.
@varghesereji

Copy link
Copy Markdown
Author

@hamogu Thanks for the suggestion. The coordinate I added before was actually given in the website. Now I have added the coordinates given in the google maps.
Thank you.

@hamogu

hamogu commented Jun 2, 2026

Copy link
Copy Markdown
Member

Thank you!

@hamogu
hamogu merged commit c07c19f into astropy:gh-pages Jun 2, 2026
2 checks passed
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.

2 participants