Skip to content

BUG: Update variable names for dosimeters#27

Merged
jklenzing merged 9 commits intodevelopfrom
bug/22_variable_names
Aug 26, 2025
Merged

BUG: Update variable names for dosimeters#27
jklenzing merged 9 commits intodevelopfrom
bug/22_variable_names

Conversation

@jklenzing
Copy link
Copy Markdown
Collaborator

@jklenzing jklenzing commented Jun 24, 2025

Description

Addresses #22

The label names for several parameters are not unique, leading to confusing info on the CDAweb search. This updates these variables to indicate which dosimeter is used.

Also updated usage of chained indices to improve compliance with pandas 3.0. Added flag to disable multi-day load tests, which is functionality not used in the current context of this software.

Also fixed the pkg_resources deprecation

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

By generating test files via scripts/netcdf_gen.py and checking the metadata in xarray.

Checklist:

  • Make sure you are merging into the develop (not main) branch
  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • Add a note to CHANGELOG.md, summarizing the changes

@jklenzing jklenzing changed the title Bug/22_variable_names BUG: Update variable names for dosimeters Jun 24, 2025
@jklenzing jklenzing requested a review from JonathonMSmith June 24, 2025 18:02
@jklenzing jklenzing merged commit b79db7c into develop Aug 26, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MAINT: use importlib instead of pkg_resources BUG: label name duplicated for bowtie fluxes

2 participants