-
Notifications
You must be signed in to change notification settings - Fork 0
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
Integrate exposure time calculator into the plugin? #66
Comments
I agree that would be a very desirable feature for the TOM facilities module to have in general. We took a preliminary look at the LT's ETC code and it appears to be a purely JS application and doesn't offer an API that we could query. In principle, we could integrate this code with the LT module in the Toolkit but we'd prefer to query a service offered by the observatory because that would make it easier for them to update, every time their re-coat the mirrors for instance. This led to a wider discussion about how we can best implement such a service from an AEON perspective. Given the very wide range of instruments at all the observatories, a fully general-purpose solution may potentially become very complex. Rather than let the perfect be the enemy of the good, I think the first step is for us to start a conversation with the LT folks and explore whether they can add an API for their tool. Similarly, we'd like to update the one for LCO. |
Fair point. An API would be the probably the best solution and I'm sure this would work for LCO, and also probably for LT, if we help them re-write their webpage (no people available at the moment, as far as I know). Some telescopes we work with in BHTOM do not even have their own webpages, let alone an Exposure Time Calculator. I have been collaborating with a group of students to develop a generic universal Exposure Time Calculator that utilizes all the data we have collected for a given telescope to provide an estimated exposure time. However, this would need to be integrated into the telescope plugin. What do you think about offering both solutions? We could create a simple built-in exposure time calculator for each observatory, but if they provide an API, that could override the built-in version. |
Here is a re-write of the JS code I made (with some help of ChatGPT), we could use this in the plugin as a good start.
|
Chris Copperwheat responded from the LT to explain that while they'd like to implement an API, this is out of scope for the foreseeable future. So it looks like embedding the ETC code is the way forward in the short term. Thanks for the refactored code @wyrzykow. I'm increasing the priority of this issue and putting it back in triage for the TOM Team to discuss when we plan the next sprint. |
Thanks for increasing the priority, @rachel3834 ! I have tested this code in my simple bhtom3, https://github.com/BHTOM-Team/bhtom3 |
LT has a very nice exp.time calculator on the web: https://telescope.livjm.ac.uk/TelInst/calc/
If that was part of the AEON plugin, it would help automatise the requests from TOMs.
I asked LT for the source code.
The text was updated successfully, but these errors were encountered: