Skip to content

Conversation

burtenshaw
Copy link

This PR fixes the latex in unit 1.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Member

@pcuenca pcuenca left a comment

Choose a reason for hiding this comment

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

This will fix the docs but break rendering in Colab / jupyter, right?

@burtenshaw
Copy link
Author

Sorry to ping, I'm still just playing with the CI preview so it's not ready for proper review.

This will fix the docs but break rendering in Colab / jupyter, right?

@pcuenca Yeah, I think so. The two conflict (see here).

@burtenshaw
Copy link
Author

ok. this example parses correctly but spacing is still kinda wrong:

Screenshot 2025-09-12 at 09 46 17

@pcuenca what do you think I should about the colab/nb vs docs issue?

@pcuenca
Copy link
Member

pcuenca commented Sep 12, 2025

I'm not sure. Fix it where you think it helps readers most? And maybe add a note to recommend that location, close to where the latex equations start showing up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants