[GitHub] [tvm] huajsj edited a comment on pull request #9108: [Runtime] Pipeline Executor Second patch, configuration load and executor export/import.

[GitBox <gi...@apache.org>]

huajsj edited a comment on pull request #9108:
URL: https://github.com/apache/tvm/pull/9108#issuecomment-934084988


   > @huajsj Why `mod_index` etc starts from 1? What does index 0 corresponds to? Unless there is a good reason for this weird design, please make all indices start from 0.
   
   @masahi, mod_index coming from configure file, and to make the configuration file more user friendly , the mod_index start from 1 instead of 0, index 0 means , this module is global module that represent whole pipeline , then the input/output of module 0 is whole pipeline input/output.  I can do the change to make it start from 0 if that cause confuse.


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

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org