-
Notifications
You must be signed in to change notification settings - Fork 11
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
Added set_obj_integral
Method
#101
Conversation
Update: Managed to get |
Codecov Report
@@ Coverage Diff @@
## main #101 +/- ##
==========================================
+ Coverage 91.29% 91.36% +0.06%
==========================================
Files 13 13
Lines 804 810 +6
==========================================
+ Hits 734 740 +6
Misses 70 70
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Thank you for the useful contribution. Just one minor change request and would be ready to merge, please don't forget to update the changelog.
Oh, I realize now you meant CHANGELOG.md - will do that |
Perfect, thanks again @CodingTil! Will merge right after the pipelines pass, looking forward to your next contribution :) |
See Issue #95
cargo build
runs through, as well ascargo fmt
(at least for my code).I am unable to run
cargo test
at the moment, it currently says it cannot findlibscip.8.0.dylib
- seems unrelated to my code.Changelog:
set_obj_integral
inscip.rs
which callsSCIPsetObjIntegral
set_obj_integral
toModel<ProblemCreated>
to allow specifying that the objective value is always integral