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
Week 3 blog post. #619
Week 3 blog post. #619
Conversation
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.
Hello @m-agour ,
Nice Blog. It generates well and is visible in blog.html
too.
Mentioned some punctuation error I found.
Also if you could reduce the video size then it would be great as it kind of overlaps the On this page
section on the right-hand side as shown in the image.
Thanks!
|
||
- Composed the playback panel as a single component `PlaybackPanel` (slider does not function yet). | ||
|
||
- As we tried to come up with a way to do keyframe animations on a partial subset of the actor's primitives not all of them, we found that there is no currently implemented way to get the primitives count of a single actor. So I made this PR `#617`_ so that the primitives' count is saved inside the polydata of the actor as suggested by Filipi and Javier so that the primitives' count can be used to distinguish the vertices of different primitives. |
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.
Maybe primitives
instead of primitives'
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.
I meant here the count of the primitives
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 @m-agour, Nice Blog and good work. I noticed if you want to make some word code like (eg: this
) you need to use two backtick symbols. for eg:
``word`` # code like
`word` # italic
I have a few suggestions below, PTAL
What did you do this week? | ||
-------------------------- | ||
|
||
- Redesigned the keyframe animations API to optimize having a lot of timelines by composing them into a parent `Timeline` called `CompositeTimeline` while maintaining playing individual timelines seperately. |
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.
typo separately
Thank you @ganimtron-10 and @xtanion for your amazing reviews. |
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 @m-agour,
See below for 2 small comments and after that, it is ready to go
This week I will do the following | ||
|
||
- Finish up the ``PlaybackPanel``. | ||
- Implement all other interpolators in GLSL (color interpolators included) |
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.
missing .
at the end of the sentence
- Finish up the ``PlaybackPanel``. | ||
- Implement all other interpolators in GLSL (color interpolators included) | ||
- Start Implementing slerp interpolation using quaternions in both Python and GLSL. | ||
- Add tests for the Previous work. |
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.
Previous
does not need to be capital
Thanks @m-agour, merging |
My week 3 blog post.