Skip to content

Conversation

kanpuriyanawab
Copy link
Collaborator

Partially fixes #812

@mattdangerw
Copy link
Member

This is accidentally pull the metrics changes too, can you make it just for tokenizers?

@mattdangerw mattdangerw self-requested a review March 9, 2023 22:47
@mattdangerw
Copy link
Member

Since the other PR was landed, just a merge with master should do it.

Copy link
Collaborator

@fchollet fchollet left a comment

Choose a reason for hiding this comment

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

Looks rebased now. LGTM, thanks for the help!

@fchollet fchollet merged commit 61f6443 into keras-team:master Mar 9, 2023
@kanpuriyanawab kanpuriyanawab deleted the api-export-tokenizers branch March 10, 2023 06:48
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

Successfully merging this pull request may close these issues.

Add public API export statements
3 participants