readthedocs are missing dependencies

[pmav99]

E.g. https://ensembleperturbation.readthedocs.io/en/latest/client.html#perturb-tracks

There are more similar errors

[SorooshMani-NOAA]

This issue seems to be due to pooch dependency in adcircpy, but adcircpy already defines pooch as a dependency:
https://github.com/oceanmodeling/adcircpy/blob/main/pyproject.toml

@pmav99 do you have any suggestion as why it's still missing that dependency when generating the documentation?
https://github.com/noaa-ocs-modeling/EnsemblePerturbation/blob/main/.readthedocs.yml

[pmav99]

Not sure. I guess that adding pooch as a dependency will probably fix the issue, but we will still have not understood the underlying issue.

For the record there is another similar error mentioning cartopy which seems to be an optional dependency. I guess that this should be added to the documentation extras, too.

[pmav99]

Hmm... could it be that the adcircpy version you get on readthedocs is a rather old one? I.e. one before pooch became a mandatory deendency of adcircpy?

oceanmodeling/adcircpy@5d3c9e9

[pmav99]

Your readthedocs builds have been failing for the past 20 months, too: https://readthedocs.org/projects/ensembleperturbation/builds/?page=2

[SorooshMani-NOAA]

Thanks for your comments. I haven't been paying attention to the readthedocs logs! That's a good place to start for me.

[SorooshMani-NOAA]

@zacharyburnett is ensembleperturbation's readthedoc project still under your account? I can't seem to add it in my rtd projects because of existing project name.

[zacharyburnett]

oh yes, it looks like all of these are failing because of missing dependencies in the extras section; @SorooshMani-NOAA I'll add you as a maintainer as well

[zacharyburnett]

#117
noaa-ocs-modeling/CoupledModelDriver#161
noaa-ocs-modeling/nemspy#62
oceanmodeling/adcircpy#181

I've made these PRs to add dunamai to the docs dependencies (as per the most recent failing build logs), but I'm still not sure why pooch is not available at runtime to this docs build. It might be that that was the last successful build and thus represents an outdated version of the docs

Sorry about that! I also did not pay attention to the build logs

[pmav99]

It might be that that was the last successful build and thus represents an outdated version of the docs

That's probably the case.

[SorooshMani-NOAA]

@zacharyburnett thank you, I was the one who should have paid more attention!!

[SorooshMani-NOAA]

Still after #117 and #119 the issue is not fixed:

I need to add UDUNITS2 as well!

[SorooshMani-NOAA]

Fixed by #121