Skip to content

Fix IndexError due to missing key 'observer_zenith' in prebuilt LUT.#688

Merged
pgbrodrick merged 7 commits into
isofit:devfrom
unbohn:bugfix/missing_observer_zenith_key
May 12, 2025
Merged

Fix IndexError due to missing key 'observer_zenith' in prebuilt LUT.#688
pgbrodrick merged 7 commits into
isofit:devfrom
unbohn:bugfix/missing_observer_zenith_key

Conversation

@unbohn

@unbohn unbohn commented May 12, 2025

Copy link
Copy Markdown
Collaborator

If apply_oe is called with a prebuilt LUT that does not include the key 'observer_zenith', ISOFIT currently fails during the template construction since the key is expected to be present in order to apply the conversion to MODTRAN convention if needed. This PR moves those lines to a try-except block to avoid the error.

@unbohn unbohn added the bugfix Fixing an error label May 12, 2025
@pgbrodrick pgbrodrick merged commit 217b3c8 into isofit:dev May 12, 2025
17 checks passed
@unbohn unbohn deleted the bugfix/missing_observer_zenith_key branch May 13, 2025 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixing an error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants