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

Improved rendering algorithms #106

Merged
merged 21 commits into from
Nov 11, 2024
Merged

Conversation

mbasaglia
Copy link
Member

New system to define algorithms for shapes.

The code is defined in python in the markdown file.

The extension renders the python code as pseudocode, with a clearer layout than the previous version

Old:

New:

You can also switch to code examples in a couple languages (currently Python and Typescript, we could add more if we need to) so it's easier for developers to understand what's going on.

Since the algorithm is defined as actual python code, I can add scripts to show the output shape so we can double check the algorithm is correct.

@mbasaglia
Copy link
Member Author

Looks like the pseudo-code for polystar was wrong, this also fixes that

@mbasaglia
Copy link
Member Author

since this PR fixes some issues with existing descriptions, is it worthwhile to merge this into a 1.0.1 release?

@mbasaglia
Copy link
Member Author

@cla-bot check

@cla-bot cla-bot bot added the cla-signed label Oct 22, 2024
Copy link

cla-bot bot commented Oct 22, 2024

The cla-bot has been summoned, and re-checked this pull request!

@mbasaglia mbasaglia marked this pull request as ready for review October 29, 2024 10:43
@mbasaglia mbasaglia requested a review from Aidosmf October 30, 2024 08:10
@mbasaglia mbasaglia merged commit 57ac706 into lottie:main Nov 11, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants