Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed custom latex definitions errors #112

Open
wants to merge 62 commits into
base: main
Choose a base branch
from
Open

Conversation

ricvigi
Copy link

@ricvigi ricvigi commented Jun 10, 2024

Fixed some custom latex definition errors across all notebooks. Also added fonts where needed

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@iacopomasi
Copy link
Owner

@ricvigi thanks for the update; it seems a very big one. I have seen that you uploaded also some personal material. Was this your intention?

@ricvigi
Copy link
Author

ricvigi commented Jun 12, 2024

I am still trying to figure out how pull requests work... At the beginning of the course i forked the repo and i kept pulling from the origin and pushing to my forked repository. When i fixed some of the issues for me and issued the pull request, it was not very clear to me whether it was possible for you to only integrate the notebooks, instead of the whole fork (which also contains some personal notebooks about some topics we covered in class). Please let me know if this is possible, otherwise i'll try to figure out how to do it properly!

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.

None yet

2 participants