Skip to content

Conversation

@noajshu
Copy link
Contributor

@noajshu noajshu commented Aug 31, 2025

No description provided.

noajshu and others added 22 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
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@noajshu noajshu force-pushed the remove-at-most-two-errors-per-detector branch from b3eaf04 to 8f269f7 Compare August 31, 2025 00:20
@noajshu
Copy link
Contributor Author

noajshu commented Aug 31, 2025

at most two errors is complicating a lot of things with some refactors and I don't think we ever used it in the paper anyways.
I think it will be fairly straightforward to add it back in if we ever need it again :)

@noajshu noajshu marked this pull request as ready for review August 31, 2025 01:17
@noajshu noajshu merged commit 8129e15 into main Aug 31, 2025
8 checks passed
@noajshu noajshu deleted the remove-at-most-two-errors-per-detector branch August 31, 2025 01:25
@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