Skip to content

Conversation

@Rahul-JOON
Copy link
Contributor

Replaced the video generation functionality from moviepy to ffmpeg because it is much faster and makes it uniform among other examples.

Ps: The codebase was failing some callbacks checks on my local machine, but I believe they are unrelated to my modification.

@bhosale2 bhosale2 changed the base branch from master to update-0.3.1 February 24, 2023 14:30
@Rahul-JOON Rahul-JOON changed the title Replaced 'moviepy' with 'ffmpeg' for video generation # 169: Replaced 'moviepy' with 'ffmpeg' for video generation Feb 24, 2023
@bhosale2 bhosale2 added update Update the codebase prio:low Priority level: low labels Feb 24, 2023
@bhosale2 bhosale2 linked an issue Feb 24, 2023 that may be closed by this pull request
@codecov-commenter
Copy link

Codecov Report

Base: 87.42% // Head: 87.53% // Increases project coverage by +0.11% 🎉

Coverage data is based on head (b44dbb1) compared to base (addd631).
Patch has no changes to coverable lines.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files
@@               Coverage Diff                @@
##           update-0.3.1     #232      +/-   ##
================================================
+ Coverage         87.42%   87.53%   +0.11%     
================================================
  Files                43       44       +1     
  Lines              2624     2817     +193     
  Branches            312      320       +8     
================================================
+ Hits               2294     2466     +172     
- Misses              309      330      +21     
  Partials             21       21              
Impacted Files Coverage Δ
elastica/interaction.py 89.11% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Contributor

@armantekinalp armantekinalp 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

Copy link
Collaborator

@bhosale2 bhosale2 left a comment

Choose a reason for hiding this comment

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

LGTM

@armantekinalp armantekinalp merged commit a9a69b8 into GazzolaLab:update-0.3.1 Feb 24, 2023
@Rahul-JOON Rahul-JOON deleted the 169_update_visualization_example branch April 10, 2023 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

prio:low Priority level: low update Update the codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update visualisation example to use ffmpeg instead of moviepy

4 participants