Skip to content

Conversation

@shaoyuyoung
Copy link

@shaoyuyoung shaoyuyoung commented May 22, 2024

Here are some dead links of resnet in the TVM doc.

one of these dead links has been caught in #17017 and a PR is submitted #17015

This PR fixes other dead links of resnet in tvm.

CC @leandron. When you do something about cache file in S3, maybe you need to pay attention to this part :)

@shaoyuyoung shaoyuyoung changed the title [doc] fix the dead link of resnet [CI] [doc] fix the dead link of resnet May 22, 2024
@yongwww
Copy link
Member

yongwww commented May 22, 2024

@tvm-bot rerun

@shaoyuyoung
Copy link
Author

Well, what went wrong? :(

@tqchen
Copy link
Member

tqchen commented May 24, 2024

@tvm-bot rerun

onnx = pytest.importorskip("onnx")

model_url = "https://github.com/onnx/models/raw/main/vision/classification/mobilenet/model/mobilenetv2-7.onnx" # pylint: disable=line-too-long
model_url = "https://github.com/onnx/models/raw/main/validated/vision/classification/mobilenet/model/mobilenetv2-7.onnx" # pylint: disable=line-too-long
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems the https://github.com/apache/tvm/blob/main/tests/scripts/request_hook/request_hook.py should be updated.

Copy link
Author

@shaoyuyoung shaoyuyoung May 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think (maybe) it can work now

@shaoyuyoung shaoyuyoung changed the title [CI] [doc] fix the dead link of resnet [CI] [doc] fix the dead link of resnet and mobilenet May 26, 2024
@shaoyuyoung
Copy link
Author

@leandron sry to bother you :(

Can you take a look at this?

I'm not sure I did the right thing

@lhutton1
Copy link
Contributor

lhutton1 commented Jun 7, 2024

Thanks for the patch @shaoyuyoung, I believe this has been superseded by #17015, so I'll close this patch. Please reopen if that's not the case.

@lhutton1 lhutton1 closed this Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants