-
Notifications
You must be signed in to change notification settings - Fork 165
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
Added GSoC blog posts for remaining weeks #496
Conversation
Codecov Report
@@ Coverage Diff @@
## master #496 +/- ##
=======================================
Coverage 89.00% 89.00%
=======================================
Files 33 33
Lines 7093 7093
Branches 834 834
=======================================
Hits 6313 6313
Misses 547 547
Partials 233 233 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @antrikshmisri,
Overall, it looks good. Thank you for this. See below for some minor comments. could you also rebase. After those changes, it will be ready to go. Thanks!
------------------------ | ||
Below are the tasks that I worked on: | ||
|
||
* `Created PR for sprite sheet animation <https://github.com/fury-gl/fury/pull/491>`_ : This Pr adds support for playing animations from a sprite sheet. This feature will be used in Card2D to create a tutorial in which the card will show the animation in the image box. Previously, the utility functions for this were added directly inside the tutorial but now they are refactored to go in their respective modules. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR
instead
Hello @antrikshmisri! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the update @antrikshmisri!
LGTM, merging.
No description provided.