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

release notes for 0.10.2 #1208

Merged
merged 2 commits into from
Jun 28, 2023
Merged

release notes for 0.10.2 #1208

merged 2 commits into from
Jun 28, 2023

Conversation

mikemhenry
Copy link
Contributor

Release PR for 0.10.2

@mikemhenry mikemhenry requested a review from jchodera June 22, 2023 23:39
@codecov
Copy link

codecov bot commented Jun 22, 2023

Codecov Report

Merging #1208 (e75f7cf) into main (206a6b1) will decrease coverage by 2.67%.
The diff coverage is n/a.

@mikemhenry
Copy link
Contributor Author

@jchodera can you take a look at this when you get a chance? Thanks!

Copy link
Member

@jchodera jchodera left a comment

Choose a reason for hiding this comment

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

This looks good after the comments are addressed!

Enhancements
^^^^^^^^^^^^

* Added upport for reading input files (ex yaml, sdf, pdbs) from AWS, GCP, and Azure. See the documentation for [cloudpathlib](https://cloudpathlib.drivendata.org/stable/authentication/) for how to setup authentication. Currently only reading the yaml from S3 is unit tested (ie `perses-cli --yaml s3://perses-testing/template_s3.yaml`), but other cloud providers and input files should work. Please report any issues on our issue tracker! by @mikemhenry in https://github.com/choderalab/perses/pull/1073
Copy link
Member

Choose a reason for hiding this comment

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

"upport" -> "support"

Do we document how you specify AWS, GCP, and Azure files in the CLI or docs or somewhere?
Also, do all input files work this way, including those paths specified in the YAML? Or just --yaml?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

All input files, I mention --yaml since that is the only one we test. I will add a note to see cloud documentation on the format of the URI.

docs/changelog.rst Outdated Show resolved Hide resolved
docs/changelog.rst Outdated Show resolved Hide resolved
docs/changelog.rst Outdated Show resolved Hide resolved
@mikemhenry
Copy link
Contributor Author

ooof, forgot to run this through my spellcheck! 🙈 Will address everything else 😄

@mikemhenry mikemhenry requested a review from jchodera June 27, 2023 22:46
Copy link
Member

@jchodera jchodera 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, thanks!

@mikemhenry mikemhenry added this pull request to the merge queue Jun 28, 2023
Merged via the queue into main with commit 06bbd50 Jun 28, 2023
6 checks passed
@mikemhenry mikemhenry deleted the release/0.10.2 branch June 28, 2023 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants