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

Replace old FlowFields with new one, but make it go down instead of up if you can! #2

Closed
robertlipe opened this issue Aug 8, 2023 · 2 comments

Comments

@robertlipe
Copy link
Owner

Bug report

Problem

Steps

Example

Notes

@robertlipe
Copy link
Owner Author

I think you handled the replacement (thank you, @davepl ) but I changed the direction this effect flows in
SMFlowFields.h
[balls e2bf77c] Make FlowFields run the opposite Y coordinate
1 file changed, 2 insertions(+)
➜ matrix git:(balls) ✗ git push
Enumerating objects: 11, done.
Counting objects: 100% (11/11), done.
Delta compression using up to 8 threads
Compressing objects: 100% (6/6), done.
Writing objects: 100% (6/6), 552 bytes | 552.00 KiB/s, done.
Total 6 (delta 4), reused 0 (delta 0), pack-reused 0
remote: Resolving deltas: 100% (4/4), completed with 4 local objects.
To github.com:robertlipe/NightDriverStrip.git
7037669..e2bf77c balls -> balls

You can cherry-pick PlummersSoftwareLLC@bef35c3 and PlummersSoftwareLLC@e2bf77c or you can just pick up the tip of the 'balls' branch for this fix.

We get a nice 30fps for this:
092, PSRAM Free: 404167/4173195, LED FPS: 30 Refresh: 60 Hz, Power: 2930 mW, Brite: 100%, Audio FPS: 41, MinVU: 194.9, PeakVU: 203.0, VURatio: 0.1 Buffer: 0/500, CPU: 060%, 042%, FreeDraw: 0.018
(I) (loop)(C1) WiFi: WL_CONNECTED, IP: 192.168.2.165, Mem: 79456, LargestBlk: 47092, PSRAM Free: 404167/4173195, LED FPS: 30 Refresh: 60 Hz, Power: 2960 mW, Brite: 100%, Audio FPS: 45, MinVU: 203.1, PeakVU: 218.4, VURatio: 0.2 Buffer: 0/500, CPU: 061%, 043%, FreeDraw: 0.019

If you agree this is working optimally (and have verifieid that it's above or below the threshold), please close this report as accepted or closed or whatever. If there's more work here, LMK.

@robertlipe
Copy link
Owner Author

I inverted it in e2bf77c and the original is gone. Done.

robertlipe added a commit that referenced this issue Aug 17, 2023
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

No branches or pull requests

1 participant