[Bonmin-tickets] [Bonmin] #22: patch for MUMPS integration

[Bonmin]

#22: patch for MUMPS integration
--------------------------+-------------------------------------------------
Reporter:  feh            |        Owner:  pbonami 
    Type:  enhancement    |       Status:  reopened
Priority:  major          |    Component:  Bonmin  
 Version:  release/0.1.3  |   Resolution:          
Keywords:                 |  
--------------------------+-------------------------------------------------
Comment (by andreasw):

 Replying to [comment:3 feh]:

 Hi Harald,

 > Hello Pierre, Andreas
 >
 > > If there is something to patch in Ipopt please submit a ticket to
 Ipopt.
 > How can I do that?

 Go to https://projects.coin-or.org/Ipopt and submit the ticket just as you
 did for Bonmin.  However, Mumps is very frequently used within Ipopt, and
 nobody has been complaining.  And I also cannot see a problem.

 >
 > > What is  the purpose of the indentations of the preprocessor lines?
 > Indentation helps to "guide the eye" through a cascade of ifdef - else -
 endif's. It is a simple visual aid to make the code self-explaining and to
 avoid spaghetti code.

 I think we usually do that.  I'm not sure however if every C preprocessor
 is happy with blanks preceding the '#' and not afterwards.  In any case,
 the file has changed in the meantime and I'm happy with the way it now
 looks ;-)

 >
 > > why would  you not delete the array a?
 > I delete it , but only once, on line 105. The original code deletes
 twice, once on line 100 and the second time on line 105, and this results
 in a double free plus program crash (Linux x86_64).

 I cannot reproduce this error, but maybe the issue is that you are using a
 relatively old version of Ipopt (since the latest Bonmin release is from
 last year).  In the current development version of Bonmin, I cannot see a
 problem related to memory in the Mumps interface.

 If you want, you can try the Bonmin/trunk development version.  Maybe
 there is another stable branch that you could use - Pierre, do you know?

 Definitely, the best will be if we make a new release, and we are now
 planning to do this relatively soon (within a few weeks).

 >
 > > Bonmin should work with Pardiso
 > Yes it does, but we cannot use Pardiso for several reasons, and we
 therefore chose MUMPS as an unencumbered alternative. Ubasel distributes
 Pardiso only in binary format, they do not support our target platform
 (currently Solaris 10, SPARC, 64bit), and their licensing scheme
 disqualifies the code for productive deployment.

 Makes sense,  I actually just mistyped; I meant to write "Bonmin should
 work with Mumps"

 Andreas

-- 
Ticket URL: <https://projects.coin-or.org/Bonmin/ticket/22#comment:4>
Bonmin <http://projects.coin-or.org/Bonmin>
Basic Open-source Nonlinear Mixed INteger programming