readthedocs fails to auto-import API docs

[ctb]

See log file - the problem seems to be

WARNING: autodoc: failed to import module 'sourmash'; the following exception was raised:
No module named 'sourmash._lowlevel'

this is presumably because the rust stuff is not built?

[ctb]

ok, but it is built? odd. still looking.

[ctb]

even odder, it works on my test branch... https://sourmash.readthedocs.io/en/refactor-minhash/api.html

[luizirber]

I think this is related to our non-src/ repo layout. I had similar issues in #696 (with coverage counting not lines in the installed package, but in the sourmash/ dir), and in this case it's because the sourmash package is being installed... but it's the sourmash/ dir that is being used during runtime.

Short term solution: install the package with -e . (and then _lowlevel.py will be available).

But I think we should move to src/ layout =]

[luizirber]

Update: I don't think readthedocs supports -e . or develop installs...

[ctb]

On Sat, Feb 08, 2020 at 07:15:38PM -0800, Luiz Irber wrote: But I think we should move to `src/` layout =]

I'm good with this for 4.0.

[ctb]

this seems to be working now. see: https://sourmash.readthedocs.io/en/latest/api.html