yotarazona / scikit-eo

A Python package for Remote Sensing Data Analysis
https://yotarazona.github.io/scikit-eo/
Other
107 stars 16 forks source link

None of the links on the API webpages work #14

Closed dbuscombe-usgs closed 3 months ago

dbuscombe-usgs commented 4 months ago

ping https://github.com/openjournals/joss-reviews/issues/6692

Every single 'SOURCE' link on the API pages of the website is broken (404 - Not found)

yotarazona commented 3 months ago

Could you please share the link is not working?. I took a look at all the links and everything is working.

dbuscombe-usgs commented 3 months ago

All of the 'SOURCE' links are broken

I just checked and they are all still down, e.g.

https://yotarazona.github.io/scikit-eo/scikeo/pca.py#L0

yotarazona commented 3 months ago

Hi @dbuscombe-usgs. Please find attached a gif where you can see all the links related to the package are working.

gif

Maybe you have some old version of the website in your browser cache. Best.

dbuscombe-usgs commented 3 months ago

image

Thanks for the gif. I'm only talking about the SOURCE links on the API pages. It's not a big deal, but probably an easy fix. My cache is clear. This is a real bug.

Nowosad commented 3 months ago

Hi @dbuscombe-usgs -- I believe that you are using a wrong website. The correct one is https://yotarazona.github.io/scikit-eo/

dbuscombe-usgs commented 3 months ago

Thanks, but that's the same site. So bizarre... I click on that link in a browser that I never use, and I still have a 404 error

image

KBodolai commented 3 months ago

I'm getting the same as @dbuscombe-usgs above. I think it's because the links it points to are something like:

https://yotarazona.github.io/scikit-eo/scikeo/mla.py#L23

where it should be:

https://github.com/yotarazona/scikit-eo/blob/main/scikeo/mla.py#L23

yotarazona commented 3 months ago

Hi @KBodolai, thanks for this unnoticed tip!. All links were updated now :).

KBodolai commented 3 months ago

have the docs been deployed? I'm still getting the same issues when I try to get to the source code.

yotarazona commented 3 months ago

Hi @KBodolai, sorry. It was fixed.

KBodolai commented 3 months ago

I can confirm, great to have this working :)