Skip to content

Conversation

leotrs
Copy link
Contributor

@leotrs leotrs commented May 27, 2020

Closes #96 and #85.

Copy link
Member

@Aathish04 Aathish04 left a comment

Choose a reason for hiding this comment

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

Could you please remove media_dir.txt as well?

Its path is manim/scene/media_dir.txt

https://www.github.com/ManimCommunity/manim/tree/master/manim%2Fscene%2Fmedia_dir.txt

@leotrs
Copy link
Contributor Author

leotrs commented May 27, 2020

@Aathish04 done

@Aathish04
Copy link
Member

Aathish04 commented May 27, 2020

Awesome. Also, I think you also have to remove the import for SceneFromVideo from __init__.py as well.

It's at line 54 of the file.

https://www.github.com/ManimCommunity/manim/tree/remove_unnecessary_files/manim%2F__init__.py#54

@leotrs
Copy link
Contributor Author

leotrs commented May 27, 2020

Ah good one @Aathish04 . Just did that. Thanks

@leotrs leotrs merged commit a29eabd into master May 27, 2020
@leotrs leotrs deleted the remove_unnecessary_files branch May 27, 2020 18:34
@leotrs leotrs mentioned this pull request May 27, 2020
@PgBiel PgBiel added the pr:deprecation Deprecation, or removal of deprecated code label May 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:deprecation Deprecation, or removal of deprecated code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

remove all of manim/files/
4 participants