-
Notifications
You must be signed in to change notification settings - Fork 351
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
models: Switch to kornia AugmentationSequential #1979
Conversation
7214116
to
f4f4a81
Compare
Can you resolve the merge conflicts? |
f4f4a81
to
f28254c
Compare
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.
There are likely many lines in pyproject.toml
pytest ignores that could be removed now that we're bumping the min version. Same with all of the resame/align_corners stuff in the data modules. The former we could do now. The latter maybe in a separate PR that updates the data modules?
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.
Cool patch. What still has the necessity of the augmentation sequential torch geo wrap?
I think we're just waiting on the next release? #1978 |
2b9574b
to
800b6b2
Compare
Switch to kornia's
AugmentationSequential
inmodels
submodule