-
Notifications
You must be signed in to change notification settings - Fork 22
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
Epic/cv2 5050 text vectorization via presto #2112
base: develop
Are you sure you want to change the base?
Conversation
…similarity tests]
…take 2 [similarity tests]
test/run-tests.sh
Outdated
else | ||
./parallel-similarity-test.sh | ||
./parallel-media-similarity-test.sh | ||
./parallel-text-similarity-test.sh |
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.
Here it should be one or the other depending on whether we're running media similarity tests or text similarity tests.
1686c3e
to
a55973f
Compare
Just a refactoring. Two separate builds in CI, one for text and another one for media. --------- Co-authored-by: danielevalverde <[email protected]> Co-authored-by: Caio <[email protected]>
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.
@danielevalverde, @DGaffney and I worked together on this.
Description
no-op PR for doing integration tests on check-api changes for alegre/text on presto.
Type of change
How has this been tested?
NA
Things to pay attention to during code review
NA
Checklist
propTypes
are declared and they use React Hooks and, if data-fetching is required, they use Relay Modern with fragment containers