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 )
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
- Description modified (diff)
- Status changed from new to needs_review
comment:2 Changed 9 years ago by
comment:3 follow-up: ↓ 4 Changed 9 years ago by
- 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
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
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
- Status changed from needs_work to needs_review
comment:7 Changed 9 years ago by
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: ↓ 9 Changed 9 years ago by
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
- 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
- Resolution set to fixed
- Reviewers set to Dmitrii Pasechnik
- Status changed from positive_review to closed
Merged 2010/04/20.
I just updated the spkg to make it support multithreading through Cbc !
Nathann