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

This is the pull Request for Issue 21 (Updating Readme with specific MacOS instructions) #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aneeshs1729
Copy link
Contributor

This is for Issue21; Updated the MacOS Makefile instructions

@aneeshs1729 aneeshs1729 changed the title Updated Readme.md This is the pull Request for Issue 21 (Updating Readme with specific MacOS instructions) Dec 6, 2024
This is for Issue21; Updated the MacOS Makefile instructions
@valeriabarra valeriabarra linked an issue Dec 6, 2024 that may be closed by this pull request
Comment on lines +71 to +72
To use MOLE (C++ version), you need to have _Armadillo C++_ <http://arma.sourceforge.net>, _SuperLU_
<https://portal.nersc.gov/project/sparse/superlu>, and _OpenBLAS_ <https://www.openblas.net> installed on your computer.
Copy link
Collaborator

Choose a reason for hiding this comment

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

We can try to avoid this redundancy (ref. 3: Installation (Linux))


On Macs, you can install superlu using the following steps:
a. Open a terminal
b. (type) brew install superlu
Copy link
Collaborator

Choose a reason for hiding this comment

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

A code markdown might suit better
i.e brew install superlu

Comment on lines +89 to +93
ifdef SUPERLU
LIBS += -L$(SUPERLU)/lib -Wl,-rpath,$(SUPERLU)/lib -lsuperlu -L/opt/homebrew/Cellar/superlu/7.0.0/lib
else
LIBS += -lsuperlu -L/opt/homebrew/Cellar/superlu/7.0.0/lib
endif
Copy link
Collaborator

Choose a reason for hiding this comment

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

ifdef SUPERLU
LIBS += -L$(SUPERLU)/lib -Wl,-rpath,$(SUPERLU)/lib -lsuperlu -L/opt/homebrew/Cellar/superlu/7.0.0/lib
else
LIBS += -lsuperlu -L/opt/homebrew/Cellar/superlu/7.0.0/lib
endif

else
LIBS += -lsuperlu -L/opt/homebrew/Cellar/superlu/7.0.0/lib
endif
3. Add the path to openblas namely -L/opt/homebrew/opt/openblas/lib as shown below
Copy link
Collaborator

Choose a reason for hiding this comment

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

-L/opt/homebrew/opt/openblas/lib

3. Add the path to openblas namely -L/opt/homebrew/opt/openblas/lib as shown below
LIBS = -L$(MOLE) -lmole -lopenblas -L/opt/homebrew/opt/openblas/lib

As mentioned in the Linux instllation above, do the following steps as usual -
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please inherit a summary or a reference here so that users don't have to sift through all the Linux installation steps.

you may encounter difficulties in compiling. Follow the instructions given in section 3.0 above. If you are unable to compile, use the following command

To compile the source code in mole/src/mole_C++/
- make CXX=g++-14 CC=gcc-14
Copy link
Collaborator

Choose a reason for hiding this comment

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

make CXX=g++-14 CC=gcc-14

Another quriky change is the usage of makefiles on mac. Depending on the version of the software packages installed on your mac,
you may encounter difficulties in compiling. Follow the instructions given in section 3.0 above. If you are unable to compile, use the following command

To compile the source code in mole/src/mole_C++/
Copy link
Collaborator

Choose a reason for hiding this comment

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

To compile the source code in src/cpp or examples/cpp {*or src/src/mole_C++}:
make CXX=g++-14 CC=gcc-14

  • Please verify the path here, as I am not entirely sure about the default path for the example test case.

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.

Improved documentation for mac OS makefile.
2 participants