CoinMP returns wrong status
- Dominant language
- Shell
- Stars
- 21
- Forks
- 12
- PR merge metrics
- No merged PRs in 30d
Description
Issue created by migration from Trac.
Original creator: pikachu
Original creation time: 2016-04-19 09:42:27
Assignee: somebody
Version:
According https://projects.coin-or.org/Cbc/browser/stable/2.9/Cbc/src/CbcModel.hpp#L945
The final CbcModel status need to check isXXX functions rather than use the status code directly
So the line at https://projects.coin-or.org/CoinMP/browser/stable/1.8/CoinMP/src/CoinCbc.cpp#L986 is not correct.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start in CoinMP/src/CoinCbc.cpp at line 986 and compare its final-status handling with the isXXX functions described around line 945 of Cbc/src/CbcModel.hpp. Confirm the expected status behavior for each relevant result, then verify that CoinMP reports those statuses consistently.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- api
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 48/100