Skip to content

Conversation

@noajshu
Copy link
Contributor

@noajshu noajshu commented Aug 31, 2025

The Node is just a C++ implementation detail which does not need to be exposed to the python user (I think)
overall trying to simplify and clean things up so that we can iterate more easily on the algorithm

noajshu and others added 25 commits August 10, 2025 22:46
…e-visualizer-files

Add visualization library to CMake build
…-and-fix-cmake-tests

Fix CMake Python module path and add agent build instructions
…accept-bitstring

Allow decode_to_errors to accept bitstring
Co-authored-by: Noureldin <noureldinyosri@gmail.com>
…date-tesseract

Add explicit DetIndex branch for detector ordering
…ers-and-build_det_orders

refactor detector order helpers
The Node struct is an internal implementation detail of the C++ tesseract library and should not be exposed in the Python API.

This commit removes the pybind bindings for the Node struct and also removes the corresponding test case from the Python tests.
@noajshu noajshu merged commit f73044a into main Aug 31, 2025
6 of 7 checks passed
@noajshu noajshu deleted the refactor/remove-node-from-python-api branch August 31, 2025 01:33
@NoureldinYosri NoureldinYosri mentioned this pull request Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant