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

Add instrument name in LFOA default.yaml #195

Draft
wants to merge 1 commit into
base: dev_master
Choose a base branch
from

Conversation

teutoburg
Copy link
Contributor

One of the example notebooks in ScopeSim started producing ValueError: !OBS.instrument was not found in rc.__currsys__ for the LFOA. This key is usually found in all other instrument packages. I vaguely remember fixing this, not sure if it was in a local branch on my other machine maybe...

@teutoburg teutoburg added bug Something isn't working instrument-specific Limited to a certain IRDB instrument package instrument definition Addition or modification of instrument YAMLs labels Nov 10, 2024
@teutoburg teutoburg self-assigned this Nov 10, 2024
@teutoburg
Copy link
Contributor Author

Nevermind, the LFOA package seems to have more issues than this one keyword...

@teutoburg teutoburg marked this pull request as draft November 10, 2024 12:28
@teutoburg teutoburg removed the request for review from hugobuddel November 10, 2024 12:29
@teutoburg
Copy link
Contributor Author

Also, it does actually run, the error messages are printed but the exception is caught somewhere...

@hugobuddel
Copy link
Collaborator

Nevermind, the LFOA package seems to have more issues than this one keyword...

Also, it does actually run, the error messages are printed but the exception is caught somewhere...

Been there, got the T-shirt!

My conclusion was that we should ensure that all the other instruments that now rely on write_header() have a FitsExtraKeywords effect so we can just remove that entire function.

Not that I got around to that either. But the exception is annoying.

@teutoburg
Copy link
Contributor Author

My conclusion was that we should ensure that all the other instruments that now rely on write_header() have a FitsExtraKeywords effect so we can just remove that entire function.

Agree. Not sure if I get around to that anytime soon...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working instrument definition Addition or modification of instrument YAMLs instrument-specific Limited to a certain IRDB instrument package
Projects
Status: 🏗 In progress
Development

Successfully merging this pull request may close these issues.

2 participants