-
Notifications
You must be signed in to change notification settings - Fork 232
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
Remove the term whitelist #387
Conversation
Related to #363 |
build |
@@ -1,3 +1,4 @@ | |||
/enable |
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.
I think this is a type that is causing the build to fail
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.
Apologies, I realized after the build failed that I ran with -DskipTests on a non-GPU machine. I reran with mvn clean verify -Dcuda.version=cuda10-2
and the tests passed. I will try rerunning the premerge tests.
build |
* Removing the term whitelist * fix typo in hash_aggregate_test.py
* Removing the term whitelist * fix typo in hash_aggregate_test.py
* Update CI/CD requirements and tests * Fix main ci (NVIDIA#377) * Fix build doc process (NVIDIA#387) * Fix app testing pt model validator * Update tests * Fix format issues * Fix integration tests * Update premerge.yml * Fix format * Fix comments
made.
Removing the term whitelist per https://tools.ietf.org/id/draft-knodel-terminology-00.html
added.
No new tests, ran
mvn clean verify -DskipTests
to verify