Skip to content

Multiple Language Support#197

Closed
chsasank wants to merge 8 commits intosphinx-gallery:masterfrom
chsasank:polyglot2
Closed

Multiple Language Support#197
chsasank wants to merge 8 commits intosphinx-gallery:masterfrom
chsasank:polyglot2

Conversation

@chsasank
Copy link
Copy Markdown

@chsasank chsasank commented Jan 17, 2017

A follow up to #196. I've had to cherry pick my commits to fix the commit history of my branch.
I'm copying text from #196 and closing it.

Following discussion #194, I've made a working version with support for multiple languages. In the process, I've done quite a bit of refactoring.

I also

  • made some functions 'private' with prepending _ to function names.
  • made some code pep8 complaint
  • Added comments here and there

Please have a look at the implementation and give me your feedback.

Right now, I've written only lua parser, adding other languages shouldn't be hard at this point.

TODO:

  • Merge changes from Titan-C/sphinx-gallery:func_rst carefully.
  • Add other language support
  • Fix Downloads bug (absolute path is being used to save zip)
  • Write Tests

Titan-C and others added 7 commits January 12, 2017 21:00
This allows Sphinx-Gallery to be build by a direct call to
sphinx-build, specifying the sources directory and the build output
directory. There is no more the constrain to launch the build from the
makefile is the sources directory.

- Correct relative path when embedding links
- Correct path during build
- Summary of failing examples to work on relative path builds
- Update tests
- Update docs clean and build configs
- Figure test for mayavi
New modules
- execute blocks
- save images
- write rst

review image test

Load Agg in execute_blocks.py because tests load order is alphabetic

Correct rendering of traceback in rst files

Fix orphan repetition
@lesteve
Copy link
Copy Markdown
Member

lesteve commented Jan 17, 2017

Is it already working for SimpleITK doc? I would be interested to look at the output.

@chsasank
Copy link
Copy Markdown
Author

@lesteve
Copy link
Copy Markdown
Member

lesteve commented Jan 17, 2017

I have put it up here: https://chsasank.github.io/sitk-docs/auto_examples/index.html

Thanks!

Amended based on better commit from @Titan-C
@larsoner
Copy link
Copy Markdown
Contributor

@chsasank this PR has a pretty challenging rebase problem now. I'll close this PR but feel free to open a new one if you can get to it. Hopefully it's easier in the context of our recent refactoring!

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.

4 participants