Skip to content

Conversation

@Aathish04
Copy link
Member

List of Changes

  • Rename ExampleFileScene to TexTemplateFromCLI
  • Rename ExampleClassScene to InCodeTexTemplate
  • Remove comments and rewrite them as docstrings for corresponding Scenes.

Motivation

The "better" way of drawing the end users' attention to how to use the custom TexTemplate related functions was discussed in #199 , and the conversation stopped with @leotrs suggesting that the instructions should probably go in the documentation instead of as comments in the file.

For clarity, ExampleFileScene has been renamed to TexTemplateFromCLI and ExampleClassScene has been renamed to InCodeTexTemplate to better reflect what they demonstrate.

Testing Status

As this is only a name change in an example scene, it shouldn't really impact anything else.
The scenes themselves still work as intended.

Acknowledgement

Rewrite comments about proper way to render scenes as Docstrings for each scene.
@Aathish04 Aathish04 requested a review from leotrs July 27, 2020 07:53
@leotrs leotrs merged commit 28f4d98 into ManimCommunity:master Jul 27, 2020
@Aathish04 Aathish04 deleted the rename-custom-texamples branch July 27, 2020 16:40
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.

Need to re-implement custom LaTeX related methods for current config system.

2 participants