Skip to content

Two patches for scons to run properly #946

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

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

Conversation

lijun99
Copy link
Contributor

@lijun99 lijun99 commented Apr 3, 2025

When using scons install method with python 13 and gcc/gfortan 13.3, I ran into two issues

  1. an error with getLineBand in denseoffsetsRead.F and estimateoffsetsRead.F. The solution is to use the respective getLineBand method for real or complex data, as defined in components/iscesys/ImageApi/DataAccessor/src/DataAccessorF.cpp.
    (these filed are not included in the CMake file, are they no longer in use?)
  2. warnings issued during PyCuAmpcor compilation, "Two environments with different actions...". Basically, all the functions are included in the pybind11 module and we don't need an additional lib (which causes the two different actions).

@jeffrey1102
Copy link

I am having problems with the scons installation, it gives me different errors and asks for license

@lijun99
Copy link
Contributor Author

lijun99 commented Apr 4, 2025

Which version are you using? The most recent releases should not concern license. You may try my steps to use either scons or cmake.

BTW, please post your issues on Issues section and post more details so that someone can better assist you.

@lijun99
Copy link
Contributor Author

lijun99 commented Apr 6, 2025

For those who want to get scons working before this PR is merged, please use my repo at first,

git remote add lijun99  https://github.com/lijun99/isce2.git
git fetch lijun99
git checkout patch

@jeffrey1102
Copy link

Your Repo works and scons ends the installation.
When you go back to your guide and making the isce2.mod, the command module load doesn’t work even if I install the (enviroment-module) the command is not found. I also did it with the bash but it also says that it doesn’t find isce in the path
how do you proceed after your RP?

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