Skip to content

Conversation

@abul4fia
Copy link
Contributor

Overview: What does this pull request change?

Minor fix, added a return self to AbstractImageMobject.set_resampling_algorithm()

Motivation and Explanation: Why and how do your changes improve the library?

Since the function returned None, it could not be used in chains of calls. This is not consistent with most of the MObject setting methods.

Links to added or changed documentation pages

Further Information and Comments

Change triggered by a question in the Discord community: https://discord.com/channels/581738731934056449/1105958826852753418/1105965772691484713

Reviewer Checklist

  • The PR title is descriptive enough for the changelog, and the PR is labeled correctly
  • If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
  • If applicable: newly added functions and classes are tested

abul4fia and others added 6 commits May 1, 2023 14:59
Avoiding the creation of an Arc object at import time
This resolves [last
comment](ManimCommunity#485 (comment))
of ManimCommunity#485.

Removes one line of dead code.

Co-authored-by: Benjamin Hackl <[email protected]>
…tion class (ManimCommunity#3229)

* Make errors during docbuild somewhat easier to debug

* fix an issue with docbuild under py3.10+
Copy link
Member

@behackl behackl left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@behackl behackl merged commit 7dd9bb6 into ManimCommunity:main Jun 20, 2023
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.

3 participants