HISE Logo Forum
    • Categories
    • Register
    • Login

    Please Increase parameter limit on Scriptnode custom nodes!

    Scheduled Pinned Locked Moved Feature Requests
    25 Posts 7 Posters 724 Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • griffinboyG
      griffinboy @Orvillain
      last edited by

      @Orvillain

      Thank you very much, super good.

      However I would wish it to be changed in the official version of hise, as I like to share nodes and I'm trying to keep the amount of source code tampering to a minimum - some of my nodes already require changing a few things in the Hise Source.

      O 1 Reply Last reply Reply Quote 0
      • O
        Orvillain @griffinboy
        last edited by

        @griffinboy said in Please Increase parameter limit on Scriptnode custom nodes!:

        @Orvillain

        Thank you very much, super good.

        However I would wish it to be changed in the official version of hise, as I like to share nodes and I'm trying to keep the amount of source code tampering to a minimum - some of my nodes already require changing a few things in the Hise Source.

        @Christoph-Hart - it might be worth considering updating the code so that a global variable determines the number of supported parameters, much like the NUM_MAX_CHANNELS variable???

        griffinboyG 1 Reply Last reply Reply Quote 0
        • griffinboyG
          griffinboy @Orvillain
          last edited by

          @Christoph-Hart

          Can we get something like this please?

          O 1 Reply Last reply Reply Quote 1
          • O
            Orvillain @griffinboy
            last edited by

            @griffinboy I looked into this, and it seems like dynamically altering the switch based on a global parameter is not that straight forward. So I think the simplest solution is as Christoph did in the past, which was to manually extend the cases, as I also suggested.

            The problem is..... 16 was enough for me in that moment, but maybe not the next.... and for you, you need at least 40 or so... and for the next guy, he might be a madman and want 128....

            There has to be a better solution to this, but I'm not sure what it is. Maybe storing each case in a table and using it to build the switch out at compile time, but my c++ knowledge isn't good enough for that!

            griffinboyG 1 Reply Last reply Reply Quote 1
            • griffinboyG
              griffinboy @Orvillain
              last edited by

              @Orvillain

              That should be possible

              @Christoph-Hart ?

              O 1 Reply Last reply Reply Quote 0
              • O
                Orvillain @griffinboy
                last edited by

                @griffinboy Here's a code snippet if you wanna just bash it into that file. I tend to have a few customizations to HISE that I stash and re-apply each time I pull the latest develop. Hasn't broken anything yet!

                switch (pIndex)
{
case 0:  p = parameter::inner<SnexSource::ParameterHandler, 0>(pHandler); break;
case 1:  p = parameter::inner<SnexSource::ParameterHandler, 1>(pHandler); break;
case 2:  p = parameter::inner<SnexSource::ParameterHandler, 2>(pHandler); break;
case 3:  p = parameter::inner<SnexSource::ParameterHandler, 3>(pHandler); break;
case 4:  p = parameter::inner<SnexSource::ParameterHandler, 4>(pHandler); break;
case 5:  p = parameter::inner<SnexSource::ParameterHandler, 5>(pHandler); break;
case 6:  p = parameter::inner<SnexSource::ParameterHandler, 6>(pHandler); break;
case 7:  p = parameter::inner<SnexSource::ParameterHandler, 7>(pHandler); break;
case 8:  p = parameter::inner<SnexSource::ParameterHandler, 8>(pHandler); break;
case 9:  p = parameter::inner<SnexSource::ParameterHandler, 9>(pHandler); break;
case 10: p = parameter::inner<SnexSource::ParameterHandler, 10>(pHandler); break;
case 11: p = parameter::inner<SnexSource::ParameterHandler, 11>(pHandler); break;
case 12: p = parameter::inner<SnexSource::ParameterHandler, 12>(pHandler); break;
case 13: p = parameter::inner<SnexSource::ParameterHandler, 13>(pHandler); break;
case 14: p = parameter::inner<SnexSource::ParameterHandler, 14>(pHandler); break;
case 15: p = parameter::inner<SnexSource::ParameterHandler, 15>(pHandler); break;
case 16: p = parameter::inner<SnexSource::ParameterHandler, 16>(pHandler); break;
case 17: p = parameter::inner<SnexSource::ParameterHandler, 17>(pHandler); break;
case 18: p = parameter::inner<SnexSource::ParameterHandler, 18>(pHandler); break;
case 19: p = parameter::inner<SnexSource::ParameterHandler, 19>(pHandler); break;
case 20: p = parameter::inner<SnexSource::ParameterHandler, 20>(pHandler); break;
case 21: p = parameter::inner<SnexSource::ParameterHandler, 21>(pHandler); break;
case 22: p = parameter::inner<SnexSource::ParameterHandler, 22>(pHandler); break;
case 23: p = parameter::inner<SnexSource::ParameterHandler, 23>(pHandler); break;
case 24: p = parameter::inner<SnexSource::ParameterHandler, 24>(pHandler); break;
case 25: p = parameter::inner<SnexSource::ParameterHandler, 25>(pHandler); break;
case 26: p = parameter::inner<SnexSource::ParameterHandler, 26>(pHandler); break;
case 27: p = parameter::inner<SnexSource::ParameterHandler, 27>(pHandler); break;
case 28: p = parameter::inner<SnexSource::ParameterHandler, 28>(pHandler); break;
case 29: p = parameter::inner<SnexSource::ParameterHandler, 29>(pHandler); break;
case 30: p = parameter::inner<SnexSource::ParameterHandler, 30>(pHandler); break;
case 31: p = parameter::inner<SnexSource::ParameterHandler, 31>(pHandler); break;
case 32: p = parameter::inner<SnexSource::ParameterHandler, 32>(pHandler); break;
case 33: p = parameter::inner<SnexSource::ParameterHandler, 33>(pHandler); break;
case 34: p = parameter::inner<SnexSource::ParameterHandler, 34>(pHandler); break;
case 35: p = parameter::inner<SnexSource::ParameterHandler, 35>(pHandler); break;
case 36: p = parameter::inner<SnexSource::ParameterHandler, 36>(pHandler); break;
case 37: p = parameter::inner<SnexSource::ParameterHandler, 37>(pHandler); break;
case 38: p = parameter::inner<SnexSource::ParameterHandler, 38>(pHandler); break;
case 39: p = parameter::inner<SnexSource::ParameterHandler, 39>(pHandler); break;
case 40: p = parameter::inner<SnexSource::ParameterHandler, 40>(pHandler); break;
case 41: p = parameter::inner<SnexSource::ParameterHandler, 41>(pHandler); break;
case 42: p = parameter::inner<SnexSource::ParameterHandler, 42>(pHandler); break;
case 43: p = parameter::inner<SnexSource::ParameterHandler, 43>(pHandler); break;
case 44: p = parameter::inner<SnexSource::ParameterHandler, 44>(pHandler); break;
case 45: p = parameter::inner<SnexSource::ParameterHandler, 45>(pHandler); break;
case 46: p = parameter::inner<SnexSource::ParameterHandler, 46>(pHandler); break;
case 47: p = parameter::inner<SnexSource::ParameterHandler, 47>(pHandler); break;
case 48: p = parameter::inner<SnexSource::ParameterHandler, 48>(pHandler); break;
case 49: p = parameter::inner<SnexSource::ParameterHandler, 49>(pHandler); break;
case 50: p = parameter::inner<SnexSource::ParameterHandler, 50>(pHandler); break;
case 51: p = parameter::inner<SnexSource::ParameterHandler, 51>(pHandler); break;
case 52: p = parameter::inner<SnexSource::ParameterHandler, 52>(pHandler); break;
case 53: p = parameter::inner<SnexSource::ParameterHandler, 53>(pHandler); break;
case 54: p = parameter::inner<SnexSource::ParameterHandler, 54>(pHandler); break;
case 55: p = parameter::inner<SnexSource::ParameterHandler, 55>(pHandler); break;
case 56: p = parameter::inner<SnexSource::ParameterHandler, 56>(pHandler); break;
case 57: p = parameter::inner<SnexSource::ParameterHandler, 57>(pHandler); break;
case 58: p = parameter::inner<SnexSource::ParameterHandler, 58>(pHandler); break;
case 59: p = parameter::inner<SnexSource::ParameterHandler, 59>(pHandler); break;
case 60: p = parameter::inner<SnexSource::ParameterHandler, 60>(pHandler); break;
case 61: p = parameter::inner<SnexSource::ParameterHandler, 61>(pHandler); break;
case 62: p = parameter::inner<SnexSource::ParameterHandler, 62>(pHandler); break;
case 63: p = parameter::inner<SnexSource::ParameterHandler, 63>(pHandler); break;
default:
    jassertfalse;
}
                griffinboyG 1 Reply Last reply Reply Quote 1
                • griffinboyG
                  griffinboy @Orvillain
                  last edited by

                  @Orvillain

                  Thanks man. Honestly I'm still hoping it gets officially changed, because I like to share nodes on here. @Christoph-Hart said he is looking to improve certain areas of the c++ node side of hise, so hopefully it'll get sorted at somepoint!

                  I'll use your fix for now, so that I can continue development.
                  : )

                  Christoph HartC 1 Reply Last reply Reply Quote 1
                  • Christoph HartC
                    Christoph Hart @griffinboy
                    last edited by

                    There has to be a better solution to this, but I'm not sure what it is.

                    Nope, that's the way to go. The problem is the intersection between runtime code and compile time code which causes these monstrous switch statements.

                    @Orvillain if you sign the CLA, I'm happy to accept this beauty of a pull request.

                    O 1 Reply Last reply Reply Quote 1
                    • O
                      Orvillain @Christoph Hart
                      last edited by

                      @Christoph-Hart Will do!!

                      1 Reply Last reply Reply Quote 0
                      • A
                        Allen
                        last edited by

                        @Orvillain said in Please Increase parameter limit on Scriptnode custom nodes!:

                        and for the next guy, he might be a madman and want 128....

                        okay so the mad man is here 🤣
                        I just found out one of my node with about 100 parameters won't update the internal value and ready to increase this limit to 256

                        griffinboyG 1 Reply Last reply Reply Quote 1
                        • griffinboyG
                          griffinboy @Allen
                          last edited by griffinboy

                          @Allen

                          😆

                          I haven't had the chance to revist this yet

                          @Christoph-Hart Did we ever get an update for the compile time parameter callbacks? A bit higher in this thread, a solution was proposed to accommodate arbitrary parameter counts.

                          1 Reply Last reply Reply Quote 1
                          • A
                            Allen
                            last edited by

                            @Allen said in Please Increase parameter limit on Scriptnode custom nodes!:

                            I just found out one of my node with about 100 parameters won't update the internal value and ready to increase this limit to 256

                            Haven't tested all the params but just wanna comfirm so far it works in both script fx and compiled fx with max 256 params limit! :)

                            I found there's also a similar param number limit in the rnbo node so don't forget to change that one as well if you're using a rnbo node

                            1 Reply Last reply Reply Quote 1
                            • First post
                              Last post

                            57

                            Online

                            1.7k

                            Users

                            11.7k

                            Topics

                            101.7k

                            Posts