search

dmlc / gluon-cv

Gluon CV Toolkit
http://gluon-cv.mxnet.io
Apache License 2.0
5.79k stars 1.21k forks source link

Fix additional Apache MXNet branding issues #1747

Closed josephevans closed 2 years ago

josephevans commented 2 years ago
josephevans commented 2 years ago

Thanks for the PR. It seems the change doesn't pass the CI doc test. Is there a permission issue?

Hi, I'm not sure why the CI is not passing. It looks like it was trying to comment on this PR but failed. Can you re-run the CI pipeline again?

bryanyzhu commented 2 years ago

Hi @yinweisu @zhreshold , sorry to interrupt, any idea why the CI test fails on comment on PR? Does this affect our CI pipeline or the website? Thank you.

yinweisu commented 2 years ago

The workflow failed on building doc instead of commenting on PR. I think it failed because the build process took too long and the token granted to github action expired. Not sure why it took so long though

bryanyzhu commented 2 years ago

Hi @josephevans We have fixed the CI pipeline (thanks to @yinweisu ). Can you rebase with master? I will merge your PR once the rebase is done. Thank you.

josephevans commented 2 years ago

Hi @josephevans We have fixed the CI pipeline (thanks to @yinweisu ). Can you rebase with master? I will merge your PR once the rebase is done. Thank you.

Done, thanks for all your help!

github-actions[bot] commented 2 years ago

Job PR-1747-340e381 is done. Docs are uploaded to http://gluon-vision-staging.s3-website-us-west-2.amazonaws.com/PR-1747/340e381/index.html