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

Extend range of supported notes in piano roll #4073

Closed
teeberg opened this issue Dec 27, 2017 · 6 comments
Closed

Extend range of supported notes in piano roll #4073

teeberg opened this issue Dec 27, 2017 · 6 comments

Comments

@teeberg
Copy link
Contributor

teeberg commented Dec 27, 2017

Reported in Discord: https://discordapp.com/channels/203559236729438208/332258319228207114?jump=395583597127532544

I'm looking to create a lightshow on a launchpad and some of the lights can't be lit due to the piano roll not going high enough to have a note corrosponding to that light.

@zonkmachine
Copy link
Member

I think the range is moderated here.

void Note::setKey( const int key )
{
	const int k = qBound( 0, key, NumKeys - 1 );
	m_key = k;
}

@DeRobyJ
Copy link
Contributor

DeRobyJ commented Jan 3, 2018

The pianoroll has every possible note it can get from midi input...
You know, a midi note is described by 7 bits, so it can only go from 0 to 127...

What note number should the light get? How is it done by other software? Perhaps custom control change messages trigger by a "note"?

@zonkmachine
Copy link
Member

You know, a midi note is described by 7 bits, so it can only go from 0 to 127...

Yes, but not in Note::Note() or Note::setKey() where qBound is used to limit the register.

Example:
const int NumKeys = NumOctaves * KeysPerOctave;

NumOctaves = 9
KeysPerOctave = 12

Note::setKey() max value is NumKeys - 1 which is ( ( 9 * 12 ) - 1 ) = 107.
The top 20 MIDI keys are set to 107. Note::Note is one higher so there you get 108.

@husamalhomsi husamalhomsi removed the ux label Jul 31, 2019
@musikBear
Copy link

Looking for 'doables', but isent this one misunderstood?
In respect to audible notes, lmms is doing fine.
This very specific request is much better addressed with a new type of track -we could name it MIDICtrTCO a track where events are inserted as points, and where each point has a midi-output-value.
Infact -an automation-track, but only with points.
Like:
image

But how many users has this need? Is this usecase general enough to spend hours of implementation for?

@Spekular
Copy link
Member

Spekular commented Aug 3, 2021

@he29-net, do you know if this is still relevant after #5868? I would assume it's fixed now, but don't want to close without knowing for sure.

@he29-net
Copy link
Contributor

he29-net commented Aug 3, 2021

Yes, I believe this issue should be resolved in the recent alpha release, all 128 MIDI keys are now available.

@Spekular Spekular closed this as completed Aug 3, 2021
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

7 participants