-
Notifications
You must be signed in to change notification settings - Fork 6
Torchgeo mlm #102
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
Torchgeo mlm #102
Conversation
… extract dtype and mapping
…ion if no weights provided
Following previous comment on crim-ca/mlm-extension#41 , here's my up to date change. Let me know if other fixes are needed. Thanks |
pyproject.toml
Outdated
@@ -13,7 +13,7 @@ authors = [ | |||
{name = "Francis Charette-Migneault", email = "francis.charette-migneault@crim.ca"}, | |||
] | |||
license = {text = "Apache Software License 2.0"} | |||
requires-python = "<4.0,>=3.10" | |||
requires-python = "<4.0,>=3.11" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Self Note
Merge other features and release MLM 1.5.0 & stac_model
0.4.0 before merging this to avoid the potentially breaking change for users employing the package.
CHANGELOG.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Self-note
Make sure the items are ordered correctly considering the intermediate release of MLM / stac_model
.
@Nazim-crim The other failures for Python 3.10 should be handled properly when merging with #97. |
Temp fix for STAC validation is to used
|
See Line 420 in 27bc757
|
Description
from_torch
function intorch/
a utility to programmatically generate a STAC Item with the Machine Learning Model (MLM) Extension from a PyTorch nn.Module.Related Issue
Type of Change
Checklist
CONTRIBUTING.md
guide;make check
;Google
format for all the methods and classes that I used.