Consolidate test assets #61
Labels
Comments
|
We should also use tmp_dir throughout rather than overwriting actual test assets (as we currently do in |
ejm714
added a commit
that referenced
this issue
Mar 3, 2025
…each test run; note: this is a workaround for better test asset management tracked in #61
ejm714
added a commit
that referenced
this issue
Mar 4, 2025
* set minimum scikit-learn version and use root_mean_squared_error * sentinel-2 data has been reprocessed leading to changes: part 1 -- test assets * sentinel-2 data has been reprocessed leading to changes: part 2 -- expected features * avoid writing out git hash to config yaml so it does not change with each test run; note: this is a workaround for better test asset management tracked in #61 * be more flexible with scikit-learn * override save path in experiment test to use tmp_path * remove ceiling for matplotlib and remove duplicate clean command * update test assets with latest matplotlib * address pandas warnings * fix test from writing out to assets * latest hash * make assets should be runnable again if it failed halfway through
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now we have test assets for
evaluate_data.csv,train_data.csv, andpredict_data.csv. See if we can consolidate into fewer. Eg. We may be able to have a longer file (~20 rows) and then divvy it up into train/test files as neededThe text was updated successfully, but these errors were encountered: