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

Migrate away from deprecated np.float #5898

Merged

Conversation

mweinelt
Copy link
Contributor

@mweinelt mweinelt commented Apr 3, 2023

It is an alias for the python float type, and got deprecated in 1.20 and was removed in 1.24. The rest of the project already uses float32 (single), so I believe this is also correct here, as opposed to float64 (double).

https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations

It is an alias for the python float type, and got deprecated in 1.20 and
was removed in 1.24. The rest of the project already uses float32
(single), so I believe this is also correct here, as opposed to float64
(double).
@netlify
Copy link

netlify bot commented Apr 3, 2023

Deploy Preview for frigate-docs ready!

Name Link
🔨 Latest commit 5f83e01
🔍 Latest deploy log https://app.netlify.com/sites/frigate-docs/deploys/642a2048f7155b00076d3fd8
😎 Deploy Preview https://deploy-preview-5898--frigate-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@blakeblackshear blakeblackshear merged commit cb73d0c into blakeblackshear:dev Apr 16, 2023
@mweinelt mweinelt deleted the numpy-float-deprecation branch April 16, 2023 12:14
mew1033 pushed a commit to mew1033/frigate that referenced this pull request Apr 19, 2023
It is an alias for the python float type, and got deprecated in 1.20 and
was removed in 1.24. The rest of the project already uses float32
(single), so I believe this is also correct here, as opposed to float64
(double).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants