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

Permit Gameduino 3X Dazzler support on Metro M4 AirLift Lite. #4054

Merged
merged 1 commit into from
Jan 25, 2021
Merged

Permit Gameduino 3X Dazzler support on Metro M4 AirLift Lite. #4054

merged 1 commit into from
Jan 25, 2021

Conversation

dglaude
Copy link

@dglaude dglaude commented Jan 24, 2021

Totally untested, change request based on change made in #2581.
It might be my first PR in CircuitPython core...
Maybe this should be confirmed by @jamesbowman

Those are the two boards that seems supported:

  • Metro M4 Express
  • Metro nRF52840 Express

The only Metro that this PR concern:

  • Metro M4 AirLift Lite

Other Metro I found and are maybe not supported:

  • Metro ESP32-S2
  • Metro M0 Express
  • Metro M7 1011

Totally untested, change request based on change made in #2581.
It might be my first PR in CircuitPython core...
Maybe this should be confirmed by @jamesbowman

Those are the two boards that seems supported:
* Metro M4 Express
* Metro nRF52840 Express

The only Metro that this PR concern:
* Metro M4 AirLift Lite

Other Metro I found and are maybe not supported:
* Metro ESP32-S2
* Metro M0 Express
* Metro M7 1011
@dglaude
Copy link
Author

dglaude commented Jan 24, 2021

I do not fully master the Gameduino yet, but now I can run my test code without _eve.py and the benchmark goes much faster when using the artefact from this PR:

code.py output:
0.0922852
1000 points took 92.3 ms

VS what I have with the current firmware without this PR included.

code.py output:
0.454002
1000 points took 454.0 ms

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Thanks @dglaude. Feel free to enable it on others in a follow up PR.

@tannewt tannewt merged commit c075e89 into adafruit:main Jan 25, 2021
@dglaude dglaude deleted the patch-1 branch January 26, 2021 07:25
@dglaude
Copy link
Author

dglaude commented Jan 27, 2021

Thanks @dglaude. Feel free to enable it on others in a follow up PR.

I don't have any other Metro board... I can try to write a PR, but no way to test the result.
Please advise if you want me to try.

@tannewt
Copy link
Member

tannewt commented Jan 27, 2021

@dglaude No need. This is a good example for anyone who wants it on another metro. Thanks!

@jamesbowman
Copy link

@tannewt I enabled it for my raspberry_pi_pico. Works great. Is that possible?

@tannewt
Copy link
Member

tannewt commented Jan 29, 2021

@jamesbowman Totally, just make a PR for it.

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.

None yet

3 participants