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

Attack of the fifty foot jellyfish!!! #5779

Open
wants to merge 64 commits into
base: master
Choose a base branch
from

Conversation

Paul-Licameli
Copy link
Collaborator

@Paul-Licameli Paul-Licameli commented Dec 12, 2023

Oh no! That four-year-old laboratory experiment that was long left for dead has REVIVED!

(Especially for you at your request to see it again, @petersampsonaudacity)

  • I signed CLA
  • The title of the pull request describes an issue it addresses
  • If changes are extensive, then there is a sequence of easily reviewable commits
  • Each commit's message describes its purpose and effects
  • There are no behavior changes unnecessary for the stated purpose of the PR

Recommended:

  • Each commit compiles and runs on my machine without known undesirable changes of behavior

@Paul-Licameli Paul-Licameli self-assigned this Dec 12, 2023
@Paul-Licameli Paul-Licameli added the Experimental Exploratory, proof-of-concept PRs, never to me merged label Dec 12, 2023
@Paul-Licameli Paul-Licameli force-pushed the ATTACK-OF-THE-FIFTY-FOOT-JELLYFISH!!! branch from 8035a24 to 792c074 Compare December 15, 2023 18:14
@Paul-Licameli
Copy link
Collaborator Author

Updated -- I did some important work on the problem of per-track metering.

Try it out again, with multiple, different stereo tracks, and observe that the goniometers display the correct per-track output, not the master output.

Also vary the gain and pan sliders during play and observe how the meters respond.

The work on internals to make this happen is serious work that should be merged to master, even if the display code is not.

@Paul-Licameli Paul-Licameli force-pushed the ATTACK-OF-THE-FIFTY-FOOT-JELLYFISH!!! branch 2 times, most recently from ee55b8a to c4d916a Compare December 17, 2023 23:32
@Paul-Licameli Paul-Licameli mentioned this pull request Dec 30, 2023
6 tasks
@Paul-Licameli Paul-Licameli force-pushed the ATTACK-OF-THE-FIFTY-FOOT-JELLYFISH!!! branch from c4d916a to dbd1b3f Compare January 9, 2024 13:40
... The real work happens only in AILAProcess which is still not called without
the experimental flag turned on.
... see also f509d9c where GetMaxPeak became unused
... also consistency of signatures with the pure virtuals of MeterPanelBase
... Don't emit start events until success is known.

Only ProjectAudioManager cared, and anyway ignored start events.
... Control areas are narrower, for WaveTrack.  For other kinds of tracks,
which don't have meters, the control areas are now wider.
@Paul-Licameli Paul-Licameli force-pushed the ATTACK-OF-THE-FIFTY-FOOT-JELLYFISH!!! branch from c8cbf29 to 47244ef Compare January 12, 2024 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Experimental Exploratory, proof-of-concept PRs, never to me merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant