-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add pipeline pre/post-processing steps breakdown in the utterance detail page #263
Conversation
6e187d7
to
463ca2f
Compare
463ca2f
to
a705df5
Compare
6f7c241
to
d912020
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small architecture question...
tests/test_modules/test_model_contracts/test_text_classification.py
Outdated
Show resolved
Hide resolved
959e961
to
76714aa
Compare
e84de16
to
a80e7d1
Compare
4a5dc7f
to
d690cc4
Compare
ThresholdConfig(threshold=0.5), | ||
] | ||
) | ||
] = Field([ThresholdConfig(threshold=0.5)], nullable=True) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the problem that @nandhinibsn has been experiencing; it was missing nullable=True
. It's not related to this PR, but I committed it here to avoid conflicts: 7a053daa36f66f9cca442b04a1816a7821903153
Co-authored-by: Joseph Marinier <joseph.marinier@servicenow.com>
6af5eb9
to
015b5a0
Compare
015b5a0
to
8cbd68b
Compare
Hello @christyler3030, @nandhinibsn, could you please review my work on the front end? I recommend reviewing those 3 commits separately:
|
Resolve #283
Description:
Checklist:
You should check all boxes before the PR is ready. If a box does not apply, check it to acknowledge it.
ran
pre-commit run --all-files
at the end.our users.
README
files and our wiki for any big design decisions, if relevant.