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

Update Ubuntu script to match version restrictions of current PyNE #65

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

gonuke
Copy link
Contributor

@gonuke gonuke commented Feb 17, 2022

In preparing for the PyNE 0.7.6 release, we discovered some changes in the python installation world that may break our scripts. Hopefully this provides a good temporary fix.

Testing in the ubuntu script before modifying the docker files....

Hoping to fix #64

@ahmadMIbrahim
Copy link

I do not see the PyNE PYTHONPATH defined in gonuke:update_ubuntu_build.

When I run the script without defining my own PYTHONPATH, I encounter the same error I used to have while running the tests file, travis-run-tests.bash. This error is:
ModuleNotFoundError: No module named 'pyne'.

I tried to add my PYTHONPATH to point to the folders that include .py files in the new/fresh installations that the script has already made. On my system, these are:
PYTHONPATH=$HOME/anaconda3/lib/python3.7/site-packages/pip/_vendor/:$PYTHONPATH
PYTHONPATH=$HOME/.local/lib/python3.7/site-packages/:$PYTHONPATH
PYTHONPATH=$HOME/opt/moab/lib/python3.7/site-packages/pymoab-5.1.0-py3.7-linux-x86_64.egg/:$PYTHONPATH
PYTHONPATH=$HOME/opt/openmc/:$PYTHONPATH

I am getting much better response than before. I see most of the tests passing, but some did not pass. Some of the errors I am getting are related to OpenMC such as "NameError: name 'openmc' is not defined." It is worth mentioning that I had commented the "exit $status" line in travis-run-tests.bash to allow the script to run until the end.

Thank you very much. I feel I am in better state now.

Copy link
Contributor

@ngranda ngranda left a comment

Choose a reason for hiding this comment

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

LTGTM. My only comment is to remove a duplicate line

ubuntu.sh Outdated Show resolved Hide resolved
gonuke and others added 2 commits May 18, 2022 14:03
remove extra line

Co-authored-by: Nancy Granda <[email protected]>
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.

Installation script does not define PyNE PYTHON path and other errors probably caused by conflicts in versions
3 participants