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 #126

Closed
wants to merge 3 commits into from

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

@connoramoreno Could you trigger the CI again? Curious to see if this passes in light of the CI for #125 also failing

@paulromano
Copy link
Collaborator Author

Sorry @connoramoreno, can you trigger one more time? 🤞

@gonuke
Copy link
Member

gonuke commented Jul 11, 2024

This will also likely need to be PR'd from the upstream repo

@paulromano paulromano closed this Jul 11, 2024
@paulromano paulromano deleted the pystell-path branch October 8, 2024 18:56
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