-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
Dropout 2D function and Test in Paddle #17477
Conversation
Hi @HaiderSultanArc please also make sure that the formatting test passes or I cannot merge the PR. You can check the logs here on github or locally to see what went wrong! |
@DragosStoican I fixed the formatting. Let me know if any other changes are required. |
@handle_frontend_test( | ||
fn_tree="paddle.nn.functional.common.dropout2d", | ||
d_type_and_x=helpers.dtype_and_values( | ||
available_dtypes=helpers.get_dtypes("float"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should be ideally testing for all valid dtypes, and not only a subset.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it
When testing the function, we should ideally test for all valid dtypes, not only a subset. Can you use the key "valid" when generating dtypes and making sure the test is still passing? I added a comment on that line to help you understand what I'm referring to :) |
Thank you for your contribution! |
Close #17475