DickJC123 opened a new issue #19082:
URL: https://github.com/apache/incubator-mxnet/issues/19082
While we generally insist that a new feature come with unittests at time of
introduction, it's also important to ensure those tests are not removed or
marked 'skipped' for too long to protect the feature from falling into
disrepair.
Back in February, @anirudh2290's PR
https://github.com/apache/incubator-mxnet/pull/16654 introduced multi-threaded
inferencing, with new files ./src/imperative/cached_op_threadsafe.{h,cc} and a
unittest under ./tests/cpp/thread_safety/. However in June, @eric-haibin-lin's
PR https://github.com/apache/incubator-mxnet/pull/18598 "graph executor c api
removal" removed the test files, but left the cached_op_threadsafe.{h,cc}
files. Is there some other unittest of multi-threaded inferencing, and with
which front end? If not, can the old test be reinstated or replaced?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
