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.

Sign up for GitHub

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

Jump to bottom

makeModifiedInput to correctly set _id and _meta on new examples. #1208

Closed

copybara-service wants to merge 0 commits into dev from test_552336983

copybara-service bot commented Jul 31, 2023

makeModifiedInput to correctly set _id and _meta on new examples.

Fixes a bug where modified examples still had the original ID, and thus were
returning cached predictions from the original example.

Fix manual-example-creation flow, which was incorrectly inheriting the parent ID.
Fix LM prediction module and TDA module which use ephemeral examples with
empty ID.

copybara-service bot force-pushed the test_552336983 branch from ebaa3b6 to 8dac782 Compare

July 31, 2023 20:26

copybara-service bot closed this

copybara-service bot force-pushed the test_552336983 branch from 8dac782 to 0ec1527 Compare

July 31, 2023 20:43

copybara-service bot deleted the test_552336983 branch

July 31, 2023 20:43

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet