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 links to pass check_links #826

Closed
1 task
atteggiani opened this issue Oct 16, 2024 · 14 comments · Fixed by #836
Closed
1 task

fix links to pass check_links #826

atteggiani opened this issue Oct 16, 2024 · 14 comments · Fixed by #836
Assignees

Comments

@atteggiani
Copy link
Contributor

atteggiani commented Oct 16, 2024

Fix the links urls in the MED section (and a few other sections) to pass check_links action.

Requirements:

  • All links should be formatted in markdown (if possible)
  • All URLs should be formatted as an absolute URL, starting with a forward slash (/) that represents the base website (https://access-hive.org.au/ or http://localhost:<port-number>/ if you are deploying locally through mkdocs serve)
  • All URLs should be relative to the website navigation tree set in the mkdocs.yaml file (this means also that no links should have the index.md file in it).

Two practical examples:

Wrong link --> link doesn't work

The reference link is the first link in https://access-hive.org.au/tutorials/ilamb:

The International Land Model Benchmarking (ILAMB) system is a python framework used to quantitatively compare a defined set of observable variables with a number of land models. For more information, check [ACCESS-NRI ILAMB-workflow at NCI](/model_evaluation/model_evaluation_on_gadi/model_evaluation_on_gadi_ilamb).

This link points to /model_evaluation/model_evaluation_on_gadi/model_evaluation_on_gadi_ilamb. This means there should be a page at https://access-hive.org.au/model_evaluation/model_evaluation_on_gadi/model_evaluation_on_gadi_ilamb, but if you click on this link it gives you a 404 error (page doesn't exist).
The link should be instead /model_evaluation/evaluation_on_gadi/ilamb_workflow/, pointing to https://access-hive.org.au/model_evaluation/evaluation_on_gadi/ilamb_workflow/

Internal relative link (and also index.md file in it) --> link works but is falsely failed by the link_checker

The reference link is in https://access-hive.org.au/community_resources/community_med/community_med_recipes:

Here, we collate lists of useful resources for Model Evaluation and Diagnostics (MED). Contrary to the supported content of our [ACCESS-NRI Model Evaluation pages](../../model_evaluation/index.md), the information below is not supported by ACCESS-NRI unless stated otherwise.

The link is ../../model_evaluation/index.md (note the ../../ at the beginning and the index.md at the end).
In this case the change is easier:

  1. Go to the linked page (if it works, otherwise refer to the example above)
  2. Copy the part after the base website
  3. Paste that as the new link

In the case above, the new link should be /model_evaluation.

Tasks to fix the issue

@atteggiani
Copy link
Contributor Author

@flicj191 Added more specification on this.
To trigger the check_links action you only need to open a PR (also as a draft for testing while the links are changed).

@atteggiani atteggiani changed the title fix links in MED to pass check_links fix links to pass check_links Oct 17, 2024
@pboubel
Copy link
Contributor

pboubel commented Oct 25, 2024

The ACCESS-NRI Intake Catalog section in the tutorials has a broken link but there is not yet a corresponding section in 'Model Evaluation'. Do we just want to link this to the Intake Catalog docs? (https://access-nri-intake-catalog.readthedocs.io/en/latest/index.html)

@pboubel pboubel closed this as completed Oct 25, 2024
@pboubel pboubel reopened this Oct 25, 2024
@atteggiani
Copy link
Contributor Author

The ACCESS-NRI Intake Catalog section in the tutorials has a broken link but there is not yet a corresponding section in 'Model Evaluation'. Do we just want to link this to the Intake Catalog docs? (https://access-nri-intake-catalog.readthedocs.io/en/latest/index.html)

Hi @pboubel,
can you please point to the page in the Hive Docs where there is the broken link?

@pboubel
Copy link
Contributor

pboubel commented Oct 25, 2024

The ACCESS-NRI Intake Catalog section in the tutorials has a broken link but there is not yet a corresponding section in 'Model Evaluation'. Do we just want to link this to the Intake Catalog docs? (https://access-nri-intake-catalog.readthedocs.io/en/latest/index.html)

Hi @pboubel, can you please point to the page in the Hive Docs where there is the broken link?

yes, here: https://access-hive.org.au/tutorials/intake/

@pboubel
Copy link
Contributor

pboubel commented Oct 25, 2024

Also, there is a link to the 'recipe gallery' here https://access-hive.org.au/community_resources/community_med/community_med_recipes/
Doesn't seem to exist anywhere that I have looked

@flicj191
Copy link
Contributor

hi @pboubel, the link in the intake tutorials page is supposed to point to here:
https://access-hive.org.au/model_evaluation/data/model_catalogs/
the page was renamed to cover model data more broadly for the MED restructure before going into access-nri intake

you can leave the one in the community resources section, it was done a while ago and may need to look into the history, there's a separate issue to review it and organise so we will come to that with that issue #827 and #831

thanks for looking at this!

@atteggiani
Copy link
Contributor Author

atteggiani commented Oct 25, 2024

you can leave the one in the community resources section, it was done a while ago and may need to look into the history, there's a separate issue to review it and organise so we will come to that with that issue #827 and #831

Even though we are going to properly address the community resources section in the issues linked, after this issue is fixed the link checker should pass.
Therefore, for the moment, I would say to simply remove the sentence: "We are trying to ingest more and more model evaluation and diagnostics recipes in your currated recipe gallery on this website." in that page.

Thank you @pboubel!

@pboubel pboubel mentioned this issue Oct 28, 2024
1 task
@pboubel
Copy link
Contributor

pboubel commented Oct 28, 2024

@atteggiani So I have a few remaining cases that are failing check_links that I'm not sure how to deal with (see this run: https://github.com/ACCESS-NRI/access-hive.org.au/actions/runs/11545655642):

[ERR] %[email protected] | Failed: Unreachable mail address: %[email protected]: Invalid: Email doesn't exist or is syntactically incorrect

check_links is mistaking this bit of text in the terminal window for an email it needs to check

@atteggiani
Copy link
Contributor Author

  • Internal links that point to images or .md files (and one external link) that no longer exist, but the text/code that references these links has been commented out E.g. all the commented-out html code in docs/model_evaluation/evaluation_on_gadi/esmvaltool_workflow.md

Please just delete all the commented out text generating issues with links.

I am not sure where those links wanted to point to.
Do you have any idea @flicj191?

That link has been removed because the UM partneship has been "superseeded" by the Momentum partneship for the next generation models.

I would substitute the link with the following link to a document about the UM partneship: https://www.metoffice.gov.uk/binaries/content/assets/metofficegovuk/pdf/research/foundation-science/um_partnership_handout.pdf

  • This error:

[ERR] %[email protected] | Failed: Unreachable mail address: %[email protected]: Invalid: Email doesn't exist or is syntactically incorrect

check_links is mistaking this bit of text in the terminal window for an email it needs to check

This needs to be excluded from the check.
It can be done by adding the following line:

'^%intel@',

to the exclude field of the lychee-config.toml:

exclude = [
'^https://code\.metoffice\.gov\.uk',
'^https://web\.archive\.org/web/',
'^http://127\.0\.0\.1:8000',
'^https://www.linkedin.com/',
]

@flicj191
Copy link
Contributor

I am not sure where those links wanted to point to. Do you have any idea @flicj191?

Sorry I missed this earlier!!
You mean in this section: https://access-hive.org.au/model_evaluation/evaluation_on_gadi/ilamb_workflow/#loading-the-ilamb-workflow-modules it looks like these were copied and condensed from the documentation: https://ilamb-workflow.readthedocs.io/en/latest/source/ILAMB.html#ilamb-cmip-confrontations-maintained-by-access-nri so those links don't go anywhere, I'll flag this with @rhaegar325

But looking at that page, it looks like its an old version missing edits done from a review, I've found a newer version from my branches and and added to another branch and removed those broken links.
@pboubel can you add the ilamb page from the branch here? https://github.com/ACCESS-NRI/access-hive.org.au/blob/flick/ilamb_edit/docs/model_evaluation/evaluation_on_gadi/ilamb_workflow.md

@pboubel
Copy link
Contributor

pboubel commented Nov 21, 2024

Thanks @flicj191, I changed the ilamb page to what you have in that branch, with one small fix to a broken link (getting_started/first_steps/ -> getting_started/set_up_nci_account/).

@atteggiani However when I run link checker now there are 4 new errors that did not appear before (https://github.com/ACCESS-NRI/access-hive.org.au/actions/runs/11945377713).

These links all work fine for me and they passed the check last week so they may just be temporary errors? Let me know if you'd like me to do anything about them or just submit the pull request.

@atteggiani
Copy link
Contributor Author

atteggiani commented Nov 21, 2024

These links all work fine for me and they passed the check last week so they may just be temporary errors? Let me know if you'd like me to do anything about them or just submit the pull request.

Hmm it might have been a temporary problem as they do work fine for me too.
Can you please try to run the link checker again and see if it still gets them as errors?

@pboubel
Copy link
Contributor

pboubel commented Nov 22, 2024

These links all work fine for me and they passed the check last week so they may just be temporary errors? Let me know if you'd like me to do anything about them or just submit the pull request.

Hmm it might have been a temporary problem as they do work fine for me too. Can you please try to run the link checker again and see if it still gets them as errors?

@atteggiani The 504 errors have disappeared but no dice for https://www.lanl.gov/... Odd because it was not generating the 404 errors before.

@atteggiani
Copy link
Contributor Author

These links all work fine for me and they passed the check last week so they may just be temporary errors? Let me know if you'd like me to do anything about them or just submit the pull request.

Hmm it might have been a temporary problem as they do work fine for me too. Can you please try to run the link checker again and see if it still gets them as errors?

@atteggiani The 504 errors have disappeared but no dice for https://www.lanl.gov/... Odd because it was not generating the 404 errors before.

This is so weird and I'm not sure why it is happening.

I would say for now let's not worry about it.
If you think the PR is ready (disregarding this problem mentioned here), please go on and mark it as reviewable and I'll review it (probably on Monday).
Thank you! 😄

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 a pull request may close this issue.

3 participants