You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tvm.apache.org by 雾雨魔理沙 <no...@github.com> on 2019/07/10 21:45:06 UTC

[dmlc/tvm] [RFC] Deprecation (#3530)

Currently, when we want to deprecate a feature, we add LOG(WARNING) or warning at python side. However, they do slowly creep back into the master's CI.
For example, I once removed all deprecation warning at build_module.py:186, but it got reintroduced to the CI.
![image](https://user-images.githubusercontent.com/3397377/61006726-d6870f80-a31f-11e9-985e-e127c2924729.png)
I proposed that we add a deprecate(msg, version, ci_count) to battle this.
`msg` is the warning messaged print.
`version` is the version number that this feature will go completely unsupported. If the current version is greater or equal then `version`, it will be an error rather than a warning.
`ci_count` is the number of time this function could be called. after it got exceed it will fail the ci. this will stop pr that add more deprecated used to pass the ci.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/dmlc/tvm/issues/3530

Re: [dmlc/tvm] [RFC] Deprecation (#3530)

Posted by Tianqi Chen <no...@github.com>.
Closed #3530.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/dmlc/tvm/issues/3530#event-2633603942

Re: [dmlc/tvm] [RFC] Deprecation (#3530)

Posted by Tianqi Chen <no...@github.com>.
Close this RFC for now as it has been inactive

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/dmlc/tvm/issues/3530#issuecomment-531396557

Re: [dmlc/tvm] [RFC] Deprecation (#3530)

Posted by Tianqi Chen <no...@github.com>.
It is a bit harder to build into the CI infra, but would be great if we could work to fix deprecation warnings as much as we can

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/dmlc/tvm/issues/3530#issuecomment-510285357