forked from nv-morpheus/Morpheus
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Misc import fixes and reset request_id #9
Closed
dagardner-nv
wants to merge
14
commits into
mdemoret-nv:mdd_use-grpc-camouflage
from
dagardner-nv:david-mdd_use-grpc-camouflage
Closed
Misc import fixes and reset request_id #9
dagardner-nv
wants to merge
14
commits into
mdemoret-nv:mdd_use-grpc-camouflage
from
dagardner-nv:david-mdd_use-grpc-camouflage
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ow (nv-morpheus#429) Fixes an issue where without specifying the `nvidia/label/dev` channel conda/mamba will install an old version of tensorflow (2.4.1) which is unable to load the model file. fixes nv-morpheus#324 Authors: - David Gardner (https://github.com/dagardner-nv) Approvers: - Christopher Harris (https://github.com/cwharris) - Pete MacKinnon (https://github.com/pdmack) - Michael Demoret (https://github.com/mdemoret-nv) URL: nv-morpheus#429
…v-morpheus#419) * `MultiInferenceMessage` & `MultiResponseMessage` now inherit a new class `MultiTensorMessage` reducing the duplicated code in each * `MultiResponseProbsMessage` class now uses `ResponseMemoryProbs` * Add missing docstrings * Common CSV & Json reading code moved to deserializers.[hpp|cpp] fixes nv-morpheus#313 fixes nv-morpheus#311 Authors: - David Gardner (https://github.com/dagardner-nv) Approvers: - Michael Demoret (https://github.com/mdemoret-nv) URL: nv-morpheus#419
Builds two different images: `morpheus-ci-driver` and `morpheus-ci-test`. The driver image is 15.9GB and the test image is 14.2GB. In this setup only the wheel is transferred from the build to the test and documentation stages(nv-morpheus#112). Base image contains: - Pre-populated conda env - libnuma1 Driver image contains: - libnvidia-compute-495 - ibcublas-dev-11-5 - libcufft-dev-11-5 - libcurand-dev-11-5 - libcusolver-dev-11-5 Test image contains - pytest-kafka - Kafka - openjdk-11 (needed for kafka) - nodejs - npm - camouflage fixed nv-morpheus#112 fixes nv-morpheus#400 Authors: - David Gardner (https://github.com/dagardner-nv) Approvers: - Michael Demoret (https://github.com/mdemoret-nv) URL: nv-morpheus#408
If a test crashes or a developer stops a test in the middle of debugging, Camouflage will be left open. Any subsequent runs of pytest will fail to start. This fixes the issue by binding the child process to the pytest process, ensuring that it will always be closed. Also added support for using already running Camouflage servers. Closes nv-morpheus#329 Authors: - Michael Demoret (https://github.com/mdemoret-nv) Approvers: - David Gardner (https://github.com/dagardner-nv) URL: nv-morpheus#436
* Avoid unnecessary copies in file source stage * When we do a copy, perform it before emitting * Skip benchmarks by default * Adds a `benchmark` pytest marker and associated `--run_benchmark` flag fixes nv-morpheus#443 Authors: - David Gardner (https://github.com/dagardner-nv) Approvers: - Michael Demoret (https://github.com/mdemoret-nv) URL: nv-morpheus#444
This requires ensuring that all newly built Tensors using the `create` method have their strides specified in elements not bytes. Authors: - David Gardner (https://github.com/dagardner-nv) - Michael Demoret (https://github.com/mdemoret-nv) Approvers: - Michael Demoret (https://github.com/mdemoret-nv) URL: nv-morpheus#438
dagardner-nv
pushed a commit
that referenced
this pull request
Aug 4, 2023
I have a few changes here. Some of them might not make it depending on how others feel. - Move test utilities in to a library to share across tests. - Move tests in to individual executable to enforce individual tests to not sharing code. - Use `gtest_discover_tests` to let `ctest` run and show all tests individually. - Use `gtest::gtest_main` and remove explicit main entry file. - ~Build and run tests by default.~ - Ability to filter tests by `ctest --test-dir build -R "TestCuda.*"` ``` + echo 'Running CTest...' Running CTest... + ctest --test-dir build Internal ctest changing into directory: /workspaces/morpheus/build Test project /workspaces/morpheus/build Start 1: TestCuda.LargeShape 1/35 Test #1: TestCuda.LargeShape ....................................... Passed 0.17 sec Start 2: TestDataLoader.DataLoaderInitializationTest 2/35 Test #2: TestDataLoader.DataLoaderInitializationTest ............... Passed 0.07 sec Start 3: TestDataLoader.DataLoaderRegisterLoaderTest 3/35 Test #3: TestDataLoader.DataLoaderRegisterLoaderTest ............... Passed 0.07 sec Start 4: TestDataLoader.DataLoaderRemoveLoaderTest 4/35 Test #4: TestDataLoader.DataLoaderRemoveLoaderTest ................. Passed 0.07 sec Start 5: TestDataLoader.PayloadLoaderTest 5/35 Test #5: TestDataLoader.PayloadLoaderTest .......................... Passed 1.42 sec Start 6: TestDataLoader.FileLoaderTest 6/35 Test #6: TestDataLoader.FileLoaderTest ............................. Passed 2.65 sec Start 7: TestControlMessage.InitializationTest 7/35 Test #7: TestControlMessage.InitializationTest ..................... Passed 0.07 sec Start 8: TestControlMessage.SetMessageTest 8/35 Test #8: TestControlMessage.SetMessageTest ......................... Passed 0.07 sec Start 9: TestControlMessage.TaskTest 9/35 Test #9: TestControlMessage.TaskTest ............................... Passed 0.07 sec Start 10: TestControlMessage.PayloadTest 10/35 Test #10: TestControlMessage.PayloadTest ............................ Passed 1.45 sec Start 11: TestDataLoaderModule.DataLoaderModuleInitializationTest 11/35 Test #11: TestDataLoaderModule.DataLoaderModuleInitializationTest ... Passed 0.07 sec Start 12: TestDataLoaderModule.EndToEndPayloadDataLoaderTest 12/35 Test #12: TestDataLoaderModule.EndToEndPayloadDataLoaderTest ........ Passed 0.12 sec Start 13: TestDeserializers.GetIndexColCountNoIdxFromFile 13/35 Test #13: TestDeserializers.GetIndexColCountNoIdxFromFile ........... Passed 1.39 sec Start 14: TestDeserializers.GetIndexColCountWithIdxFromFile 14/35 Test #14: TestDeserializers.GetIndexColCountWithIdxFromFile ......... Passed 1.37 sec Start 15: TestDeserializers.GetIndexColCountNoIdxSimilarName 15/35 Test nv-morpheus#15: TestDeserializers.GetIndexColCountNoIdxSimilarName ........ Passed 0.19 sec Start 16: TestDeserializers.GetIndexColCountIdx 16/35 Test nv-morpheus#16: TestDeserializers.GetIndexColCountIdx ..................... Passed 0.18 sec Start 17: TestDeserializers.GetIndexColCountValidNameInvalidType 17/35 Test nv-morpheus#17: TestDeserializers.GetIndexColCountValidNameInvalidType .... Passed 0.16 sec Start 18: TestDevMemInfo.RmmBufferConstructor 18/35 Test nv-morpheus#18: TestDevMemInfo.RmmBufferConstructor ....................... Passed 0.16 sec Start 19: TestDevMemInfo.VoidPtrConstructor 19/35 Test nv-morpheus#19: TestDevMemInfo.VoidPtrConstructor ......................... Passed 0.25 sec Start 20: TestDevMemInfo.MakeNewBuffer 20/35 Test nv-morpheus#20: TestDevMemInfo.MakeNewBuffer .............................. Passed 0.25 sec Start 21: TestFileInOut.RoundTripCSV 21/35 Test nv-morpheus#21: TestFileInOut.RoundTripCSV ................................ Passed 2.67 sec Start 22: TestFileInOut.RoundTripJSON 22/35 Test nv-morpheus#22: TestFileInOut.RoundTripJSON ............................... Passed 2.69 sec Start 23: TestMatxUtil.ReduceMax1d 23/35 Test nv-morpheus#23: TestMatxUtil.ReduceMax1d .................................. Passed 0.17 sec Start 24: TestMatxUtil.ReduceMax2dRowMajor 24/35 Test nv-morpheus#24: TestMatxUtil.ReduceMax2dRowMajor .......................... Passed 0.17 sec Start 25: TestMatxUtil.ReduceMax2dColMajor 25/35 Test nv-morpheus#25: TestMatxUtil.ReduceMax2dColMajor .......................... Passed 1.40 sec Start 26: TestMatxUtil.Cast 26/35 Test nv-morpheus#26: TestMatxUtil.Cast ......................................... Passed 0.16 sec Start 27: TestMatxUtil.Threshold 27/35 Test nv-morpheus#27: TestMatxUtil.Threshold .................................... Passed 0.16 sec Start 28: TestMatxUtil.ThresholdByRow 28/35 Test nv-morpheus#28: TestMatxUtil.ThresholdByRow ............................... Passed 0.19 sec Start 29: TestMultiSlices.Ranges 29/35 Test nv-morpheus#29: TestMultiSlices.Ranges .................................... Passed 1.37 sec Start 30: TestTensor.UtilsShapeString 30/35 Test nv-morpheus#30: TestTensor.UtilsShapeString ............................... Passed 0.06 sec Start 31: TestTensor.GetElementStride 31/35 Test nv-morpheus#31: TestTensor.GetElementStride ............................... Passed 0.06 sec Start 32: TestTensor.AsType 32/35 Test nv-morpheus#32: TestTensor.AsType ......................................... Passed 0.16 sec Start 33: TestTensor.Create 33/35 Test nv-morpheus#33: TestTensor.Create ......................................... Passed 0.16 sec Start 34: TestTensor.UtilsValidateShapeAndStride 34/35 Test nv-morpheus#34: TestTensor.UtilsValidateShapeAndStride .................... Passed 0.16 sec Start 35: TestTypeUtils.DTypeCopy 35/35 Test nv-morpheus#35: TestTypeUtils.DTypeCopy ................................... Passed 0.06 sec 100% tests passed, 0 tests failed out of 35 Total Test time (real) = 19.91 sec ``` Authors: - Christopher Harris (https://github.com/cwharris) Approvers: - David Gardner (https://github.com/dagardner-nv) URL: nv-morpheus#1095
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.