-
Notifications
You must be signed in to change notification settings - Fork 1
Update decoding_mom6.md #9
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
base: main
Are you sure you want to change the base?
Conversation
|
@atteggiani the lychee/link checker complained about a few re-directs and other minor looking things. I have a feeling most of those errors are ok, can I just ignore them? Also it reports that the preview built: But when I click on it I get: |
There are 14 errors in total. The redirects don't cause errors. Some of the 404 errors are actually non-existent pages. I would control the related links in the referenced files. The To reference specific functions (or in general portions of the text), you should assign an HTML For example I can assign the So for this case, all link-checker failures should be fixed (As it's very often the case). |
|
Thanks @atteggiani that's really helpful. So I've fixed the reference ones (well, a hack for now -- e28278f) but I'm still a little confused by these four remaining ones:
Here's the details |
tiny tweaks
The solution is not to use code blocks with backticks (e.g., `...`), as this completely changes the rendering you want. This is not a Latex document, so not every Latex notation is supported. Latex references are not supported.
For these you need to update the # Change the URL segment for pages in MkDocs to hide the "/pages" prefix, so that pages inside the
# "pages" directory are served directly at the root URL.
from mkdocs.plugins import event_priority
@event_priority(-100)
def on_page_markdown(markdown, *, page, config, files):
if page.file.url.startswith("pages/"):
page.file.url = page.file.url.removeprefix("pages/")
page.file.dest_uri = page.file.dest_uri.removeprefix("pages/")
page.file.abs_dest_path = page.file.abs_dest_path.removeprefix("pages/")
+ page.canonical_url = config.site_url + page.file.urlAnd also change decoding-om3/documentation/mkdocs.yml Line 7 in 5ea94d7
- site_url: !ENV [SITE_URL, "https://access-community-hub.github.io/decoding-om3/"]
+ site_url: !ENV READTHEDOCS_CANONICAL_URL |
Clarified instructions for finding model versions in the release database and GitHub.
|
@chrisb13 I made some changes that were intended to be suggestions but I accidentally commiteed them sorry! Hopefully they are acceptable. |
Awesome, all good on committing them directly (actually my preference for this kind of thing). I'll fix up the links that Davide has helped me with, then merge. |
as suggested by @atteggiani #9 (comment)
as suggested by @atteggiani #9 (comment)
Notes taken by me for @angus-g and @jbisits great intro this morning into finding source code related to mom6/om3 builds.
@angus-g, I've just sent you an invite to the repo' (I couldn't add you as a reviewer I think until you've accepted but feel free to review if you wish.)
Note: I'm only looking for a review on this file:
documentation/docs/pages/decoding_mom6.md. You can ignore the other changes.Here's the rendered version:
https://decoding-access-om3--9.org.readthedocs.build/decoding_mom6/#how-do-i-find-out-which-version-of-mom6-i-am-currently-using