Skip to content
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

Port 'Animation' to Python #29

Merged
merged 5 commits into from
Apr 22, 2024

Conversation

UrtsiSantsi
Copy link
Contributor

Spring animation is not working currently

Copy link
Contributor

@rolandlo rolandlo left a comment

Choose a reason for hiding this comment

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

I have found two mistakes, that make the Spring animation not work.

demos/Animation/main.py Outdated Show resolved Hide resolved
demos/Animation/main.py Outdated Show resolved Hide resolved
demos/Animation/main.py Outdated Show resolved Hide resolved
demos/Animation/main.py Outdated Show resolved Hide resolved
@sonnyp
Copy link
Contributor

sonnyp commented Mar 23, 2024

@UrtsiSantsi do you want to finish this one?

@sonnyp
Copy link
Contributor

sonnyp commented Apr 13, 2024

@rolandlo would you like to take over this PR?

@rolandlo
Copy link
Contributor

@rolandlo would you like to take over this PR?

Ok, I will.

@rolandlo rolandlo marked this pull request as ready for review April 13, 2024 17:52
@rolandlo
Copy link
Contributor

@theCapypara @sonnyp This is ready for review now. The pipeline failed, but it doesn't look like the fault of this demo. I also rebased on the main branch thus solving the merge conflicts.

@theCapypara
Copy link
Contributor

Hi and thanks! I'm currently out of town, I'll check your changes at the end of the week, when I'm back!

Copy link
Contributor

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

@theCapypara theCapypara merged commit 38a687d into workbenchdev:main Apr 22, 2024
1 check passed
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.

4 participants