Skip to content

Conversation

@alejoe91
Copy link
Member

Starting from @h-mayorquin PR #3481, but one thing at a time.

This is only for numpy 2.0 (with NEO 0.14.0)

@alejoe91 alejoe91 added the packaging Related to packaging/style label Jan 24, 2025
@alejoe91 alejoe91 requested review from h-mayorquin and zm711 January 24, 2025 16:38

dependencies = [
"numpy>=1.20, <2.0", # 1.20 np.ptp, 1.26 might be necessary for avoiding pickling errors when numpy >2.0
"numpy>=1.20",
Copy link
Member

@zm711 zm711 Jan 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we sure we have enough of pickling tests to feel okay deleting Heberto's note?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think so! :) let's see

Copy link
Member

@zm711 zm711 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Copy link
Collaborator

@h-mayorquin h-mayorquin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alejoe91 alejoe91 merged commit bed6308 into SpikeInterface:main Jan 24, 2025
15 checks passed
dhmjhu added a commit to regro-cf-autotick-bot/spikeinterface-feedstock that referenced this pull request Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packaging Related to packaging/style

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants