Mo Tiwari
Mo Tiwari
Hi @retzerjj ; I think I was able to build the Python wheel for M1 Macs; could you install banditpam via `pip install -i https://test.pypi.org/simple/ banditpam==3.0.3a1` and let me know...
Thanks @retzerjj ! It's strange; `pip install -i https://test.pypi.org/simple/ banditpam==3.0.3a1` should not have succeeded unless you're using CPython. In the `pip` output, did it somehow fall back to a previous...
Hmmm it seems Sudhir deleted his comment, but we owe a thank you to him! @retzerjj you can install Python from Anaconda [here](https://www.anaconda.com/); it should provide instructions on how to...
Sorry for the late response @retzerjj . I'm not sure how to attach pictures by email-replying to Github email notifications, but if you sign into Github and view [this link](https://github.com/ThrunGroup/BanditPAM/issues/167),...
I like the proposed solution, as supporting sparse matrices natively would be significantly more complicated
Not fully resolved since global `algorithm` and some others used throughout
Hi @Asber777 , we created a [fork](https://github.com/ThrunGroup/implicit-hyper-opt) of this repo to attempt to patch it but still couldn't get it to work.
This would also make the behavior consistent with `LoadEdgeList`, which has a `Separator` parameter. Also this function, `LoadEdgeListStr`, only detects '\n' as newlines and not '\r'
Indeed, that fixed it - but shouldn't snap throw an error with that message instead of segfaulting?
Thank you for submitting a PR! @bnaras could you take a look?