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

Updated Max pool1d #15776

Closed
wants to merge 7 commits into from
Closed

Updated Max pool1d #15776

wants to merge 7 commits into from

Conversation

mtnvdsk
Copy link
Contributor

@mtnvdsk mtnvdsk commented May 23, 2023

Adding max_pool1d #15775

@mtnvdsk mtnvdsk requested a review from CatB1t as a code owner May 23, 2023 04:30
@mtnvdsk
Copy link
Contributor Author

mtnvdsk commented May 23, 2023

close #15776

@mtnvdsk
Copy link
Contributor Author

mtnvdsk commented May 24, 2023

sir @kurshakuz , All the test cases are passing except a jax test case failure could you please guide me to rectify the error and code.

Comment on lines +1544 to +1558
kernel = []
new_kernel = []
dilations = []

for i in range(array_dim):
if i + 1 < len(in_shape):
kernel.append(draw(st.integers(1, in_shape[i+1])))
else:
kernel.append(draw(st.integers(1, 10))) # Replace 10 with an appropriate maximum value

if return_dilation:
if kernel[i] > 1:
max_dilation = (in_shape[i + 1] - kernel[i]) // (kernel[i] - 1) + 1
max_dilation = (in_shape[i+1] - kernel[i]) // (kernel[i] - 1) + 1
dilations.append(draw(st.integers(1, max_dilation)))
new_kernel.append(kernel[i] + (kernel[i] - 1) * (dilations[i] - 1))
new_kernel.append(kernel[i] + (kernel[i] - 1) * (dilations[i-1] - 1))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @mtnvdsk , can you please explain the rationale behind those changes? what is the bug/issue you're trying to fix?

@ivy-seed ivy-seed added the Stale label Jun 20, 2023
@ivy-seed
Copy link

This PR has been labelled as stale because it has been inactive for more than 7 days. If you would like to continue working on this PR, then please add another comment or this PR will be closed in 7 days.

1 similar comment
@ivy-seed
Copy link

This PR has been labelled as stale because it has been inactive for more than 7 days. If you would like to continue working on this PR, then please add another comment or this PR will be closed in 7 days.

@ivy-seed ivy-seed assigned xoiga123 and unassigned kurshakuz Jun 22, 2023
@ivy-seed ivy-seed closed this Jul 1, 2023
@ivy-seed
Copy link

ivy-seed commented Jul 1, 2023

This PR has been closed because it has been marked as stale for more than 7 days with no activity.

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.

5 participants