-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Add setup.py #45
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
Add setup.py #45
Conversation
What exactly would this do though? How would this manim fork be run with that setup.py in place, and would that help in running the 3b1b manim and this community manim in parallel? |
@XorUnison I think now we will be able to run this one by using |
I'm still not 100% about the manimcm name, but this can be merged and the name can be changed elsewhere... |
Merged. The next step here @eulertour is to add minimal tests for running manim using either |
…#2642) * Migrated file_ops.py and scene_file_writer.py from os.path to Pathlib (github issue #45) * Updated scene_file_writer.py to handle WindowsPaths * Updated scene_file_writer.py to handle WindowsPaths * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Updated file_ops.py to handle WindowsPaths * Updated scene_file_writer.py to handle WindowsPaths * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Fixed Merge Conflict: Added Type Hints in file_ops.py * Changed test_file_ops to use Pathlib in accordance with file_ops.py usage * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci Co-authored-by: Will Soltas <willsoltas@Wills-MacBook-Pro-2.local> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Add a
setup.py
with anentry_points
entry which replacesmanim.py
, and removemanim.py
.