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

Modification of CategoryEncoding example code #19735

Closed
wants to merge 1 commit into from

Conversation

keerthanakadiri
Copy link

The example code was not working with the previous version so made changes as required . Please find the tested gist and do the needful . Thank you !

The example code was not working with the previous version so made changes as required . Please find the tested https://colab.research.google.com/gist/keerthanakadiri/c36fb5b439494618462daba5698b69d5/categoryencoding.ipynb and do the needful . Thank you !
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation May 20, 2024
@gbaned gbaned requested review from nkovela1 May 20, 2024 05:56
@keerthanakadiri keerthanakadiri changed the title Updated the example code Modification of CategoryEncoding example code May 20, 2024
@codecov-commenter
Copy link

codecov-commenter commented May 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.53%. Comparing base (a05ac12) to head (403283e).

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #19735   +/-   ##
=======================================
  Coverage   78.53%   78.53%           
=======================================
  Files         498      498           
  Lines       45753    45753           
  Branches     8455     8455           
=======================================
  Hits        35933    35933           
  Misses       8086     8086           
  Partials     1734     1734           
Flag Coverage Δ
keras 78.38% <ø> (ø)
keras-jax 61.94% <ø> (-0.01%) ⬇️
keras-numpy 56.30% <ø> (ø)
keras-tensorflow 63.42% <ø> (ø)
keras-torch 62.01% <ø> (ø)

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.

@fchollet
Copy link
Member

Thanks for the PR! The fact that the code example didn't work was a bug -- I added a tensor conversion in the layer to fix it.

@fchollet fchollet closed this May 21, 2024
PR Queue automation moved this from Assigned Reviewer to Closed/Rejected May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
PR Queue
Closed/Rejected
Development

Successfully merging this pull request may close these issues.

None yet

4 participants