Fix: resolve dimension errors and add RC2 explorer #111
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR addresses critical issues in the
ButlerRepoValidatorusage where coverage reporting failed due to missing governor dimensions (instruments). It also introduces new utility scripts for exploring the RC2 subset repository and cleans up syntax errors caused by non-breaking spaces in existing scripts.Key Changes
get_data_coveragecalls in test scripts to explicitly passinstruments=list(result.instruments). This resolves the "governor dimension" error when querying the Butler registry.explore_rc2_repository.pyto provide deep introspection into the RC2 subset, including detailed collection analysis and dataset type discovery.\xa0) from all test scripts that were causing PythonSyntaxErrors.datasets=[dataset_type_obj]) to ensure strict API compatibility.Testing
explore_rc2_repository.pyruns successfully against therc2_subset/SMALL_HSCpath.test_fixed_coverage_report.pyruns againstdemo_datawithout throwing dimension errors.ButlerRepoValidatorcorrectly identifiescalexpandsrcdatasets in the new RC2 environment.Context
Previously, the
get_data_coveragemethod would fail on repositories requiring strict dimension governance because theinstrumentdimension was not being propagated to the query. This PR ensures the discovered instruments are passed back to the coverage calculator.