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

Bugfix/adc analog audio reactive issue #4345 #4355

Closed

Conversation

pjkerly
Copy link

@pjkerly pjkerly commented Dec 7, 2024

Bug Fixes for Issue #4345 - Analog Microphone audio source shows ADC analog - quiet after enable (audioreactive usermod)

softhack007 and others added 8 commits September 4, 2024 23:42
…3690 and #3685)

some users have reported that releases after 0.14.0 are not working reliably. So we add a few "compat" for 8266 that try to reproduce the buildenv of 0.14.0 as much as possible.

* platform and platform_packages from 0.14.0
* not using PIO_FRAMEWORK_ARDUINO_MMU_CACHE16_IRAM48
* due to smaller IRAM, we had to move some functions back from IRAM to normal flash (may cause slowdown)
Bug Fixes for Issue #4345 - Analog Microphone audio source shows ADC analog - quiet after enable (audioreactive usermod)
@netmindz netmindz requested a review from softhack007 December 10, 2024 08:54
@netmindz
Copy link
Collaborator

Not 100% sure what you have done wrong with your PR as it contains very old commits as well as your recent commit. I'm not sure if your branch is just out of date or taken as a branch from the wrong source.

It might be best if you create a fresh branch and cherry-pick your commit into that.

Would you prefer this to be done against MoonModules in the first instance @softhack007 given that's upstream for this usermod?

@pjkerly
Copy link
Author

pjkerly commented Dec 10, 2024

The instructions for PRs said to use branch 0_15 ... I see that there are also branches main and 0_15_0 . Is 0_15 the correct branch to use? I can pull a fresh copy. I can also get a copy of MoonModules. Is there a particular branch to use?

@pjkerly
Copy link
Author

pjkerly commented Dec 10, 2024

Closing this pull request ... will update later.

@pjkerly pjkerly closed this Dec 10, 2024
@netmindz
Copy link
Collaborator

From MoonModules, please branch mdev

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