[GitHub] [incubator-mxnet] szha commented on issue #16553: Fix for wrong reqs set after switching from training to inference

2019-10-20 Thread GitBox
szha commented on issue #16553: Fix for wrong reqs set after switching from 
training to inference
URL: https://github.com/apache/incubator-mxnet/pull/16553#issuecomment-544339535
 
 
   sounds good to both


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-mxnet] szha commented on issue #16553: Fix for wrong reqs set after switching from training to inference

2019-10-20 Thread GitBox
szha commented on issue #16553: Fix for wrong reqs set after switching from 
training to inference
URL: https://github.com/apache/incubator-mxnet/pull/16553#issuecomment-544334924
 
 
   As follow-up:
   1. ideally we should be able to test for this explicitly. given that we lack 
the utility for such testing, I want to avoid blocking this fix on this basis
   2. the keys/magic strings should ideally be centralized, maybe in an enum.


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:
us...@infra.apache.org


With regards,
Apache Git Services