Interface SCIP methods missing from PySCIPOpt

[adamj34]

Wrap SCIPgetNNodesLeft and append a check to an existing test

[Joao-Dionisio]

Thank you @adamj34 , this looks good! How many other methods did you interface? It may be good to add a bunch at a time, to avoid merging too many PRs and spamming our followers.

[adamj34]

@Joao-Dionisio I could interface a few more. I wanted to keep it simple for the first PR but I can add the rest in bulk, as you suggested.

I see that there's an error in the CI pipeline:
error: pyscipopt.scip.Model.getNNodesLeft is not present in stub

Should new methods be added there by hand, or it's supposed to be automated in some way (by the .pre-commit-config.yaml hook, possibly)?

[Joao-Dionisio]

I wanted to keep it simple for the first PR

That's fair, we can merge this one and do the bulk one later.

Should new methods be added there by hand, or it's supposed to be automated in some way (by the .pre-commit-config.yaml hook, possibly)?

We've started working on adding stubs this year, so for now this is still something that needs to be done manually, but hopefuly not for much longer. There is a stub generator, but it's not working yet.

[adamj34]

ok, I've added it manually

[Joao-Dionisio]

Looks good @adamj34 , thank you! If there are other methods you wish to interface, we'd be very appreciative!

[adamj34]

Sure, I'll probably find some time this week to interface a few more :)