-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: update URLS to download Prithvi Segmentation Model weights and config #11
Conversation
I now get the following output when I run
Please advise how to setup login credentials for the tests so that I can avoid The logic to cleanup files in |
@ZeLynxy let's modify the outdir in |
When creating the
|
The others are low priority and will get them fixed
…On Wed, 18 Dec 2024, 9:19 am Jeroen Van Goey, ***@***.***> wrote:
When creating the /.netrc file, the test_chip_creator and
test_download_hls_tile_with_retry tests now succeed. The remaining tests
are:
FAILED tests/data_tests/test_chip_creator.py::test_missing_flags_raises_error - Failed: DID NOT RAISE <class 'absl.app.UsageError'>
FAILED tests/data_tests/test_hls_utils.py::test_download_hls_tile - PermissionError: [Errno 1] Operation not permitted: '/tmp/jumpcloud-agent-updater.lock'
—
Reply to this email directly, view it on GitHub
<#11 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADU2QQ6EAREL5QSUIOCZIET2GEVY3AVCNFSM6AAAAABTXXC7RGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJQGY2TCNBQGY>
.
You are receiving this because your review was requested.Message ID:
***@***.***>
|
@BioGeek Can you replace the failing tests with the updated version we have in a yet to merge PR?
|
The name of the config and the weights of the Prithvi model on Huggingface have changed.
This PR updates the URLs. The commit history also shows that the weights have changed to a new format and the inference code has been updated. I haven't tested yet how this impacts the InstaGeo code.