Change Slider Value via MIDI + Velocity

trillbilly

@d-healey I messed up on naming the slider in the snippet. I have corrected it. Still does not work with the corrected slider names.

d.healey

@trillbilly What error do you see? (Slider name seems fine to me btw)

trillbilly

@d-healey Ahh, I see what you are mentioning now. I was equating it to the wrongly named slider.

Now I see the issue is this line

cAnimation(keySwitchesC.indexOf(Message.getNoteNumber()()));

Where am I going wrong?

I've tried replacing it with

slider.setValue(Message.getVelocity());

But again, it reacts to all keys and not only C.

d.healey

@trillbilly

cAnimation(keySwitchesC.indexOf(Message.getNoteNumber()()));

Look closely

aaronventure

@d-healey Boobs!

trillbilly

@aaronventure @d-healey
I need a dunce cap and a beer because I am blind to this. Maybe I should continue banging my head on the keyboard until it works, I've done that a time or 2.

d.healey

@trillbilly

trillbilly

@aaronventure @d-healey Yes, the boobs remark makes sense now. Now it is working on just the C but seems to be limited.

I have the slider set to min 0 - max 127 with steps 1.0.

With the new code, it doesn't surpass 3.

Should I be setting the slider up differently in this sense or am I missing something else.

d.healey

@trillbilly You are passing the index of the array, based on the note you play. So if you play note 24 the knob will be set to 0, if you play 36 the knob will be set to 1, etc.

trillbilly

@d-healey Thanks. Now Im super confused how to now make it follow the velocity modulator. I will continue tinkering with it.

If you have time for a snippet or anything it would be super appreciated.

d.healey

@trillbilly If you want your knob to have the same value as the incoming velocity then you have that in my first response. If you only want to respond to certain notes then use an if statement to check if the incoming note is in your array before you set the knob's value.

trillbilly

@d-healey yes, I've put

slider.setValue(Message.getVelocity());

within the IF statement in the onNoteOn like so:

	    if (keySwitchesC.indexOf(Message.getNoteNumber()))
	    {
	        cAnimation(keySwitchesC.indexOf(Message.getNoteNumber()));
	        
	        slider.setValue(Message.getVelocity());
	    }

But is apparently incorrect as it then changes to all keys again instead of just C.

I will keep plugging away.

d.healey

@trillbilly said in Change Slider Value via MIDI + Velocity:

if (keySwitchesC.indexOf(Message.getNoteNumber()))

.indexOf will return the index in the array. It doesn't check for true or false. Use .contains() instead.

aaronventure

@trillbilly I have a custom GPT that i fed with HISE source and hise docs and told it a bit about hise.

nothing fancy. it still makes a lot of mistakes as it forgets this isn't exactly javascript., but I guess I should upload more code examples.

I entered your post in there https://forum.hise.audio//post/75262 hoping that maybe the magical talking computer will do a better job of explaining it to you than we did

and this is what i got

The issue you're encountering seems to stem from a couple of misunderstandings about how to check if a note is in your keySwitchesC array and how to properly use the cAnimation function. Here's a breakdown of what needs to be fixed:

Checking for Key Presses: You are using Message.getVelocity() to check if a key is in your keySwitchesC array, but Message.getVelocity() returns the velocity of the note being played, not its pitch. You should use Message.getNoteNumber() to check if the played note's pitch matches any of the Cs in your array.

Using indexOf Properly: The indexOf method returns -1 if the element is not found in the array. Any other returned value (i.e., 0 or above) means the element was found. Therefore, you need to check if the result of indexOf is not equal to -1 to confirm the presence of the note number in your array.

The cAnimation Function: Your cAnimation function seems to be designed to set the slider's value, but it's missing the value parameter. You need to decide what value you want to set the slider to when a C note is pressed. Since the function takes an index, you might want to use this index or the note number itself to determine the slider's value.

Here's a corrected version of your script:
const var slider = Content.getComponent("slider");

const var keySwitchesC = [24, 36, 48, 60, 72, 84, 96, 108, 120]; // Cs

inline function cAnimation(value)
{   
  slider.setValue(value);
}

function onNoteOn()
{
  var noteNumber = Message.getNoteNumber(); // Get the note number
 var index = keySwitchesC.indexOf(noteNumber); // Find the index of the note in the array
  
  if (index != -1) // Check if the note is a C
 {
       // Decide what value you want to set. For example, setting the slider to the note's velocity:
       var velocity = Message.getVelocity();
       cAnimation(velocity); // Call the animation function with the determined value
   }
}
In this revised version, the script correctly identifies when a C note is played by checking its note number against the keySwitchesC array. When a match is found, it uses the note's velocity (or any other logic you define) as the value for the cAnimation function to set the slider's value. This approach ensures that the animation only happens for the specified C notes.

Obviously declaring variables inside the realtime callbacks is a nono so once I reminded it about it, I got

const var slider = Content.getComponent("slider");
const var keySwitchesC = [24, 36, 48, 60, 72, 84, 96, 108, 120]; // C notes

// Use 'reg' for variables involved in real-time callbacks
reg noteNumber = -1;
reg index = -1;
reg velocity = 0;

inline function cAnimation(value)
{   
    slider.setValue(value);
}

function onNoteOn()
{
    noteNumber = Message.getNoteNumber(); // Use the 'reg' variable
    index = keySwitchesC.indexOf(noteNumber); // Update the 'reg' variable based on the note number
    
    if (index != -1) // Check if the note is a C
    {
        velocity = Message.getVelocity(); // Use the 'reg' variable for velocity
        cAnimation(velocity); // Call the animation function with the velocity
    }
}

and when paste this, it sets slider to the value of velocity whenever the note is C.

Also, a much simpler way to check for C is if ((Message.getNoteNumber() % 12) == 0).

And if you ask the GPT about it, it'll tell you the same thing:

Yes, indeed! Since you're dealing with MIDI note numbers where middle C is 60, and each octave >spans 12 notes, you can simplify the process of checking for C notes by leveraging this pattern. >Here's how you can do it:
const var slider = Content.getComponent("slider");

reg noteNumber = 0;
reg velocity = 0;

function onNoteOn()
{
  noteNumber = Message.getNoteNumber();
   
   // Check if the note is a C (where middle C is 60)
   if (noteNumber % 12 == 0)
   {
       velocity = Message.getVelocity();
       slider.setValue(velocity);
   }
}
In this version of the script:

We calculate the difference between the note number and 60, representing middle C.

We then use the modulo operator % to check if this difference is a multiple of 12, indicating >that the note is a C.

If the condition is met, we proceed with setting the slider value based on the note's velocity.

This approach simplifies the code by removing the need for a separate array to store C notes and directly computes whether the note is a C using arithmetic operations.

And do a much better job of explaining these very simple snippets, mentioning all the important things so even at the base level of skill you should be able to understand.

d.healey

@aaronventure said in Change Slider Value via MIDI + Velocity:

will do a better job of explaining it to you than we did

I was intentionally vague to try and make him work it out for himself :)

trillbilly

@d-healey it seems the .contains may have done the trick. And yes, I appreciate you allowing me to work it through,

@aaronventure Thats cool what you did with GPT. I've only ever tried to use it once or twice with HISE and got lost faster than without it.

aaronventure

@d-healey Well, me too, but sometimes it can get a bit much and lost in too many replies.

I wanted to point out that ChatGPT will not only write the code, but will explain all the reasoning behind all the decisions it made and thoroughly comment the code itself which is far better than just handing down a clear solution without context and is often beyond the scope of forum posts for such simple questions, at least based on what I've seen in years of browsing VI-C and in the last half a year or so, here.

The GPT generation contains everything one needs to know about why the initial snippet didn't work, and what to do in order to make it work.

d.healey

@aaronventure said in Change Slider Value via MIDI + Velocity:

wanted to point out that ChatGPT will not only write the code, but will explain all the reasoning behind all the decisions it made

I think this is great if you know that what it's giving you is correct. The problem is as a beginner you don't know if the info is accurate or not. It could provide a wonderful explanation that is totally wrong and code doesn't work.

aaronventure

@d-healey Yeah absolutely, it's also still not following HISEScript coding practices (maybe I should hardcode that in the settings).

I guess what I'm trying to say is, we don't deserve the robots; we could've pasted the post into it, and just copied back the response without the code

The issue you're encountering seems to stem from a couple of misunderstandings about how to check if a note is in your keySwitchesC array and how to properly use the cAnimation function. Here's a breakdown of what needs to be fixed:

The issue you're encountering seems to stem from a couple of misunderstandings about how to check if a note is in your keySwitchesC array and how to properly use the cAnimation function. Here's a breakdown of what needs to be fixed:

Checking for Key Presses: You are using Message.getVelocity() to check if a key is in your keySwitchesC array, but Message.getVelocity() returns the velocity of the note being played, not its pitch. You should use Message.getNoteNumber() to check if the played note's pitch matches any of the Cs in your array.

Using indexOf Properly: The indexOf method returns -1 if the element is not found in the array. Any other returned value (i.e., 0 or above) means the element was found. Therefore, you need to check if the result of indexOf is not equal to -1 to confirm the presence of the note number in your array.

The cAnimation Function: Your cAnimation function seems to be designed to set the slider's value, but it's missing the value parameter. You need to decide what value you want to set the slider to when a C note is pressed. Since the function takes an index, you might want to use this index or the note number itself to determine the slider's value.

which we basically did write, a lot more sparsely and over a bunch of posts where it appears scattered and might be tough to navigate, just because we're lazy to go in depth in the first place, ultimately wasting more time than necessary.

The response still requires the OP to take a look at their own code, check the docs etc. to figure out the proper syntax for things if they don't know.

Just musing; my fever might be coming back.

trillbilly

@d-healey One last issue. When releasing a key, it forces all animations back to 0. I'd like if a key is released, only that animation is returned to 0 while the others stay out.

I tried using an "if/else" statement in the onNoteOff but it throw an error on the "else".

Could you point me in the right direction?

Change Slider Value via MIDI + Velocity

28

1.9k

10.7k

93.0k