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

Python3 fixes #52

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Python3 fixes #52

wants to merge 4 commits into from

Conversation

errosica
Copy link

@errosica errosica commented Nov 27, 2019

Addressed several issues with conversion from .ipynb to .tex via latex_with_lenvs in the file latex_envs.py

  1. Fixed Python 3 compatibility? #47 which caused a compilation error in Python 3.6/3.7
  2. I also fixed a bug where the references section of the TeX document was incomplete and caused the TeX document to be unable to compile.
  3. Some points for future work: Unfortunately, I couldn't figure out how to access the bibtex bibliography name that the user enters in the GUI. Thus at the moment, the user has to edit the variable bibliography_name in latex_envs.py to specify their bibtex library name. It might also be good to set up a command line argument where the user can specify their bibtex library name.

I also made several changes to the article style.

  1. The title and author of the .ipynb is now used directly as the title of the article (Fixed Duplicate Titles with thmsInNb_article.tplx #49).
  2. Added the References section to the ToC.

… Article Template: 1. Removed Duplicate Title Information. 2. Ensure the TOC follows the Title
…fined errors. Now thing should play nice with amsmath package
…ere the refrences appeard as \hypertarget{possible-improvements}{%, which caused an error as the last { didn't close. Now a user can specify their bibliography name in latex_envs.py. In the future, it would be nice to add a command line switch to specify the bibtex file
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.

Duplicate Titles with thmsInNb_article.tplx Python 3 compatibility?
1 participant