Skip to content

Simplify options #2

Open
Open

Description

@netmindz

As we look to pulling the latest version of AR into the main WLED release, we probably need hide some of the experimental features of the usermod to be more appropriate for the different user base

Activity

  1. netmindz commented on Feb 6, 2026

    @netmindz
    CollaboratorAuthor

    @softhack007 - do you want to take the first pass at adding suitable ifdefs for which you think we should be excluding from the main WLED release?

  2. DedeHai commented on Mar 4, 2026

    @DedeHai
    Collaborator

    I ran some compile tests: the new mic filter-table use ~3k of flash (no idea why so much but they do)
    The FFT windows do not appear to use any flash, looks like they are created on the fly in RAM.
    New audio sources add 1.5k to flash use, could ont find out where the rest (about 5k worth) is coming from, maybe from advanced algorithms.

    I would suggest to strip the advanced/experimental features for AC but keep the new sources, I have it ready with some ifdefs. What is the better/preferred approach opt-in or opt-out?

    #ifndef MM_AR_NO_ADVANCED_FEATURES -> AC defines this

    or

    #ifdef MM_AR_USE_ADVANCED_FEATURES -> MM defines this

  3. netmindz commented on Mar 5, 2026

    @netmindz
    CollaboratorAuthor

    A good starting point is probably feature parity with the current AC version or AR, with flag(s) to enable extra features

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions