Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Is pinning to isosplit5==0.1.3 necessary? #10

Closed
rly opened this issue Sep 26, 2024 · 5 comments
Closed

Is pinning to isosplit5==0.1.3 necessary? #10

rly opened this issue Sep 26, 2024 · 5 comments

Comments

@rly
Copy link

rly commented Sep 26, 2024

'isosplit5==0.1.3',

isosplit5 0.2.0+ includes this fix magland/isosplit5_python#8 which installs pybind11 before setup.

I suspect this might also fix #7 (and the OP in #6 mentioned that installing isosplit5 from source fixed a similar issue) but I am not sure because that might just be a windows thing.

@magland
Copy link
Owner

magland commented Sep 26, 2024

Oh I didn't realize it was pinned. I guess I should put it as >=0.2.0? I don't anticipate developing isosplit5 any further since there is now isosplit6. But also it would not be wise to move to isosplit6 for mountainsort4 since there were changes in the implementation.

@rly
Copy link
Author

rly commented Sep 26, 2024

I think >=0.2.0 would be good. Installing mountainsort4 naively (pip install mountainsort4) fails on my mac because pybind11 isn't installed first, and I think updating this requirement would fix that. Super minor, because I think most people will/should use the newer mountainsort, but I need to use mountainsort4 to reproduce some older results.

@rly
Copy link
Author

rly commented Sep 26, 2024

(I can get around this issue easily - just thought I would help the next person who might encounter this!)

@magland
Copy link
Owner

magland commented Sep 26, 2024

Okay this is done. mountainsort4 1.0.6 on pypi. Thanks @rly

@magland magland closed this as completed Sep 26, 2024
@rly
Copy link
Author

rly commented Sep 26, 2024

Thank you! pip install mountainsort4 works like a charm now.

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

No branches or pull requests

2 participants