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

Added an instruction on how to fix ImportError for rmg binary #2327

Merged
merged 1 commit into from
Aug 11, 2022

Conversation

yunsiechung
Copy link
Contributor

@yunsiechung yunsiechung commented Aug 10, 2022

Motivation or Problem

When trying to run a RMG job using binary, ,multiple people encountered the following ImportError:

    Traceback (most recent call last):
      File "/PATH-TO-YOUR-ANACONDA/envs/rmg_env/bin/rmg.py", line 48, in <module>
        from rmgpy.rmg.main import RMG, initialize_log, process_profile_stats, make_profile_graph
      File "/PATH-TO-YOUR-ANACONDA/envs/rmg_env/lib/python3.7/site-packages/rmgpy/rmg/main.py", line 51, in <module>
        from cantera import ck2cti
      File "/PATH-TO-YOUR-ANACONDA/envs/rmg_env/lib/python3.7/site-packages/cantera/__init__.py", line 4, in <module>
        from ._cantera import *
    ImportError: libmkl_rt.so.2: cannot open shared object file: No such file or directory

This PR adds the instruction on how to solve this ImportError.

Description of Changes

I added the instruction on how to fix the error to the Binary Installation documentation. This issue can be solved by
the following line:
cp /PATH-TO-YOUR-ANACONDA/rmg_env/lib/libmkl_rt.so.1 /PATH-TO-YOUR-ANACONDA/rmg_env/lib/libmkl_rt.so.2

Testing

Three different people tested this method and it solved the ImportError.

Reviewer Tips

Try make documentation and then open RMG-Py/documentation/build/html/users/rmg/installation/anacondaUser.html to check the updated documentation.

@codecov
Copy link

codecov bot commented Aug 11, 2022

Codecov Report

Merging #2327 (7fb8eac) into main (b2c917f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2327   +/-   ##
=======================================
  Coverage   48.23%   48.23%           
=======================================
  Files         110      110           
  Lines       30507    30507           
  Branches     7950     7950           
=======================================
  Hits        14716    14716           
+ Misses      14245    14244    -1     
- Partials     1546     1547    +1     
Impacted Files Coverage Δ
arkane/kinetics.py 12.24% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@xiaoruiDong xiaoruiDong left a comment

Choose a reason for hiding this comment

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

I visualize the updated documentation, it looks great. Very clear instruction, thank you @yunsiechung !

@xiaoruiDong xiaoruiDong merged commit 8197792 into main Aug 11, 2022
@xiaoruiDong xiaoruiDong deleted the rmg_bin_installation branch August 11, 2022 15:28
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