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

print CUTEst environment variables #226

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

print CUTEst environment variables #226

wants to merge 1 commit into from

Conversation

dpo
Copy link
Member

@dpo dpo commented May 11, 2020

closes #225

@dpo
Copy link
Member Author

dpo commented May 11, 2020

Doesn't work out of the box on macOS unfortunately, even after setting DYLD_FALLBACK_LIBRARY_PATH:

$ sifdecoder ROSENBR
dyld: Library not loaded: @rpath/libgfortran.5.dylib
  Referenced from: /Users/dpo/.julia/artifacts/83e267cb7d8652c72b9dac8674d69220810103c2/libexec/SIFDecode-2.0.3/objects/mac64.
osx.gfo/double/run_sifdecode
  Reason: image not found
/Users/dpo/.julia/artifacts/83e267cb7d8652c72b9dac8674d69220810103c2/libexec/SIFDecode-2.0.3/bin/sifdecoder: line 459: 48619 A
bort trap: 6           $DECODER
 ERROR: Error exit from decoding stage. terminating execution.

@codecov
Copy link

codecov bot commented May 11, 2020

Codecov Report

Merging #226 (4743a8d) into main (4743a8d) will not change coverage.
The diff coverage is n/a.

❗ Current head 4743a8d differs from pull request most recent head ad25f01. Consider uploading reports for the commit ad25f01 to get more accurate results

@@           Coverage Diff           @@
##             main     #226   +/-   ##
=======================================
  Coverage   87.07%   87.07%           
=======================================
  Files           5        5           
  Lines         712      712           
=======================================
  Hits          620      620           
  Misses         92       92           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@dpo
Copy link
Member Author

dpo commented May 11, 2020

The updated version works for me on macOS.

@abelsiqueira Does it work for you on Linux?

@dpo
Copy link
Member Author

dpo commented May 11, 2020

There's still a caveat that running runcutest tries to update the CUTEst library, which results in a lot of "permission denied" messages being printed. It's inconsequential, but annoying.

@abelsiqueira
Copy link
Member

Unfortunately doesn't work for me either:

/home/abel/.julia/artifacts/c990de2425d3a17849e2f6bbe47decf0f81036d8/libexec/SIFDecode-2.0.3/objects/pc64.lnx.gfo/double/run_sifdecode: error while loading shared libraries: libgfortran.so.4: cannot open shared object file: No such file or directory

@dpo
Copy link
Member Author

dpo commented May 11, 2020

@abelsiqueira
Copy link
Member

Almost worked, needs a ; at the end: ... bash -c "..."; }.
With that change, run_sifdecoder ROSENBR worked.

@dpo dpo force-pushed the show_bashrc branch 2 times, most recently from a28d689 to 51c0cbf Compare May 12, 2020 00:20
@dpo
Copy link
Member Author

dpo commented May 12, 2020

Should be good now.

@abelsiqueira
Copy link
Member

We need a similar one for runcutest. I tried adding a

run_cutest() { ... source $CUTEST/bin/runcutest $*"; }

but it failed when I ran run_cutest -p genc -D ROSENBR with a bunch of these:

ar: /home/abel/packages/julias/julia-1.4-latest/bin/../lib/julia/libstdc++.so.6: version `GLIBCXX_3.4.26' not found (required by /usr/lib/libLLVM-9.so)

The sifdecoder part ran correctly.

PS. The $* instead of $@ was necessary to parse the arguments correctly.

@dpo
Copy link
Member Author

dpo commented May 12, 2020

Does this work though

run_sifdecoder ROSENBR
run_cutest -p genc

@abelsiqueira
Copy link
Member

No, it has the same error. The final line is Error opening file OUTSDIF.d., which I guess it's because the compilation failed.

@dpo
Copy link
Member Author

dpo commented May 12, 2020

Hm. It works for me on macOS. That seems to indicate an issue with the JLL... :-(

@dpo
Copy link
Member Author

dpo commented Aug 8, 2022

there are conflict markers in these changes.

@tmigot
Copy link
Member

tmigot commented Aug 8, 2022

Thanks, I just noticed too.
I rebased this just in case, if this is still of interest ?

@dpo
Copy link
Member Author

dpo commented Aug 8, 2022

It is, but what's a bit weird is that runcutest modifies the artifacts. It works, but it's not ideal. I should modify CUTEst itself so it only modifies files in ~/.cutest (say), and leaves the compiled object files alone.

@amontoison
Copy link
Member

Can I close this PR?
All the environment variables have been removed because they not needed anymore with the new artifact SIFDecode_jll.jl.

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.

Return environment so CUTEst can be used outside julia
4 participants