Skip to content
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

Add pad_size arg to OSCDDataModule #341

Merged
merged 5 commits into from
Jan 1, 2022

Conversation

isaaccorley
Copy link
Collaborator

This PR adds:

  • pad_size argument to the OSCDDataModule. Previously the pad size for the PadTo transform used in the val/test steps was hardcoded to (1280, 1280).
  • Updated default OSCD configs to use (64, 64) pad size which is the smallest I could get a U-Net to take due to downsampling. This should reduce the memory consumption used when testing the trainers/datamodules.

@github-actions github-actions bot added the datamodules PyTorch Lightning datamodules label Dec 31, 2021
@isaaccorley isaaccorley self-assigned this Dec 31, 2021
@isaaccorley isaaccorley added the testing Continuous integration testing label Dec 31, 2021
@isaaccorley isaaccorley added this to the 0.2.0 milestone Dec 31, 2021
@github-actions github-actions bot removed the testing Continuous integration testing label Dec 31, 2021
calebrob6
calebrob6 previously approved these changes Jan 1, 2022
@calebrob6 calebrob6 enabled auto-merge (squash) January 1, 2022 00:25
@calebrob6
Copy link
Member

Merging this because mypy issues are unrelated.

@calebrob6 calebrob6 merged commit 7347efe into microsoft:main Jan 1, 2022
@adamjstewart adamjstewart added utilities Utilities for working with geospatial data and removed utilities Utilities for working with geospatial data labels Jan 2, 2022
@isaaccorley isaaccorley deleted the datasets/oscd-pad-size branch January 7, 2022 23:54
yichiac pushed a commit to yichiac/torchgeo that referenced this pull request Apr 29, 2023
* add pad_size arg to OSCDDataModule, update configs

* add EOL to configs

* hacking config files

* Update oscd_all.yaml

* Update oscd_rgb.yaml

Co-authored-by: Caleb Robinson <calebrob6@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
datamodules PyTorch Lightning datamodules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants