Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@fehiepsi
Copy link
Member

@fehiepsi fehiepsi commented May 11, 2021

Addresses a doc issue that is blocking current PRs in #2826.


def setup(app):
app.add_stylesheet('css/pyro.css')
app.add_css_file('css/pyro.css')
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add_stylesheet is renamed to add_css_file since version 1.8

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried the same; however, I got an error for funsors docs in CI. Any idea why this is not triggered here?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure. Probably you got some sort of circular import issues (e.g. from ... import * statement or type annotation...)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That might be it. I'll dig into it.

Copy link
Member

@OlaRonning OlaRonning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for a quick fix.

Copy link
Member

@fritzo fritzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's check readthedocs after this merges, to ensure latest is still built.

@fritzo fritzo merged commit d228e7c into pyro-ppl:dev May 11, 2021
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