oscarhiggott
oscarhiggott
Hi Craig, thanks for reporting this, I agree it looks very suspicious. I suspect this problem may arise only when a boundary is present, since we didn't see any issues...
True, though it would good to fix this asap so I'll see if a tweak to the algorithm as described above will remove the problem as well
I agree it would be good to add this
Although I agree this is fixed with the protected main method for sinter, I also plan to add `pymatching.Matching.decode_bit_packed_batch(det_data)` at some point, so reopening this one
The reason for this issue is that parallel edges aren't currently supported in pymatching (for the check matrices you have both given, there are duplicated columns) although I will add...
This is now fixed and will be in the next release. There will be a `merge_strategies` argument when loading from check matrix or adding an edge, which can be used...
How are you installing it? Did you use `pip install pymatching` or are you cloning the repo and building the CLI or Python package from source?
For now, you could see if `pip install PyMatching==0.7.0` works? [Version 0.7](https://pypi.org/project/PyMatching/0.7.0/) doesn't depend on stim so won't have this same issue, though I've never tested compiling it on a...
Thank you for the contribution, please could you merge master into your branch to fix the CI failure?
Hi @yoyocni, apologies for the slow response. You can use the [detector_error_model_to_check_matrices](https://github.com/oscarhiggott/BeliefMatching/blob/2ff4f1dfe703c6efc146f268cafea24adb6474e1/src/beliefmatching/belief_matching.py#L61) method from the [BeliefMatching](https://pypi.org/project/beliefmatching/) python package. Is that the check matrix you were looking for?