You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2022/04/18 13:20:43 UTC

[GitHub] [tvm] BBuf opened a new pull request, #11036: Add oneflow fronted tutorials

BBuf opened a new pull request, #11036:
URL: https://github.com/apache/tvm/pull/11036

   - [x] upgrade oneflow version to v0.7.0 and fix some bug.
   - [x] add tutorial for oneflow fronted and run success.
   
   ![image](https://user-images.githubusercontent.com/35585791/163814079-89703c82-5530-429d-bed8-526326133d88.png)
   
   


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


[GitHub] [tvm] masahi commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
masahi commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1107349983

   @BBuf Merged! finally


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


[GitHub] [tvm] BBuf commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
BBuf commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1105362819

   @masahi this pr can be merged too, thanks!


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


[GitHub] [tvm] BBuf commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
BBuf commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1101905292

   Hello, I want to upgrade the version of oneflow to 0.7.0 and solve the bug caused by the interface change. The local tests has passed. But the tests of running oneflow front-end in ci would fail. It seems that the mirror does not successfully install the oneflow 0.7.0 version. Can you give me some advice? thanks!
   
   @masahi 


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


[GitHub] [tvm] masahi commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
masahi commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1101910215

   Changing the docker script won't immediately change CI environments. We need to go through proper steps to update CI.
   
   Please send the docker script change in a separate PR.


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


[GitHub] [tvm] BBuf commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
BBuf commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1103340074

   @masahi  Hey, there is an error that the flowvision package cannot be found when build docs, but I have pip flowvision in https://github.com/apache/tvm/pull/11052, I don't know why? And the oneflow front-end test also failed, probably because the upgraded image didn't work. It seems that the version of oneflow is still v0.6.0. Do you have any idea? Thanks!


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


[GitHub] [tvm] masahi merged pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
masahi merged PR #11036:
URL: https://github.com/apache/tvm/pull/11036


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


[GitHub] [tvm] BBuf commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
BBuf commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1101910658

   > Changing the docker script won't immediately change CI environments. We need to go through proper steps to update CI.
   > 
   > Please send the docker script change in a separate PR.
   
   ok, thanks.


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


[GitHub] [tvm] BBuf commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
BBuf commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1103488770

   > CI update needs several steps, changing the docker script is only the first one. The next step is to wait until a new nightly image, containing the change from your PR yesterday, is pushed to https://hub.docker.com/r/tlcpackstaging/ci_gpu/tags.
   > 
   > Then, I'll test that image on the branch https://github.com/apache/tvm/tree/ci-docker-staging. After we pass CI tests on that branch, we'll open a PR to `main` updating `https://github.com/apache/tvm/blob/ci-docker-staging/Jenkinsfile`. CI update is complete after `Jenkinsfile` change is merged.
   
   Ok, thank you!


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


[GitHub] [tvm] masahi commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
masahi commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1103835161

   Running oneflow tests with the new oneflow version resulted in some errors https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/ci-docker-staging/252/pipeline/340
   
   Please take a look


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


[GitHub] [tvm] masahi commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
masahi commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1103436492

   CI update needs several steps, changing the docker script is only the first one. The next step is to wait until a new nightly image, containing the change from your PR yesterday, is pushed to https://hub.docker.com/r/tlcpackstaging/ci_gpu/tags.
   
   Then, I'll test that image on the branch https://github.com/apache/tvm/tree/ci-docker-staging. After we pass CI tests on that branch, we'll open a PR to `main` updating `https://github.com/apache/tvm/blob/ci-docker-staging/Jenkinsfile`. CI update is complete after `Jenkinsfile` change is merged.


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


[GitHub] [tvm] masahi commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
masahi commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1106807721

   Another related failure, please run another job...


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


[GitHub] [tvm] BBuf commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
BBuf commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1103939308

   > Running oneflow tests with the new oneflow version resulted in some errors https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/ci-docker-staging/252/pipeline/340
   > 
   > Please take a look
   
   
   
   > 
   
   I want to confirm whether https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/ci-docker-staging/252/pipeline/340 is running this pr(https://github.com/apache/tvm/pull/11036),  because this pr has changed some bug to adopt oneflow v0.7.0 [in](https://github.com/apache/tvm/pull/11036/files#diff-f157bdee2a307b68e8221725a49b304a12860237afaa562d8d85eaf62e0b7817R274-R1707) 。


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


[GitHub] [tvm] BBuf commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
BBuf commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1103931391

   > Running oneflow tests with the new oneflow version resulted in some errors https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/ci-docker-staging/252/pipeline/340
   > 
   > Please take a look
   
   I have now reinstalled the oneflow v0.7.0 cpu version locally and run this ci failed sample 100 times but without any failed in locally, it's strange.


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


[GitHub] [tvm] BBuf commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
BBuf commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1107159931

   > Another unrelated failure, please run another job...
   
   ok


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


[GitHub] [tvm] BBuf commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
BBuf commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1107350268

   > @BBuf Merged! finally
   
   nice, thanks!


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


[GitHub] [tvm] BBuf commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
BBuf commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1101915290

   > Changing the docker script won't immediately change CI environments. We need to go through proper steps to update CI.
   > 
   > Please send the docker script change in a separate PR.
   
   I have solved it in this pr: https://github.com/apache/tvm/pull/11052


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


[GitHub] [tvm] BBuf commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
BBuf commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1106334142

   > @BBuf Please run another job, there is unrelated failure on MaxOS build. Glad that there is no error from running oneflow tutorial.
   
   Ok, thanks!


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


[GitHub] [tvm] masahi commented on pull request #11036: Add oneflow fronted tutorials

Posted by GitBox <gi...@apache.org>.
masahi commented on PR #11036:
URL: https://github.com/apache/tvm/pull/11036#issuecomment-1106332805

   @BBuf Please run another job, there is unrelated failure on MaxOS build. Glad that there is no error from running oneflow tutorial.


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