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

Assume pystell is installed properly in Python environment #128

Merged
merged 7 commits into from
Jul 16, 2024
Merged

Conversation

paulromano
Copy link
Collaborator

Right now, imports to pystell are of the form import src.pystell. This appears to be the case because pystell uses a src directory as the root for its Python package. So, if you add the root directory of the pystell git repository to your PYTHONPATH, the imports work correctly. However, pystell has a pyproject.toml file that should be used rather than manipulating the PYTHONPATH environment variable. Specifically, pystell can be installed by simply running pip install git+https://github.com/aaroncbader/pystell_uw.git.

@paulromano
Copy link
Collaborator Author

CI passing on this one now ✔️

@paulromano paulromano requested a review from gonuke July 11, 2024 16:23
@Edgar-21
Copy link
Contributor

I didn't realize that pip could be used like that, nice addition. It got me thinking that it should be possible to just include pystell in the environment.yml file, tested it out and seems to be working. I PR'ed ir to this branch here #131

Copy link
Member

@gonuke gonuke left a comment

Choose a reason for hiding this comment

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

Thanks @paulromano (& @Edgar-21 ) for making this all cleaner

@gonuke gonuke merged commit 837de9d into main Jul 16, 2024
3 checks passed
@connoramoreno connoramoreno deleted the pystell-path branch September 9, 2024 14:51
connoramoreno pushed a commit that referenced this pull request Sep 10, 2024
Assume pystell is installed properly in Python environment
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.

3 participants