Opened 9 years ago

Closed 9 years ago

#8171 closed enhancement (fixed)

New Cbc spkg with Cplex support

Reported by: ncohen Owned by: tbd
Priority: major Milestone: sage-4.4
Component: packages: optional Keywords:
Cc: malb, schilly Merged in:
Authors: Nathann Cohen Reviewers: Dmitrii Pasechnik
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by ncohen)

This new spkg for Cbc includes several new lines in spkg-install to activate CPLEX support in the Coin Library.

It is to be found at : ~ncohen/cbc-2.3.p2.spkg

This functionality can be used with the corresponding patch #8172

Nathann

Change History (10)

comment:1 Changed 9 years ago by ncohen

  • Description modified (diff)
  • Status changed from new to needs_review

comment:2 Changed 9 years ago by ncohen

I just updated the spkg to make it support multithreading through Cbc !

Nathann

comment:3 follow-up: Changed 9 years ago by drkirkby

  • Status changed from needs_review to needs_work

Since this is new, you need to state whether it is intended to go into experimental or optional. Also, since this is new, you should remove the .p2 from the spkg name and instead call it cbc-2.3.spkg.

If you update this, before it is committed to Sage, just replace it, or provide a new link. The patch number increments each time a new version is added to Sage - not each time you change your version.

Dave

comment:4 in reply to: ↑ 3 Changed 9 years ago by malb

Sorry, there is some confusion here.

Replying to drkirkby:

Since this is new, you need to state whether it is intended to go into experimental or optional.

The package is intended for optional, because that's where CBC is right now.

Also, since this is new, you should remove the .p2 from the spkg name and instead call it cbc-2.3.spkg.

As hinted above the SPKG is indeed not new but an update. The CPLEX support in the ticket #8172 is new but *this* ticket only updates the CBC SPKG to work with the new interface. A true update. There is and never will be a CPLEX SPKG because CPLEX is proprietary.

Hope that clarifies the situation somewhat.

comment:5 Changed 9 years ago by drkirkby

Yes, it does clarify this. I was under the impression this was a new package, rather than an update to a pre-existing one.

I've stuck it back to needs review. I'm personally unable to review it, as it is outside my level of expertese.

comment:6 Changed 9 years ago by drkirkby

  • Status changed from needs_work to needs_review

comment:7 Changed 9 years ago by malb

Harald, can you take a look at the SPKG? I tried it and it works but someone needs to check for the basics (hg status, SPKG.txt, etc.) and I'm busy for the next few days.

comment:8 follow-up: Changed 9 years ago by ncohen

Finally ..... This package has been tested on t2 (Solaris). It compiles and runs ! :-)

As soon as this and #8172 are in Sage, it will be possible to work on the inclusion of SCIP !

Nathann

comment:9 in reply to: ↑ 8 Changed 9 years ago by dimpase

  • Status changed from needs_review to positive_review

Replying to ncohen:

Finally ..... This package has been tested on t2 (Solaris). It compiles and runs ! :-)

As soon as this and #8172 are in Sage, it will be possible to work on the inclusion of SCIP !

Nathann

Positive review, all works, good, but please take care of the update I posted on #8172

Thanks, Dima

comment:10 Changed 9 years ago by jhpalmieri

  • Authors set to Nathann Cohen
  • Resolution set to fixed
  • Reviewers set to Dmitrii Pasechnik
  • Status changed from positive_review to closed

Merged 2010/04/20.

Note: See TracTickets for help on using tickets.