Sorry this took so long - I had a lot of other obligations suddenly pop up.
As discussed previously in #28, I decided to simply go with the override implementation, and have re-enabled tests pertaining to it in this PR.
With the re-enabled tests, I've also discovered that the Approach parameter wasn't working correctly (wasn't correctly converting from string), so I've fixed that as well. In addition, I also have done a simple pass through the code and fixed some errors.
Changes made:
Bumped nanobind version up to 1.8.0 (changelog here)
Sorry this took so long - I had a lot of other obligations suddenly pop up. As discussed previously in #28, I decided to simply go with the override implementation, and have re-enabled tests pertaining to it in this PR. With the re-enabled tests, I've also discovered that the Approach parameter wasn't working correctly (wasn't correctly converting from string), so I've fixed that as well. In addition, I also have done a simple pass through the code and fixed some errors.