-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 chainerx test in observation_aggregator #8384
Conversation
tests/chainermn_tests/extensions_tests/test_observation_aggregator.py
Outdated
Show resolved
Hide resolved
Jenkins, test this please! |
Jenkins, test this please! |
Jenkins CI test (for commit 111cf0b, target branch master) succeeded! |
tests/chainermn_tests/extensions_tests/test_observation_aggregator.py
Outdated
Show resolved
Hide resolved
pfnci, test this please. |
Jenkins CI test (for commit 97a9918, target branch master) succeeded! |
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.
LGTM
pfnci, test this please. |
Jenkins CI test (for commit ee785bc, target branch master) succeeded! |
This PR depends on #8508
This PR is a replacement of #8322 by @belldandyxtq
This PR adds ChainerX tests into the observation_aggregator
This PR partially solves #8031. This PR depends on #8321
NOTE: This PR is completed w.r.t. the code, but it causes SEGV depending on the order of test cases in test_observatgion_aggregator.py. I guess it is because of the behavior of ChainerX's device setting. We need a deeper investigation on this issue.fixed!