Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[skip ci] Use Stanford Cars mirror to fix CI docs build #11812

Merged
merged 1 commit into from
Jun 22, 2022

Conversation

guberti
Copy link
Member

@guberti guberti commented Jun 22, 2022

As part of gallery/how_to/work_with_microtvm/micro_train.py, we download a part of the Stanford Cars dataset from https://ai.stanford.edu/. Unfortunately, https://ai.stanford.edu/ is currently down, with no indication of when it will be back up. This PR provides a temporary CI fix by switching this tutorial to use a mirror link. Note that the link is not an exact mirror (containing all files, not just the train ones), so a few other lines had to be adjusted too.

cc @Mousius @areusch @driazati

@mehrdadh mehrdadh changed the title [ci] Use Stanford Cars mirror to fix CI docs build [skip-ci] Use Stanford Cars mirror to fix CI docs build Jun 22, 2022
@mehrdadh
Copy link
Member

added skip-ci to unlock the CI quickly.

@guberti
Copy link
Member Author

guberti commented Jun 22, 2022

I'm surprised http://ai.stanford.edu/ went down - Pytorch and Tensorflow both use this link for their datasets packages, so it seems to be trusted. I'm not sure what the best long-term fix for this issue is - see the discussion on #11811 for more.

@driazati driazati changed the title [skip-ci] Use Stanford Cars mirror to fix CI docs build [skip ci] Use Stanford Cars mirror to fix CI docs build Jun 22, 2022
Copy link
Member

@driazati driazati left a comment

Choose a reason for hiding this comment

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

lgtm assuming it works locally

@guberti
Copy link
Member Author

guberti commented Jun 22, 2022

Can confirm the fix works locally. You can try it yourself on Google Colab at https://colab.research.google.com/drive/1ycMRXm3ulghiBKsdrp1wiKdFpmXNej9o?usp=sharing (click "Run All").

@driazati
Copy link
Member

also fyi to skip Jenkins [skip ci] must be prepended to the PR title, but to skip GitHub actions jobs it must be on the commit message subject (so to skip CI for this commit you could git commit --amend and add [skip ci] then re-push)

@guberti
Copy link
Member Author

guberti commented Jun 22, 2022

also fyi to skip Jenkins [skip ci] must be prepended to the PR title, but to skip GitHub actions jobs it must be on the commit message subject (so to skip CI for this commit you could git commit --amend and add [skip ci] then re-push)

Good to know, thanks!

@mehrdadh mehrdadh merged commit 32d16eb into apache:main Jun 22, 2022
blackkker pushed a commit to blackkker/tvm that referenced this pull request Jul 7, 2022
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.

3 participants