Run ./tests/integration_tests/full_install.sh to make sure your test data gets installed. Please let me know what you think about the folder renaming in check_system_requirements.sh. This was needed as some datasets installed as test_data_<ds_name>_<version_number>_github.com and we needed them to be either test_data_<ds_name> or test_data_<ds_name>_<version_number>.
Summary
Portions setup/check_system_requirements.sh test data install code was outdated due
to the old instance of NextCloud being taken down recently. We've created a new instance
of NextCloud which hosts a large majority of the data used for testing GeoIPS, and this
required updating portions of the code which used the old links to the new link
locations.
As we were making those changes, we also found that setup/download_test_data.py
would not work for non .git hosted datsets. This is because a change was made to
setup/check_system_requirements.sh which sent the output of the raw repsonse from
requests.get to a logfile rather than piping it to tar extraction then a logfile. To
fix this, we added a conditional in setup/check_system_requirements.sh which
determined the source of the dataset, and either send the output of
setup/download_test_data.py directly to a logfile (.git-based), or piped it to
tar extraction then to a log file.
These changes address the new data locations and the bug introduced to
setup/check_system_requirements.sh.
Reviewer Checklist
Related Issues
fixes NRLMMD-GEOIPS/geoips#624 fixes NRLMMD-GEOIPS/geoips#625
Testing Instructions
Run
./tests/integration_tests/full_install.sh
to make sure your test data gets installed. Please let me know what you think about the folder renaming incheck_system_requirements.sh
. This was needed as some datasets installed astest_data_<ds_name>_<version_number>_github.com
and we needed them to be eithertest_data_<ds_name>
ortest_data_<ds_name>_<version_number>
.Summary
Portions
setup/check_system_requirements.sh
test data install code was outdated due to the old instance of NextCloud being taken down recently. We've created a new instance of NextCloud which hosts a large majority of the data used for testing GeoIPS, and this required updating portions of the code which used the old links to the new link locations.As we were making those changes, we also found that
setup/download_test_data.py
would not work for non.git
hosted datsets. This is because a change was made tosetup/check_system_requirements.sh
which sent the output of the raw repsonse fromrequests.get
to a logfile rather than piping it to tar extraction then a logfile. To fix this, we added a conditional insetup/check_system_requirements.sh
which determined the source of the dataset, and either send the output ofsetup/download_test_data.py
directly to a logfile (.git
-based), or piped it to tar extraction then to a log file.These changes address the new data locations and the bug introduced to
setup/check_system_requirements.sh
.Output