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

Update LUA and an example for usage #396

Open
wants to merge 14 commits into
base: develop
Choose a base branch
from

Conversation

zwim
Copy link
Contributor

@zwim zwim commented May 29, 2024

In this PR you find:

1.) Use anti aliased lines, circles and arcs in the existing watchfaces
2.) Use dynamic clock
3.) Update LUA and an example watchface using it.
4.) Simplify API for the anti-aliasing feature

zwim added 12 commits May 25, 2024 17:37
minor cleanups

Avoid flickering and optimize

new line drawing algorithm

Use more floats (save 1k of Flash)

Fix AA-circle quirk

more float to go

use hypotf

Different ending for thick lines

Try to optimize with trigonometric table.

fine tune

Optimize division of chunkHeight

Optimized arc, better AA

lower CPU freq for power savings

Saves approx 50% during on time.

disable cpufreq

honor review

Screen fill
One step to lua

Make Twilight watchface with LUA support

add drawArcAA and migrate OswAppWatchface

Make LUA work with emulator
@zwim
Copy link
Contributor Author

zwim commented May 29, 2024

Sorry I have missed, that the lib/LUA has to be updated as well.

@zwim zwim force-pushed the ZwilightWatchface branch from e581668 to 6e1d31d Compare May 29, 2024 06:45
@zwim zwim force-pushed the ZwilightWatchface branch 7 times, most recently from 30a0587 to 95a40f7 Compare June 1, 2024 20:00
@zwim zwim mentioned this pull request Jun 1, 2024
@zwim zwim force-pushed the ZwilightWatchface branch 4 times, most recently from 6895c24 to fdecd17 Compare June 2, 2024 17:12
@zwim
Copy link
Contributor Author

zwim commented Jun 2, 2024

What the ... 161 tests OK, 1failing -> fix that the other day ;)

@zwim zwim force-pushed the ZwilightWatchface branch from fdecd17 to 3bff07c Compare June 3, 2024 18:15
Copy link
Member

@RuffaloLavoisier RuffaloLavoisier left a comment

Choose a reason for hiding this comment

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

Hi there 😄

Could you please split the PR into small pieces and request it? If there are many changes, it is difficult to check the review and potential influence. It is also possible to make and merge sub PR separately.

sample

main PR
- sub PR feature 1
- sub PR feature 2

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.

2 participants