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

ADD: Interface to the SPRAL linear solver #414

Closed
wants to merge 21 commits into from
Closed

ADD: Interface to the SPRAL linear solver #414

wants to merge 21 commits into from

Conversation

tasseff
Copy link
Contributor

@tasseff tasseff commented Oct 13, 2020

This would close #413 and close #335. We have a report written here, a fork of SPRAL with some minor edits here, and a fork of Ipopt with the interface here. One concern is ensuring the extra license text we have had to add to open-source our project is somehow incorporated into the master repository. Please let me know what I can do to clean up the interface, as I'm sure there's some stuff that I may have missed.

@CLAassistant
Copy link

CLAassistant commented Oct 13, 2020

CLA assistant check
All committers have signed the CLA.

@svigerske
Copy link
Member

Yeah, about that license text. So what is that necessary for?
The big source files with the interface that you add say "This code is published under the Eclipse Public License."

It may be confusing if there are two LICENSE files in the main directory, specifying two different licenses for the whole Ipopt, but nothing in the docu mention any dual-licensing and every source file says EPL.

If there is a problem in adding the interface under EPL, then adding it under BSD could be an option, I believe.

@tasseff
Copy link
Contributor Author

tasseff commented Oct 13, 2020

The files I used as templates when writing the new interface were indeed previous files in Ipopt licensed under the EPL, so it would probably make sense to keep them as EPL-licensed since they're derivative works. I'm not sure if it's necessary to include the new license text, but I'm not a legal expert. Is the EPL compatible with the BSD 3-ish license I linked? I imagine since the BSD 3 license is so permissive, this shouldn't be an issue, but let me try to obtain more guidance from the legal counsel at LANL.

@svigerske
Copy link
Member

If the interface files can be published under EPL, then the other parts, which are mainly smaller patches to existing files, should also not have to change license or get an additional different license. But I'm not a legal expert, too.

I managed to compile a Spral library (had to disable OpenMP and I don't have NVCC) and get that used by Ipopt.

So, I don't think that there is much left to do. The license thing needs to be clarified. I would leave out the COMPILE.md, as that is very specific to one setup and mostly duplicates what is in the documentation.

@tkralphs
Copy link
Member

I just want to point out that the CLA, which was signed for this PR, requires that any contribution to a project be under the same license.

@svigerske
Copy link
Member

@tasseff Did you get any feedback from LANL on the license issue? Could we merge this without adding LICENSE.md?

@tasseff
Copy link
Contributor Author

tasseff commented Nov 16, 2020

@svigerske, I still have not received any firm guidance. However, considering that (as you pointed out) the relevant modified files were originally published under the (more restrictive) EPL license, it would make sense that our LICENSE.md file would not be required in this pull request. Nonetheless, let me try to get another update from the relevant people at LANL to ensure this. I'm extremely sorry for the long delay.

@svigerske svigerske changed the base branch from master to devel November 18, 2020 11:29
@tasseff
Copy link
Contributor Author

tasseff commented Jan 27, 2021

I apologize, but I'm now being told by our intellectual property team that I need to remove our Ipopt fork until they've sorted through any license incompatibility issues. I am hoping this will be resolved soon. I will close this pull request for the time being.

@tasseff tasseff closed this Jan 27, 2021
@tasseff
Copy link
Contributor Author

tasseff commented Jan 28, 2021

Update: I've been given approval to release our contributions under the EPL-2.0. I will make a new fork and fix this pull request within the next few days.

@svigerske
Copy link
Member

Good news. Would be nice if you could base your work on the branch devel.

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.

Request: SPRAL support? Feature request: IPOPT support for GPU based linear solvers
4 participants