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.

Sign up for GitHub

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

Jump to bottom

Release v0.4.0 #25

Merged
merged 27 commits into from
May 13, 2024
Merged

Release v0.4.0 #25

merged 27 commits into from
May 13, 2024

Conversation

forrestfwilliams
Copy link
Contributor

No description provided.

AndrewPlayer3 and others added 27 commits April 29, 2024 13:57
@AndrewPlayer3
GPU Support
c737975
@AndrewPlayer3
Move CUDA installation to script for conditional install
ae59337
@AndrewPlayer3
removed path export
f697e19
@AndrewPlayer3
removed gpuarch code (wont work during docker build)
84255e8
@AndrewPlayer3
updated changelog
de98536
@AndrewPlayer3
change things to not use gpu by default
f8c4210
@forrestfwilliams
Merge branch 'develop' into nvidia
38a610e
@forrestfwilliams
combine cpu/gpu build proc
53f4a36
@forrestfwilliams
correct script name
1ca96e4
@forrestfwilliams
update arg logic
9e9b0ec
@AndrewPlayer3
changed from yes to true
2d9cb3e
@AndrewPlayer3
make gpu not default
1cd89b3
@AndrewPlayer3
add cuda paths using docker's ENV rather than .bashrc
2935dfd
@forrestfwilliams
use cuda base image for GPU
0cdbcec
@forrestfwilliams
Merge branch 'nvidia' into gpu
cb80b36
@forrestfwilliams
Merge pull request #24 from ASFHyP3/gpu
d1d89c0 
use cuda base image for GPU
@forrestfwilliams
swith to use of process_parallel for gpu case
cf1033c
@forrestfwilliams
fix unzip download option
2725a0f
@forrestfwilliams
fix small bugs
a6f6cf9
@forrestfwilliams
swith back to using sentinel_scene_multigpu
e1518ca
@forrestfwilliams
fix new bugs
3602d7d
@forrestfwilliams
fix small bugs
661cc2a
@forrestfwilliams
cleanup
131f30c
@forrestfwilliams
add ec2 instructions to readme
b6c9a11
@forrestfwilliams
fix numbering
c22fa6c
@forrestfwilliams
add ec2 language
d75eca6
@forrestfwilliams
Merge pull request #21 from ASFHyP3/nvidia
f40b5f3 
Add Support for CUDA
@forrestfwilliams forrestfwilliams added the minor Bump the minor version number of this project label May 13, 2024
@forrestfwilliams forrestfwilliams requested a review from a team as a code owner May 13, 2024 15:10
@github-actions GitHub Actions
Copy link

github-actions bot commented May 13, 2024 •
edited by forrestfwilliams
Loading

Developer checklist

  • Indicated the level of changes to this package by affixing one of these labels:
    • major -- Major changes to the API that may break current workflows
    • minor -- Minor changes to the API that do not break current workflows
    • patch -- Patches and bugfixes for the current version that do not break current workflows
    • bumpless -- Changes to documentation, CI/CD pipelines, etc. that don't affect the software's version
  • (If applicable) Updated the dependencies and indicated any downstream changes that are required
  • Added/updated documentation for these changes
  • Added/updated tests for these changes
  • Verified changes in test deployment and summarized results, e.g. in PR description or comments on the related issue(s)

Reviewer checklist

  • Have all dependencies been updated?
  • Is the level of changes labeled appropriately?
  • Are all the changes described appropriately in CHANGELOG.md?
  • Has the documentation been adequately updated?
  • Are the tests adequate?
  • Have the changes been verified in the test deployment?

@forrestfwilliams
Copy link
Contributor Author

Reviewer should follow the instruction laid out in the GPU EC2 section during testing.

AndrewPlayer3
AndrewPlayer3 approved these changes May 13, 2024
View reviewed changes
Copy link
Contributor

@AndrewPlayer3 AndrewPlayer3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@forrestfwilliams forrestfwilliams merged commit df2f1a5 into main May 13, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AndrewPlayer3 AndrewPlayer3 AndrewPlayer3 approved these changes

Assignees
No one assigned
Labels
minor Bump the minor version number of this project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@forrestfwilliams @AndrewPlayer3