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

Update SequenceGeneratorDecoder to output predictions and probabilities #3152

Merged
merged 1 commit into from
Feb 28, 2023

Conversation

jeffkinnison
Copy link
Contributor

Predictor looks for <output_feature>.predictions and <output_feature>.probabilities in model output, however SequenceGeneratorDecoder.get_prediction_set only included logits. Excluding the others led to a KeyError when unflattening output features

File "/home/ray/anaconda3/lib/python3.8/site-packages/pandas/core/indexes/base.py", line 3623, in get_loc
    raise KeyError(key) from err
KeyError: 'title_probabilities' (type: RayTaskError(KeyError), retryable: true)

This update adds predictions and probabilities to get_prediction_set.

@github-actions
Copy link

github-actions bot commented Feb 27, 2023

Unit Test Results

         6 files  ±0           6 suites  ±0   5h 38m 48s ⏱️ + 4m 27s
  3 976 tests ±0    3 932 ✔️ ±0    44 💤 ±0  0 ±0 
11 949 runs  ±0  11 808 ✔️ ±0  141 💤 ±0  0 ±0 

Results for commit 5e5786c. ± Comparison against base commit ebaf32f.

♻️ This comment has been updated with latest results.

@jeffkinnison jeffkinnison merged commit 37adc92 into master Feb 28, 2023
@jeffkinnison jeffkinnison deleted the generator-predictions-failure branch February 28, 2023 23:24
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.

None yet

2 participants