Update of SNOPT version and use precompiled binary for libsnopt7 #16
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Author
What currently worksThe modified Makefile works and enables building Snopt.jl. Note that, in the case of dynamic linking, some dependencies may or may not be found at execution time. This may require to export a What currently doesn't workEven when all the dependencies are found, so far, I experienced issues with the precompiled library which throws a nice segfault. On MacOSUsing either the dynamic or static library compiled for MacOS with GCC, I currently get: This might be due to how the libs were compiled. I am thinking that the compilation may have used a I can only see On LinuxUsing the static lib compiled for Linux, this time the compilation throws an error: The error seems to indicate that Using the dynamic lib compiled for Linux, it compiles (of course) but then when I run, I get: One thing we can glean here is that this Julia (BYU supercomputer module Julia/v1.6) seems to be build on top of GCC10 and |
Author
|
FYI: @tylercritchfield and I tried this on his MAC and we observed the same behavior. |
Member
|
@DGCaprace should I merge this or hold? I know we were never able to get the supplied precompiled libraries to work. |
Author
|
The code in this branch is backward compatible, i.e. the default behavior is still to compile from the user-provided source code. So yes it could be merged, unless you want to leave the PR open until someone figures out the issue with the binaries. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Newer versions of SNOPT ship with precompiled libraries (static or dynamic), and not the source code anymore. This PR makes it possible for SNOPT.jl to either link to the provided dynamic library, or to include the content of the static library and repackage it for use with Julia.
The user can still copy the SNOPT source code files to
./deps/src, like before. However, if the source code is not available, the user can copy the binary to./deps/src. With a small modification of theMakefile(instructions provided in the README), themakecommand will be able to link to the dynamic library (libsnopt7.so/libsnopt7.dylib) or include the content of the static archive (libsnopt7.a).