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

scipy imresize is deprecated #1871

Closed
fehiepsi opened this issue May 20, 2019 · 2 comments · Fixed by #1881
Closed

scipy imresize is deprecated #1871

fehiepsi opened this issue May 20, 2019 · 2 comments · Fixed by #1881
Labels
Milestone

Comments

@fehiepsi
Copy link
Member

fehiepsi commented May 20, 2019

imresize is used in air example, but it is deprecated now. We need to replace it by something else to make tests pass.

More context about this issue: imresize is used in observations but this library is deprecated now in favour of tensorflow datasets. But multi-mnist is not available in tensorflow's datasets yet. So I'm not sure what is a good solution for this issue.

@fehiepsi fehiepsi added the bug label May 20, 2019
@neerajprad
Copy link
Member

The problem is with the observations library (that uses this function) which is not being maintained anymore. I think we need to remove dependency on observations.

@null-a
Copy link
Collaborator

null-a commented May 22, 2019

We need to replace it by something else

The docs suggest using numpy.array(Image.fromarray(arr).resize()) inplace of imresize. This requires PIL which AIR already has as a dependency.

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 a pull request may close this issue.

4 participants