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

Broken links to new EUMETSAT website #1484

Closed
gerritholl opened this issue Dec 11, 2020 · 3 comments · Fixed by #1827
Closed

Broken links to new EUMETSAT website #1484

gerritholl opened this issue Dec 11, 2020 · 3 comments · Fixed by #1827

Comments

@gerritholl
Copy link
Collaborator

gerritholl commented Dec 11, 2020

Describe the bug

In various places in satpy there are broken links to the new EUMETSAT website. For example, the documentation for the seviri_l1b_hrit contains a broken link to the level 1.5 fileformat description on the EUMETSAT website.

To Reproduce

Expected behavior

I expect that the link connects me with the image format description.

Actual results

In reality I get 404 - Page not found.

Additional context

Arguably this bug in in the EUMETSAT website because cool URIs dont change, but regrettably it's going to be up to pytroll to update the link.

We should probably search through all the docs and source code of all pytroll packages for links to the EUMETSAT website to check and potentially update them all.

@helgaweb
Copy link

@gerritholl It also holds for link in the slstr_l1b reader

# https://www.eumetsat.int/website/wcm/idc/idcplg?IdcService=GET_FILE&dDocName=PDF_S3A_PN_SLSTR_L1_06&RevisionSelectionMethod=LatestReleased&Rendition=Web

@ameraner
Copy link
Member

The links in seviri_l1b_hrit, seviri_l1b_nc, seviri_base and _geos_area will be fixed with #1485 as that PR is already touching those files (I started fixing them before seeing this issue). But I'm sure there are more around! So maybe you can change this issue to be general about all EUM links?

@gerritholl gerritholl changed the title Broken link to EUMETSAT website in seviri_l1b_hrit documentation Broken links to new EUMETSAT website Dec 14, 2020
@gerritholl
Copy link
Collaborator Author

@ameraner Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants