-
Notifications
You must be signed in to change notification settings - Fork 13
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
Skeleton refactoring #28
Conversation
An unit test ( |
bGrid unit tests in sUt_skeletonOnStreams have been temporary disabled until bGrid is extended to multi-GPUs. |
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.
Looking good! I just left a minor comment
@@ -53,7 +53,12 @@ LIBNEONCORE_EXPORT extern Logger LoggerObj; | |||
|
|||
} // namespace Neon | |||
|
|||
#if defined( NEON_ACTIVETE_TRACING) |
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 we don't need this. If you want to turn the logging completely off you can just do Neon::globalSpace::LoggerObj.set_level(spdlog::level::off)
There are a couple of failed tests on Windows. Here is the complete report |
Part of the test that are failing are not related to the skeleton. What window system are you using? |
Windows logs
|
New skeleton implementation.
Unit test results on Snoopy (DGX A100)