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

Include cloudpathlib as requirement in meta.yaml #1166

Closed
wants to merge 1 commit into from

Conversation

craabreu
Copy link

@craabreu craabreu commented Mar 1, 2023

Description

Includes package cloudpathlib as dependency in conda recipe.

Motivation and context

A cloudpathlib class is used in app/relative_setup.py

from cloudpathlib import AnyPath

How has this been tested?

It has not been tested. The mentioned package had to be installed manually.

Change log

Included conda-forge package cloudpathlib as requirement.

@mikemhenry
Copy link
Contributor

👋 Thanks for this PR @craabreu
We are actually removing this file in this PR: #1159 Since we use conda-forge for our packaging: https://github.com/conda-forge/perses-feedstock

Sorry for the confusion! If you are using peres from our main branch, be warned that it is unreleased and unstable. See this file if you are looking for the development env. It is what we use for CI so the pins are the minimum versions (and a smaller s3 only cloudpathlib package).

If you have any problems, please feel free to open an issue on our issue tracker here: https://github.com/choderalab/perses/issues/new/choose

@mikemhenry mikemhenry closed this Mar 1, 2023
@craabreu
Copy link
Author

craabreu commented Mar 1, 2023

@mikemhenry Thanks for pointing out the yaml file. I'm sorry I mixed things up (development vs. conda-forge versions).

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