mne-tools / mne-connectivity

Connectivity algorithms that leverage the MNE-Python API.
https://mne.tools/mne-connectivity/dev/index.html
BSD 3-Clause "New" or "Revised" License
68 stars 34 forks source link

Sphinx docs gallery: "There is still something wrong with the scraping" #5

Closed adam2392 closed 3 years ago

adam2392 commented 3 years ago

There is still something wrong with the scraping:

https://29-369628653-gh.circle-artifacts.com/0/dev/auto_examples/index.html

Do you want help fixing that? It's probably a pain...

Originally posted by @larsoner in https://github.com/mne-tools/mne-connectivity/issues/4#issuecomment-862634372

larsoner commented 3 years ago

I'll take a look early next week. In the meantime it's hopefully not a blocker

adam2392 commented 3 years ago

So I'm having trouble building the docs locally on my Mac and on my Linux workstation (surprisingly)... Basically it's hung on the mne_inverse_envelope_correlation.py sphinx generation of the gallery step. It's fired up all the other plots successfully. I followed the men-python installation steps, so not sure if I'm just messing things up. Everything else, style, unit tests and such all work.

I was wondering if you have any advice on how to make a release then if I can't upload the docs as specified here: https://github.com/mne-tools/mne-bids/wiki/How-to-make-a-release?

Is there a way to "deploy the docs" via a CI build instead, which at least seems to work, so I can proceed w/ the release?

larsoner commented 3 years ago

Yes I would set up CIs to automatically upload docs, it's what we do in MNE-Python and I'm not sure why BIDS did not follow suit. I'll take a look on Monday I wouldn't worry about cutting a first release + docs until next week at the earliest.

adam2392 commented 3 years ago

Closed via #12