Add getNNodesLeft method#1236
Conversation
|
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. |
|
@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: 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)? |
That's fair, we can merge this one and do the bulk one later.
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. |
|
ok, I've added it manually |
Wrap SCIPgetNNodesLeft and append a check to an existing test