Open veljanin opened 1 week ago
Thank you very much for bringing this to my attention and for your detailed feedback!
I’ve entirely re-coded the Ochiai class to make it more robust and versatile, addressing the issue you encountered. The new implementation now correctly handles both list and set inputs, ensuring consistent and accurate results. I really appreciate your contribution, as it helped identify and resolve this problem effectively.
Please feel free to review the updated code, and let me know if you notice any further issues or have additional suggestions. Thanks again for your valuable input!
I've noticed that Ochiai returns 1 every single time This is due to the following set conversion:
If you convert to
list
instead, you would get the correct result. Not sure what is the right process for PR and contributions...