Skip to content

Conversation

@melissalinkert
Copy link
Member

Fixes ome/ome-model#156. Can re-add any of these if the build ends up failing.

Note that the only reference to http://www.definiens.com is also removed, as I don't think it makes sense to keep a broken link for something that's permanently gone. https://web.archive.org/web/20190927000502/https://www.definiens.com/ is I believe the last valid archive page. Happy to follow some other approach, but this is sort of the same problem as ome/bio-formats-documentation#409.

This software is no longer available, and no relevant websites are still online.
@sbesson
Copy link
Member

sbesson commented Mar 18, 2025

From the HEAD of this PR I have

(venv) sbesson@Sebastiens-MacBook-Pro-3 ome-model-documentation % make html linkcheck | grep broken
(  ome-tiff/index: line   63) broken    http://www.cellimagelibrary.org/ - HTTPSConnectionPool(host='www.cellimagelibrary.org', port=443): Max retries exceeded with url: / (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1006)')))
(   ome-tiff/data: line   28) broken    https://eliceirilab.org/software/wiscscan/ - 404 Client Error: Not Found for url: https://eliceirilab.org/software/wiscscan/
(   ome-tiff/code: line   68) broken    https://www.awaresystems.be/imaging/tiff/bigtiff.html - 404 Client Error: Not Found for url: https://www.awaresystems.be/imaging/tiff/bigtiff.html
(  ome-tiff/tools: line   27) broken    https://imagej.nih.gov/ij/ - 404 Client Error: Not Found for url: https://imagej.nih.gov/ij/
(  ome-tiff/index: line   45) broken    https://www.cytivalifesciences.com/ - 403 Client Error: Forbidden for url: https://www.cytivalifesciences.com/
(  ome-tiff/index: line   44) broken    https://www.zeiss.com/microscopy/int/home.html - 404 Client Error: Not Found for url: https://www.zeiss.com/microscopy/int/home.html
make: *** [linkcheck] Error 1

Are these builds running anywhere since the documentation has been extracted to this repository @jburel ? Or is the linkcheck expected to be executed as part of the GitHub workflow (currently disabled)?

@melissalinkert
Copy link
Member Author

Note changes to awaresystems links are in line with ome/bio-formats-documentation#387.

@sbesson sbesson merged commit f8d48e8 into ome:main Mar 19, 2025
1 check passed
@sbesson
Copy link
Member

sbesson commented Mar 19, 2025

@jburel what is the deployment strategy for this documentation? I see https://github.com/ome/ome-model-documentation/releases/tag/2023.06.29 was pushed as of the migration. Is https://ome-model.readthedocs.io/en/stable/index.html updated on tag builds? And should the tags follow a calendar versioning scheme?

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.

Review previously backlisted URLs

2 participants