Skip to content

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

Merged
merged 1 commit into from
May 21, 2020
Merged

Add setup.py #45

merged 1 commit into from
May 21, 2020

Conversation

eulertour
Copy link
Member

Add a setup.py with an entry_points entry which replaces manim.py, and remove manim.py.

@eulertour eulertour linked an issue May 21, 2020 that may be closed by this pull request
@XorUnison
Copy link
Collaborator

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?

@leotrs
Copy link
Contributor

leotrs commented May 21, 2020

@XorUnison I think now we will be able to run this one by using python -m manim and python -m manimcm. The second one will be used when there is another version installed.

@leotrs
Copy link
Contributor

leotrs commented May 21, 2020

I'm still not 100% about the manimcm name, but this can be merged and the name can be changed elsewhere...

@PgBiel PgBiel added the enhancement Additions and improvements in general label May 21, 2020
@leotrs leotrs merged commit d85f148 into master May 21, 2020
@leotrs
Copy link
Contributor

leotrs commented May 21, 2020

Merged. The next step here @eulertour is to add minimal tests for running manim using either manim or manimcm. A simple run-this-scene-without-crashing should suffice for now. What do you think?

@leotrs leotrs deleted the setuptools branch May 21, 2020 14:45
WillSoltas pushed a commit to WillSoltas/manim that referenced this pull request Mar 28, 2022
behackl pushed a commit that referenced this pull request Apr 5, 2022
…#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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Additions and improvements in general
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Entry points, naming
4 participants