Skip to content

Nandhini/toggle post processing#73

Merged
nandhinibsn merged 17 commits intodevfrom
nandhini/TogglePostProcessing
May 20, 2022
Merged

Nandhini/toggle post processing#73
nandhinibsn merged 17 commits intodevfrom
nandhini/TogglePostProcessing

Conversation

@nandhinibsn
Copy link
Contributor

@nandhinibsn nandhinibsn commented May 18, 2022

Description: Added a new feature "Without Postprocessing" to disable/enable post processing on the data

Checklist:

You should check all boxes before the PR is ready. If a box does not apply, check it to acknowledge
it.

  • PRE-COMMIT. You ran pre-commit on all commits, or else, you
    ran pre-commit run --all-files at the end.
  • FRONTEND TYPES. Regenerate the front-ent types if you played with types and routes.
    Run cd webapp && yarn types while the back-end is running.
  • USER CHANGES. The changes are added to CHANGELOG.md and the documentation, if they impact
    our users.
  • DEV CHANGES.
    • Update the documentation if this PR changes how to develop/launch on the app.
    • Update the README files and our wiki for any big design decisions, if relevant.
    • Add unit tests, docstrings, typing and comments for complex sections.

@nandhinibsn
Copy link
Contributor Author

differences on enable/disable of Postprocessing in confidence histogram, metrics, topwords.
Disabled postprocessing:
image
Enabled postprocessing:
image

differences on enable/disable of Postprocessing in confusion matrix

Disabled postprocessing:
image
Enabled postprocessing:
image
differences on enable/disable of Postprocessing in utterance table
Disabled postprocessing:
image

Enabled postprocessing:
image

@JosephMarinier
Copy link
Contributor

You would need to Edit the PR to request against dev, and not main.

@nandhinibsn nandhinibsn changed the base branch from main to dev May 18, 2022 21:26
Copy link
Contributor

@JosephMarinier JosephMarinier left a comment

Choose a reason for hiding this comment

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

🚢 🇮🇹

@nandhinibsn nandhinibsn merged commit 7c0fd00 into dev May 20, 2022
@nandhinibsn nandhinibsn deleted the nandhini/TogglePostProcessing branch May 20, 2022 13:43
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.

3 participants