-
Notifications
You must be signed in to change notification settings - Fork 136
Merge recent commits into stable #1466
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
base: stable
Are you sure you want to change the base?
Conversation
Update default reviewers following recent repository changes.
…of_environments_for_publishing_artifacts Update workflows that publish artifacts to use specific configured environments
…isting-scans-to-git-hub-actions Implement Python Static Analysis Tools
remove repeat of kNDVI in bandindices.py docs
|
@LaurenSchenk1 , sorry I didn't mean to remove you! 😅 I was trying to add @GA-XavierGonzalez and @stephenohairga to the reviewers as I saw them contributing to some of the latest changes, and I must have mis-clicked! |
GL-S
left a comment
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.
I think a new release to PyPi is quite important, hence I support this merge. Especially for the xr_random_sampling function with the new random_seed argument, which is crucial for reproducibility
…-linters Workflows/configure linters
Planning on doing a new release of
dea-toolsto pypi so functionality added last year is more widely available. Updating stable with any recent changes to the repo before tagging a release.