This broke because of a change I made for filtering WRF WL simulations. Derived indices weren't accounted for, and now they are, in a very hot fix way. All derived indices currently use temperature, so that is the psuedo-variable we're using to replace the derived index variable when looking through the catalog for valid WRF WL simulations.
Type of change
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
Definition of Done Checklist
Practical
[x] 80% unit test coverage
[x] Documentation
[ ] All functions/adjusted functions documented in the readthedocs.
[ ] Documentation is pushed
[x] Complex code commented
[x] Naming conventions followed
[ ] Helper functions hidden with _ before the name
[x] Context of function is clearly provided
[ ] Intent of function is provided
[ ] How to test, so that it is not siloed on scientists and anyone can review
[ ] Appropriate manual testing was completed
[x] Any notebooks known to utilize the affected functions are still working
[x] Linting completed and resolved
Conceptual
[x] Doesn't replicate existing functionality
[x] Aligns with general coding standard of existing functions
[x] Matches desired functinonality from users/scientists
Description of PR
Summary of changes and related issue
Making WRF WL derived vars work again.
Relevant motivation and context
This broke because of a change I made for filtering WRF WL simulations. Derived indices weren't accounted for, and now they are, in a very hot fix way. All derived indices currently use temperature, so that is the psuedo-variable we're using to replace the derived index variable when looking through the catalog for valid WRF WL simulations.
Type of change
Definition of Done Checklist
Practical
_before the nameConceptual