Upgrade/add tests #11
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…-run-level-meta-data
…flow into upgrade/add-tests
|
Addresses #9 |
peter-madigan
added a commit
that referenced
this pull request
Jul 24, 2021
* Add mc truth associations to event builder * Skip timestamp correction for MC * Add charge hit->packet references * Update mc truth tracks dset name * Add mc truth datasets and refs to file * Add h5flow to environment (#8) * Upgrade/add run level meta data (#10) * Add h5flow to environment * Create lar_data, run_data, and units resources * Use units for unit conversions in charge2light * Fetch sample rate from run data in light/hit_finder * Fetch n samples from run_data in light/raw_event_generator * Add resources to docs * Increase h5flow version requirement * Fix typo * Fix imports * Update docstrings * Upgrade/add tests (#11) * Add h5flow to environment * Create lar_data, run_data, and units resources * Use units for unit conversions in charge2light * Fetch sample rate from run data in light/hit_finder * Fetch n samples from run_data in light/raw_event_generator * Add resources to docs * Increase h5flow version requirement * Fix typo * Fix imports * Update docstrings * Add workflow tests * Fix bugs * Fix test installation * Fix test install paths * Fix undefined names * Cache test data files * Speed up tests * Fix bug in light event generator * Move is_mc flag to RunData and bugfixes * Add caching to github test action * Empty commit * Fix workflow * Remove strict channel-priority * Fix pytest data caching * Add pytest cache to caches * Debug caching * Debug caching * Increment cache hash * Add refresh cache opt * Revert refresh cache * Fix cd * Name test steps * Debug pytest cache * Remove pytest caching * Fix workflow * Fix workflow (again) * Update default LAr temperature * Skip rollover correction for MC * Update docstrings * Increase charge event loop batch size * Get drift velocity from sim files directly * Add MC file to workflow tests * Update default temp to match MC * Update docstrings * Fix resource used for clock periods * Fix download url for mc file * Fix url order * Insert missing comma * Fix undefined * Revert grabbing drift velocity from MC file * Rework test download and execution order, remove extraneous tests
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Adds a test suite to run the standard workflow chain automatically