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

[Fix] Unified links of readthedocs in docs #1119

Merged
merged 3 commits into from
Dec 10, 2021

Conversation

MengzhangLI
Copy link
Contributor

@MengzhangLI MengzhangLI commented Dec 9, 2021

Related PR: open-mmlab/mim#89
https://github.com/open-mmlab/mmcv/blob/master/docs/conf.py#L101.

Note:
(1) MMSegmentation has adopted pytorch_sphinx_theme from #879.

(2) Currently, some repos do not have Chinese readthe docs, they are mmhuman3d, mmflow and mmgeneration, thus just keep their url. We may change it in the future.

Copy link
Collaborator

@Junjun2016 Junjun2016 left a comment

Choose a reason for hiding this comment

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

The menu is not correct.
Please check the pre-builded readthedoc.

@gaotongxiao
Copy link

These changes are no longer required and can be simplified as in open-mmlab/mmocr#655

@MengzhangLI MengzhangLI self-assigned this Dec 10, 2021
@MengzhangLI MengzhangLI added the Merging PR waited for merging label Dec 10, 2021
@Junjun2016 Junjun2016 merged commit ff044c5 into open-mmlab:master Dec 10, 2021
@MengzhangLI MengzhangLI removed the Merging PR waited for merging label Dec 14, 2021
@MengzhangLI MengzhangLI deleted the use-pytorch-sphinx-theme branch February 1, 2022 03:18
bowenroom pushed a commit to bowenroom/mmsegmentation that referenced this pull request Feb 25, 2022
* change readthedocs

* change readthedocs

* change readthedocs
aravind-h-v pushed a commit to aravind-h-v/mmsegmentation that referenced this pull request Mar 27, 2023
* Remove the hub token

* replace repos

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