-
Notifications
You must be signed in to change notification settings - Fork 63
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
Bliss API Compatability: AbstractGraph::find_automorphisms
; outdated signature
#16
Comments
The same incompatibility shows in |
Dear @ndrwnaguib |
Hi Franzi; I am also using bliss
I'm not sure why we have different results; especially that the change took effect starting from version 0.75 (according to the release notes) |
Hey @ndrwnaguib |
Franzi, thanks for the follow up; I installed it from the Arch Linux official repositories.
I think the documentation lists the third parameter as
In the referenced SCIP line, it's being used as |
As far as I can see, the code takes the new interface into account, see the part starting with
in
where Can you please check whether the define is true in your case (it should be with Bliss version 0.77). |
@pfetsch, thanks for noting that, it is not there. I was building 7.0.3 instead of 8.0.0. The latest SCIP version should be working however considering the lines you referenced; thank you. |
When trying to compile SCIP from source, the build fails with
in
scip/src/symmetry/compute_symmetry_bliss.cpp
Line 1087 in 07dfd86
I looked up the reference, seems like Bliss has changed the signature (
user_param
is no longer accepted as a param), Is SCIP using a forked version of the API?I was able to surpass the error by rewriting the
blisshook
toand the call to
It is now working as expected, if you think that's a correct fix, I can prepare a patch :).
Thanks.
The text was updated successfully, but these errors were encountered: