Skip to content

Update output_padding argument in convolution transpose layer #21383

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

Merged
merged 1 commit into from
Jun 17, 2025

Conversation

sonali-kumari1
Copy link
Contributor

@sonali-kumari1 sonali-kumari1 commented Jun 16, 2025

  • Added output_padding Argument in Conv1DTranspose, Conv2DTranspose, and Conv3DTranspose layers, which was previously missing in the constructors of these layers.

  • Improved description of dilation_rate argument.

  • Added allow_zero=True as an extra argument to standardize_tuple() in BaseConvTranspose

  • The documentation for Conv1DTranspose, Conv2DTranspose, and Conv3DTranspose layers has been updated to include the output_padding argument.

Fixes #19870 and #20408

@codecov-commenter
Copy link

codecov-commenter commented Jun 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.67%. Comparing base (764ed95) to head (9b8871d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #21383   +/-   ##
=======================================
  Coverage   82.67%   82.67%           
=======================================
  Files         565      565           
  Lines       55064    55064           
  Branches     8569     8569           
=======================================
  Hits        45525    45525           
  Misses       7441     7441           
  Partials     2098     2098           
Flag Coverage Δ
keras 82.48% <ø> (ø)
keras-jax 63.50% <ø> (ø)
keras-numpy 58.67% <ø> (ø)
keras-openvino 33.45% <ø> (ø)
keras-tensorflow 63.91% <ø> (ø)
keras-torch 63.53% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gbaned gbaned added this to PR Queue Jun 16, 2025
@github-project-automation github-project-automation bot moved this to Assigned Reviewer in PR Queue Jun 16, 2025
@gbaned gbaned requested a review from fchollet June 16, 2025 10:36
@google-ml-butler google-ml-butler bot added kokoro:force-run ready to pull Ready to be merged into the codebase labels Jun 17, 2025
@github-project-automation github-project-automation bot moved this from Assigned Reviewer to Approved by Reviewer in PR Queue Jun 17, 2025
@fchollet fchollet merged commit 92bfff0 into keras-team:master Jun 17, 2025
9 checks passed
@google-ml-butler google-ml-butler bot removed awaiting review ready to pull Ready to be merged into the codebase labels Jun 17, 2025
@github-project-automation github-project-automation bot moved this from Approved by Reviewer to Merged in PR Queue Jun 17, 2025
@sonali-kumari1 sonali-kumari1 deleted the output_paddingTest branch June 18, 2025 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

The documentation for Conv1DTranspose does not state that the CPU does not support dilation rates larger than 1
4 participants