You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is a meta ticket to cover cases of refactoring that could ease automating of testing. These may not be viable in the time we have, but to at least keep track of these decisions for the near term versus the long term.
python/resampling/resampling_hads.py
check `enforce_date_dropping.py
refactor python/resampling/check_calendar.py, which looks like a test as mentioned in HADs 360 day calendar conversion #32 but isn't incorporated in the pytest configuration. It does require mounting an external drive but doesn't import or call any code from python/resampling/resampling_hads.py. Shifting an equivalent of this to the tests/ folder may be helpful
Whether to refactor with tests
python/debiasing/pre_processing.py
Checking the preprocess_data function may require refactoring
Whether to refactor with tests
The text was updated successfully, but these errors were encountered:
griff-rees
changed the title
Python refactor for testing
Add tests as we refactor
Feb 5, 2024
griff-rees
changed the title
Add tests as we refactor
Triage whether to add tests and refactor
Feb 5, 2024
This is a meta ticket to cover cases of refactoring that could ease automating of testing. These may not be viable in the time we have, but to at least keep track of these decisions for the near term versus the long term.
python/resampling/resampling_hads.py
python/resampling/check_calendar.py
, which looks like a test as mentioned in HADs 360 day calendar conversion #32 but isn't incorporated in thepytest
configuration. It does require mounting an external drive but doesn't import or call any code frompython/resampling/resampling_hads.py
. Shifting an equivalent of this to thetests/
folder may be helpfulpython/debiasing/pre_processing.py
preprocess_data
function may require refactoringThe text was updated successfully, but these errors were encountered: