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

Addition of other pooling layers #1512

Open
Saanidhyavats opened this issue Oct 22, 2024 · 5 comments
Open

Addition of other pooling layers #1512

Saanidhyavats opened this issue Oct 22, 2024 · 5 comments

Comments

@Saanidhyavats
Copy link
Contributor

@awni We were going through pooling layers available in mlx framework. I think we can add pooling techniques for 3d data such as Avgpooling3d, Maxpooling3d and also implement AdaptivePooling for 2D data.

@Saanidhyavats
Copy link
Contributor Author

@awni what do you think about it?

@awni
Copy link
Member

awni commented Oct 24, 2024

3D pooling might be useful.. I'm not so sure how commonly used it is. I haven't seen too many requests for AdaptivePooling. In general for features like this I think it's ok to be a bit reactive (add them if needed) rather than adding them just for the sake of feature completeness.

@Saanidhyavats
Copy link
Contributor Author

3D pooling might be useful.. I'm not so sure how commonly used it is. I haven't seen too many requests for AdaptivePooling. In general for features like this I think it's ok to be a bit reactive (add them if needed) rather than adding them just for the sake of feature completeness.

I was going through the code in https://github.com/apple/ml-cvnets/blob/main/cvnets/layers/adaptive_pool.py and noticed that it's currently leveraging adaptive pooling from PyTorch. I believe we could enhance the project by incorporating adaptive pooling directly into MLX, allowing us to utilize this functionality seamlessly. Would this be something we could explore further?

@awni
Copy link
Member

awni commented Oct 25, 2024

I would suggest putting an issue in that repo about building an MLX back-end (which would be super cool)! If the project adds an MLX back-end and needs support from any missing features we'd be happy to reconsider what to add.

@Saanidhyavats Saanidhyavats mentioned this issue Oct 25, 2024
4 tasks
@cvnad1
Copy link
Contributor

cvnad1 commented Oct 25, 2024

@awni We believe that’s a good idea and will consider your advice.

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

3 participants