Mark Turner

This is on my to-do list. I remember there was an issue with adding such a function in previous versions, but can't remember what that issue was. I wouldn't be...

@Joao-Dionisio I added some changes to the branch. Summarised version with some tips for the test: - Avoid using randomness in tests unless a random seed is used - I...

@Joao-Dionisio @mmghannam I slept on this, and am now firmly against adding such functionality. It requires keeping track of some hidden overhead, and I think will confuse more users than...

Is it possible to copy the error message here? To give a summary of what the problem most likely is: From v4.4.0, when using pip to install PySCIPOPt, a new...

Glad that the solution worked! For now I think this will have to be the method that you use to install PySCIPOpt (unless it becomes a common error). I'm not...

@mrmundt Can you please run the pipeline now? I put `scip_persistent` as an expected fail for the `LP_trivial_constraint` test case. Everything is now passing locally.

Hmmmmmmmmmmmmmm. I didn't have scipy installed when I tested locally so it skipped that test. The error is that `np.float` is a non-allowed type when using `PySCIPOpt`. The user must...

@mrmundt I've added a warning now for each time a constraint is added and the RHS / LHS needs to be changed. I had to use `type(x) == y` because...

@mrmundt I'm not sure if the failing tests are because of my changes. Can I get a second opinion on that?

For counting solutions: Have you simply tried `getNSolsFound()`? That is preferable to setting the parameter `countsols` and relying on the constraint handler. Edit: After talking to a colleague: If you...