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

updating the deprecated np.int #43

Merged
merged 2 commits into from
Mar 13, 2023
Merged

Conversation

Joscelinrocha
Copy link
Contributor

np.int was a deprecated alias for the builtin int. To avoid this error in existing code, use int by itself.

`np.int` was a deprecated alias for the builtin `int`. To avoid this error in existing code, use `int` by itself.
`np.int` was a deprecated alias for the builtin `int`. To avoid this error in existing code, use `int` by itself.
@klapo
Copy link

klapo commented Mar 10, 2023

Hi @Joscelinrocha, it seems like this library is no longer maintained. I think it may be prudent for us (us in the general sense) to open a new fork and disconnect it from this repository so the package can be maintained. Would you like to open up an issue on forking this repository to a new owner? We can discuss who can take over in that case.

@regeirk
Copy link
Owner

regeirk commented Mar 13, 2023

Hi. Thank you for pointing this out and suggesting the changes. I haven't been using Wavelet analysis for a while, my apologies for taking so long.

@regeirk regeirk merged commit d53dbe4 into regeirk:master Mar 13, 2023
@enekomartinmartinez
Copy link

Hi @regeirk @Joscelinrocha

in the PR a np.interp was replaced by interp. The previous one should be recovered or interp should be explicitly imported from numpy

@regeirk
Copy link
Owner

regeirk commented Mar 15, 2023

Hi @enekomartinmartinez, thank you for pointing this out. There were other issues as well. I've made a pre-release with some additional updates.

@filefolder filefolder mentioned this pull request Sep 8, 2023
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

Successfully merging this pull request may close these issues.

4 participants