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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request] Partial exhaustion indicator in UI #103

Open
starslab opened this issue Apr 6, 2023 · 4 comments
Open

[Feature Request] Partial exhaustion indicator in UI #103

starslab opened this issue Apr 6, 2023 · 4 comments

Comments

@starslab
Copy link

starslab commented Apr 6, 2023

It would be nice to have some kind of indicator that notes are being dropped or not played due to partial exhaustion, especially because this is an emulator that can just have more partials, if the end-user knows it needs more. 馃槃

It looks like there's a bunch of free room in the 'Synth' box in the UI, though it might make more sense to put it somewhere in the 'Partial State' area - maybe next to the 'Partial State' text itself?

@sergm
Copy link
Member

sergm commented Apr 6, 2023

Yes, this is an interesting idea, but unfortunately, it is not straightforward to implement. We do keep track of partial abortion already, albeit partials may be aborted even when you have plenty of free ones, depending on the assign mode. I'll see how best to check for partial exhaustion state yet avoid degrading performance significantly...

@sergm
Copy link
Member

sergm commented Apr 22, 2023

Perhaps, a good idea may be to show a four-state icon nearby the MIDI MESSAGE indicator (or even replacing it) with the same set of colors used for partial state indicator. For example, it may show:

  • Grey - inactive synth state.
  • Green - all used partials fit into the available partials, none is terminated prematurely.
  • Yellow - some notes have been prematurely silenced due to a lack of free partials (shown for some time, e.g. 80 millis).
  • Red - some notes were recently denied to play by the partial allocator due to insufficient of free partials.

I am on the fence whether this should be a separate indicator, or we could extend the semantic of MIDI MESSAGE optionally. Of course, there is a plenty of free space in the Synth box, but two sibling indicators seems a bit overkill. OTOH, the separate insufficient partial indicator could be made even more visible.

@starslab
Copy link
Author

starslab commented Apr 22, 2023

I'm certainly a fan of the existing MIDI MESSAGE display, as it mimics the real hardware display. It's unfortunate that Roland put it right smack dab in the middle of the MIDI MESSAGE text, as that leads to a lot of wasted space.

If you want to make it an option, perhaps.....

  • Option off, existing behavior
  • Option on, move the green flasher over to the side, and put your partial state indicator over to the other side, like this:
 --------------
| MIDI MESSAGE |
|  --   [   ]  |
 --------------

Maybe it's worth considering putting this indicator down next to the Partial State text instead - there's lots of room for it there, and it's consistent with the purpose of that display.

| [ ] Partial State -----
| [ ] [ ] [ ] [ ] [ ] [ ] 
| [ ] [ ] [ ] [ ] [ ] [ ] 
| [ ] [ ] [ ] [ ] [ ] [ ] 

But I think that might not actually fit in the hardware-consistent 32-partial configuration.

@sergm
Copy link
Member

sergm commented May 18, 2023

I suppose, the new indicator should be shown even when the lower synth details pane is closed. I'm going to place it close to the MIDI MESSAGE LED.

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

No branches or pull requests

2 participants