samskalicky commented on issue #17987: Dynamic subgraph accesses elements of 
empty vector
URL: 
https://github.com/apache/incubator-mxnet/issues/17987#issuecomment-610186382
 
 
   Hi @leezu, @rondogency and I have discussed unifying the resource requests 
for custom ops and custom subgraph ops and have made the change in #17762 to 
remove `DefaultSubgraphOpResourceRequest`. This change will fix the issue such 
that `ctx.requested` will always have at least one element.
   
   But @rondogency being the overly cautious expert that he is also changed it 
to `ctx.requested.at(0)` and added `CHECK(ctx.requested.size() >= 2)` too. 
   
   Thanks for investigating this issue!

----------------------------------------------------------------
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

Reply via email to