Skip to content

Conversation

@drammock
Copy link
Member

@drammock drammock commented Apr 5, 2021

The prior fix in #8977 seems to have been short-lived. Locally for me with latest docutils (0.17) this now works again (pretty-printing dataframes in our tutorials once again creates a horizontal scrollbar). Let's see if CircleCI behaves the same way.

Also removes from website template cruft that is no longer needed due to theme updates.

@agramfort
Copy link
Member

agramfort commented Apr 6, 2021 via email

@drammock
Copy link
Member Author

drammock commented Apr 6, 2021

@larsoner it seems like CircleCI is no longer auto-executing the tutorials that are edited in a PR? The artifact for this PR (https://26834-1301584-gh.circle-artifacts.com/0/dev/auto_tutorials/epochs/plot_40_autogenerate_metadata.html) doesn't have any cell output.

@larsoner
Copy link
Member

larsoner commented Apr 6, 2021

Should be fixed now, I introduced the regression when refactoring CircleCI yesterday

@larsoner
Copy link
Member

larsoner commented Apr 6, 2021

@drammock
Copy link
Member Author

drammock commented Apr 6, 2021

Thanks @larsoner! Looks like the horizontal scrolling is working again. Ready for review/merge on my end.

@larsoner larsoner merged commit 96b1d89 into mne-tools:main Apr 6, 2021
@larsoner
Copy link
Member

larsoner commented Apr 6, 2021

Thanks @drammock

@drammock drammock deleted the fix-html-tables-again branch April 6, 2021 19:05
vagechirkov pushed a commit to vagechirkov/mne-python that referenced this pull request Apr 6, 2021
* remove cruft

* fix dataframe x-scrolling for latest docutils [skip azp][skip github]

* touch tutorial [skip github][skip azp]

* fix acronyms in BibTeX

* remove month fields

* remove keyword fields

* whitespace

* cruft

* remove language fields

* touch tutorial again (why?)

* FIX: Cats

Co-authored-by: Eric Larson <larson.eric.d@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants