LAF SliderPack Bug

[d.healey]

@DanH @ustk Did some work on this but it hasn't been merged https://github.com/christophhart/HISE/pull/327

Unmatched bipolar sliderpacks broken

Bipolar slider packs with unmatched min/max values don't work. For example min: -1, max: 10 If the min/max are equal, like -1/+1 then it works.

Forum (forum.hise.audio)

This particular bug though was only affecting bipolar sliderpacks that had different value ranges on each side of the centre, so it might be different to your issue.

[DanH]

@d-healey Thanks Dave, @Christoph-Hart is this fixable soon do you think?! :)

[d.healey]

@DanH Did you try ustk's pull request

[DanH]

@d-healey No, I'll try tomorrow

[DanH]

@d-healey the changes in the pull request haven't changed anything here. @Christoph-Hart am I doing something wrong?

[d.healey]

Just a little bit of maths

HiseSnippet 1271.3ocsVstSabDEdVfMJ1sIpopO.i3OcIXL1bKfhn0.1lPKWbhcHQBghF1cV6Ir6LtyNqgMUnlmHdE5qReD5aP5Yl0lcMwD0hZ7OVqy046blykokT3RihDRjUgNI8oHqu0tcBW0amdDFGsWcj0isa2B2bq5MvcZztCZ6j9jnHpGxxZ5c05XUXFj42e+yaSBHbWZFKD5XAyktOKjox31p1uxBBZR7ncXg4zdkZ64J36HBDw.dl1tBpOw8bRW5gDsZSYirdPCOlRHaqHJZDxZlsEdIs6Itfmp+wrH1YATMQUTavQoraJB7zHVyEsSOVfWqQwcDB7RqrrvzoYgev9.lG6F9YYiuyH.mYQ97g0TiCuoGCdUyCuJ4f2DfjUNHMSJjdhcaWIquJShFOei8dbEU5SfzddnjpKZp2Ni8NBPCtpbH4bZSIPbiENqUoRIL7YtmWrHj5iT3ADINf3i2D2f2kwokckTHX1MPbFIH0o6KDmuE2qIkF3LtgsCXdTYK3RqJ3fQmaWpZGQXeAGHblMmNydKqas+VMyYW5IuuvkDb6iznZYIsKKBBklwbWESvcl0SRtXe.0DY5wLaIr+HgcKgEm894J96EKr3h38piYQXUOJlGGdFUhE9FpHicEKnADA.CXRYB.jmmxZvPVCHAwzCExPR.6CTugRCLnWW9FUtOie9P1xbrOCrCXWnXgtkizIFMamQRunGSQg.DDpCkWHjrO.YCHA.AkC4jkOEuHdoRXxIUNU+coS0JaBn.SXiY5619h.HygoD2d2DPfJwbegTEyAYAI3DQLtGY.EqDv+ROWgGMWJ.SBEwbkwPGlOlo9wHrWBmDxbKgiinl7PeH2vU59Ssa7oJ3D09Hk+HWoaimKMW3lKWDxtzInDVVRqdDEJKcz9j4MGDkUWSeQOVVx0DrEK.nwY.9m1DWo7pfagKzBjSpdJ3YyeyiGknvOE6TsbE7B3AZSKX3uYl3ELF.RtRegPChn2xao5cWllcHKAGxSSOjqLf1GFv8JH+AicdE0UQ3cCf6uR3U.UtxTAm0FnivOqJ2TgCJNp7EK3GJTzi3NlJ3hWUDeaQ99STltcRJBBnxIJVOBV9kLzIsAoD1TyeihvbowG1Ye2C6xOK1Ms6NmhB9dbl5n9T9cMgFMbjfdv3PTAGuxLV76GNVLKghXvDvGYmKCiLPGfn0iAm4QTDj0I1qsR4W0821cd+NKu5ZIrseS3QhAUXqOvqyxUIKu5KEwungZiM9kFJumsFHOQSe37WVgs85Z4WDuyaNp+Fqt5K6tDIY+j2eDvbqMJiPudu55CYHbAjCQSepTwzIJq5zAvRwzo5ErqSiNWI5azc3TRj0C+ODbWlsAsdsjaH9Ho1ELOUuaX7W+QsdTV2dYah+3w0BGaC9epnWpFs.tvCt95q+T5F6OaaDrGT3ECiYFe4n9EACE.EQisQRu0gGwTI4ewv+aaL+2BwmX2hAColLFmZBXDtD+Zfwguy3Q1M78gADY.bF6lu8qyiJPv.IEi28.hRxfhF6CiCaC2ztT3z4bX7mtCYJc6VJcEMsNCzlx8LDeB9MTXUMs0PgUGIDERbkh24l1jpeIyCMb.LwMOhqf8AZZbVOockxUPgvipdmqqN7W.l.LYaV5dXyx2CaV4dXyp2CaV6dXyytG1r9WzF86Y2JVIBSaG.FsZXldYY0fSfJKSUH5e.vzekoH

[DanH]

@d-healey hmmmm, ok but the value 0 is half way a the top:

HiseSnippet 1231.3ocsVstaaaCElJIpX1aYXcXO.D4OSo0QwN21JJxlShs6BVt3UmlUfffBFIJatHQJPQ4D2gf0mn7JrWk8Hr2ftCojsjScJ1BV8OrMO2324vy4irqT3QSRDRjUkSFESQVegcuQb0f8FPXbz9sPVeocut3N6zpM9j18NAs6nXRRB0GYYM+Kz1XUYAj4ye+i6RBIbOZgHD5TAyid.KhoJj1s4OyBC6P7omvhJY8FM22Sv2SDJRA7LuccTLw6RRe5QDsYyYirdTaelRH6oHJZBxZgcE9i5MPbEOy9SYIrKBo5EMP8f.kItiHzWiXsTzdCXg9cGm2IHHJcKpBymUE9F6CY9rIxKpFekQAtvix0Cq4lFdyOE7ZTFd0KAuY.IqRPZgLH8X6ddRVrpPiFOet89bEUFPfxdYnjYKZtWuf8dBvBtxMhbIsiDVLwCmspWuFF9Z4mWsJT5ST3gDINjDf2F2l2mwotdRJjLuHTbAILKnGHDWtC2uCkF5Lsi8BY9TYW3PqADfw6aepZOQTrfCKbVpjMKcGu6dvNcJ4W1NefviDd2szXpqj1mk.oRmTtmhI3NK4KIWc.fZhLaaVpFNXrx90vhK9skq96Uqr5p38agYIX0.JlmFcAUhEAlUIF+pVQCHB.FvEWB.jmmIZXtngjvT5QBYDIj8Vpet1PC50suItwL9k4hkkDeA3GHtR0J8cSzEFsXmwZuZ.SQgDDTpSkeRHYuEpFPA.RJGxYqeNdU7Z0vjypet960NWarIgBMoMloOaiEgPkCSIdCljPfIo7.gTkxAcgiviDo3AjgTrR.+J88D9zRk.LIRjxUFGcXAXl5aSv9i3jHlWMbZB0TGhgZCWomO0gIfpfcTGiL4iCkdLd4rZgWoZQD6ZmvZXYMs4ITnszQGSl+xPV1XK8A8TUIOSxVsBfFmg3eXabc2MgvBGnUHm03bHxledJdbgB+DrSC253UvC0tVwHe6B0qXb.zbi9.gFlPuSzxr69bsXSVC1jmjsI2X.c.Pv8Rn9AzNuj5oH79gv4WM7FfI2X5fKFCzY3Gzka5vACG29hE7iDJ5wbGSGb0aphuqpffYpSONIEggT4LUqofkeLGcxFPpgM87SLD3kllry99I6JyE6kMcWxPAeeNScbLkeeLznbJAMwXNpfsWYnE+5bZwhBJhALfKZWpBiLPGfn0hPv7IJBx5L6s1vcFeBd5j+tdpY8t2mdW2890iieVI8qu4uryybQnWseK8ljCW.4P1DSkJltPY0hNDtTLiUuhcKZxkJQrw1bVRP7+gj65haPa0bzjEuiz7JluZvDA+0ezb.k0ePwMwu6zlQScC9epnWqFeAbkGc6s29dsbfjMtGvyke97g2MA2JJ7SARmoupT+9fbEPK0T2OouChmvTiJ+9g+2t+7eKDercWFPYMaLN2LvHbj9o.i4u5XQ61AA.cQA.Wvtyq+z7DCDPOoX79GRTRFzBYeTZTO3b2iB6NmCjg54k4zCeYqqqWqq.8nbeyh2CexU1Pu1JWYiwJQQDOo3MdYir5207YFI.l3lmzUw9P8ZbwDpcc25nH3IVuwySm9q.8ay1m0d.9r9CvmMd.9r4Cvmsd.97cO.e99OpO5W2tSpRDkMN.B511vkYY0lSfNKSWH5e.3K8WIA

[d.healey]

@DanH Oh yeah, my maths is bad. I'll leave that to you for homework :)

[DanH]

@d-healey ha thanks! Either way it's still a bug that needs fixing right? What's the method for bug reporting these days?

[d.healey]

@DanH I'm not sure it is a bug. Once you're in the world of LAF you're in control and you may want it to be mono-polar even if the range is bi-polar.

Bug reporting is through github issues after the bug has been confirmed in a forum thread.

[DanH]

@d-healey Ok thanks, @Christoph-Hart is this a bug? If so will report :)

[DanH]

@d-healey ok sorted the scaling but now my sliders above zero are kinda oblong

HiseSnippet 1637.3ocuX8+SaaDE2FvcKoi1RWk1udheXyzlllPYrNU00DHgRJAHpIkUMDp5v9bxMruyy9bfzIj5eQ8Ogs8mT+On6cmShcBATZFcQnfde897d28du6Ri.tEILjGnomoUOehl92XzrGSzYyNXJSqVEM8rFMafZUsYKj1F87wggDaMc84egTtdlEzTe93y2.6hYVjDVZZGvoVj5TOpHgaiR6Pcc2BaSZQ8Ro8ZkpYwYaxc4Q.Vl2nflO15DbaxdXoZyYnoeip1TAOno.KHgZ5KrA2tWyN7SYw5e.MjdrKQRTTqI3nX1awcskHVxUayNTW6FCh4PMvKMRx.yGmAtmwtTa5P9IYh6nDfRrHc9PetQg27i.uhogWgTvaBPZtTPZgXHsjQSq.puHQhDO2znFSPBbvPZOMTh0Uat+ZAiM4fFLQdO7IjsB.hgVXtdgB4PvWq7zrYgTen.0EGfbwNnmgpxZSYj7VADHXdgK+XrarSqy4mTlYuEg3ZNpgMco1jfFvlVQvACV21DwlbOeNCHLWNkNKOl0MpWdqT1Eux04VX2wWRkp4CHsogPnrUDyRP4LyksCvmVGPMNHdYVNGxYfv14P7i+8Ux9mYy7nGgpUAQCQhNDDKx6XR.h6nnBU1kMiDPX.LfI4w.PdZLqt8Y0E6FQ1iG3gcouiX2WpqB8xiug48orS5yNHE6iA6.1Yxloc9PYhQx1bfzS6PEDH.AgxPYad.8cP1.R.PPYhO7wGgdDZ0bH7gENR98pGIUVEPtpvFQk6s9bWHygHXqNCCHPkHlCOPDw.Yt8P83QnN3tDjfC+Ov1haSRkBPXOdDSnLzj5fnheHDY2ig8nV4PQgDUdvGxMLgr9T5FGh.VQoOh4OvUxx3UhyEVoxEdzyLcygBxIUOj.GKMk9jZuBDkEWWtQORVxRErYy.nwrK5WdFpH3TX6LC9vhGA9U8uGfFjlP2GYVLeAzCQckFlQw+YIhenx.Px4xsChaHYLuEq2kYZxhrJrH2OdQNOqLg0NuCzf6UDKgINd2TQC4SnMjjMl01E1OygVSJdnAiqvgP0I7mnCMTVE8qTaQGyURXrMg1tivbEHnW6HkyPHT1LH42mqpSRJ1j4wKTKopi.EGTjf3r83Bx9LSUcR1yyhFWjiyDkIKZC3ttjfIJV1nO3pLzLtLLGRUYMTQn62nsTMt7Vpo63aE2CIkhbVMFUruOgcYyAz523AZ+p2GUvxKTMeua+luIITMJzmcQiTYXMEzAHp+UfyrwBrl9gFquVd3S27aBeWsN7UqtRFkAZmJR5WJoqnn2nB70u0.nqWtrjqT9qkxaIk2UQ2PRuc4M0zdcsJxEoObAjCQiOIPPkIJ8Jjtvn23YGYLpPBOQv8U51uWLL73yH3NKYNcsR8FR7dboSkGKGxnz6K0QcrLQ+dk7ReOgm7OBxYhAi4ybiO7gO7IIenUteSnaZ5aT3gSstOoj+fsqZ.9tkQ4.eR61TLb9.1Pu3T7ESqgZb4X1Lxv64m5g2Euxg2oNG8sCOGMebp9V8S0aDIDblJMeKiXTzmU+CQI2fRaJ7whFuh.02e1tHdCV4haarWjWS4VPLuwbRgRSiStogzCJrLan3NJGz7D5k.iO97oIc.dAFzsCoW3+kLBDL9Dh8NL9wi6fu+hYi6z2AP40w7M3mobATYQ9iHBbm3grm9.JETVzXeXlPWxqf4BjYKZxXzrSjiiKY1NgcSipLLb7uEkbUgvU2t7dP5vi1B59jRv+2cMmZrd.wkaQE8tHVGF4+8H304Aa.q+OuCfmGG035gdpw6cMpSXsEcldzdhb0N3kvp8FdiqGZsoqrXIC3AULFwsIwEtrCeZJymni9Ni8iD9QhqI2sjwtMpB8+CF3vKTqLk082V5npL6KwMWQyzTkb2vXa3cqSnb8huMDdUJ2NxUNTK8SUkuOuu.3xViLsS9FPVHb5NcV5Z68qSKDWxnAEdxvjw3bS.ixmF+E.iCuuPUGG3DTB.WvXq27k4I9Zvs8ETV6cwh.JbIG03W3FQvnh3CMxK6pOmr6bLcAIsLCzjvrUDeB9zWXQIsdegEGHTyCaEveqU7YM4uqvWq3.Xho9IUxXrqjFkb2UiB4Kn4Qsou0xRF9ODtX0jsY0YvlGOC1r1LXyONC1r9LXyOMC17jqzF4utT4HA2Ktb.Xznp5V955wCfUmB09WHLefwJ

[DanH]

Could some one help me with this little bit of code pls? I'm not sure how to get the parent name into this - perhaps I've misunderstood...

	// linear interpolate each slider
	// unfortunately you have to hardcode the slider amount
	// (if it's dynamic, use obj.parentName to fetch the parent slider pack)
	var c = Colours.mix(l, r, parseInt(obj.Arpeggiator1) / 16); // this is normally parseInt(obj.id)

	g.setColour(c);

HiseSnippet 1637.3ocuX8+SaaDE2FvcKoi1RWk1udheXyzlllPYrNU00DHgRJAHpIkUMDp5v9bxMruyy9bfzIj5eQ8Ogs8mT+On6cmShcBATZFcQnfde897d28du6Ri.tEILjGnomoUOehl92XzrGSzYyNXJSqVEM8rFMafZUsYKj1F87wggDaMc84egTtdlEzTe93y2.6hYVjDVZZGvoVj5TOpHgaiR6Pcc2BaSZQ8Ro8ZkpYwYaxc4Q.Vl2nflO15DbaxdXoZyYnoeip1TAOno.KHgZ5KrA2tWyN7SYw5e.MjdrKQRTTqI3nX1awcskHVxUayNTW6FCh4PMvKMRx.yGmAtmwtTa5P9IYh6nDfRrHc9PetQg27i.uhogWgTvaBPZtTPZgXHsjQSq.puHQhDO2znFSPBbvPZOMTh0Uat+ZAiM4fFLQdO7IjsB.hgVXtdgB4PvWq7zrYgTen.0EGfbwNnmgpxZSYj7VADHXdgK+XrarSqy4mTlYuEg3ZNpgMco1jfFvlVQvACV21DwlbOeNCHLWNkNKOl0MpWdqT1Eux04VX2wWRkp4CHsogPnrUDyRP4LyksCvmVGPMNHdYVNGxYfv14P7i+8Ux9mYy7nGgpUAQCQhNDDKx6XR.h6nnBU1kMiDPX.LfI4w.PdZLqt8Y0E6FQ1iG3gcouiX2WpqB8xiug48orS5yNHE6iA6.1Yxloc9PYhQx1bfzS6PEDH.AgxPYad.8cP1.R.PPYhO7wGgdDZ0bH7gENR98pGIUVEPtpvFQk6s9bWHygHXqNCCHPkHlCOPDw.Yt8P83QnN3tDjfC+Ov1haSRkBPXOdDSnLzj5fnheHDY2ig8nV4PQgDUdvGxMLgr9T5FGh.VQoOh4OvUxx3UhyEVoxEdzyLcygBxIUOj.GKMk9jZuBDkEWWtQORVxRErYy.nwrK5WdFpH3TX6LC9vhGA9U8uGfFjlP2GYVLeAzCQckFlQw+YIhenx.Px4xsChaHYLuEq2kYZxhrJrH2OdQNOqLg0NuCzf6UDKgINd2TQC4SnMjjMl01E1OygVSJdnAiqvgP0I7mnCMTVE8qTaQGyURXrMg1tivbEHnW6HkyPHT1LH42mqpSRJ1j4wKTKopi.EGTjf3r83Bx9LSUcR1yyhFWjiyDkIKZC3ttjfIJV1nO3pLzLtLLGRUYMTQn62nsTMt7Vpo63aE2CIkhbVMFUruOgcYyAz523AZ+p2GUvxKTMeua+luIITMJzmcQiTYXMEzAHp+UfyrwBrl9gFquVd3S27aBeWsN7UqtRFkAZmJR5WJoqnn2nB70u0.nqWtrjqT9qkxaIk2UQ2PRuc4M0zdcsJxEoObAjCQiOIPPkIJ8Jjtvn23YGYLpPBOQv8U51uWLL73yH3NKYNcsR8FR7dboSkGKGxnz6K0QcrLQ+dk7ReOgm7OBxYhAi4ybiO7gO7IIenUteSnaZ5aT3gSstOoj+fsqZ.9tkQ4.eR61TLb9.1Pu3T7ESqgZb4X1Lxv64m5g2Euxg2oNG8sCOGMebp9V8S0aDIDblJMeKiXTzmU+CQI2fRaJ7whFuh.02e1tHdCV4haarWjWS4VPLuwbRgRSiStogzCJrLan3NJGz7D5k.iO97oIc.dAFzsCoW3+kLBDL9Dh8NL9wi6fu+hYi6z2AP40w7M3mobATYQ9iHBbm3grm9.JETVzXeXlPWxqf4BjYKZxXzrSjiiKY1NgcSipLLb7uEkbUgvU2t7dP5vi1B59jRv+2cMmZrd.wkaQE8tHVGF4+8H304Aa.q+OuCfmGG035gdpw6cMpSXsEcldzdhb0N3kvp8FdiqGZsoqrXIC3AULFwsIwEtrCeZJymni9Ni8iD9QhqI2sjwtMpB8+CF3vKTqLk082V5npL6KwMWQyzTkb2vXa3cqSnb8huMDdUJ2NxUNTK8SUkuOuu.3xViLsS9FPVHb5NcV5Z68qSKDWxnAEdxvjw3bS.ixmF+E.iCuuPUGG3DTB.WvXq27k4I9Zvs8ETV6cwh.JbIG03W3FQvnh3CMxK6pOmr6bLcAIsLCzjvrUDeB9zWXQIsdegEGHTyCaEveqU7YM4uqvWq3.Xho9IUxXrqjFkb2UiB4Kn4Qsou0xRF9ODtX0jsY0YvlGOC1r1LXyONC1r9LXyOMC17jqzF4utT4HA2Ktb.Xznp5V955wCfUmB09WHLefwJ

[ulrik]

@DanH The parent name is "obj.parentName"

if you want the number of sliders you can use this:

//	This is the amount of sliders
	var sliders = Content.getComponent(obj.parentName).getNumSliders();

However I'm not sure it's a good idea to use "Content.getComponent" from inside a LAF, someone else have to respond to that.

[DanH]

@ulrik ah ok, I think I get it now, thanks

[d.healey]

@ulrik Use Content.getAllComponents instead. This will return an array, just use the first element.

[Christoph Hart]

However I'm not sure it's a good idea to use "Content.getComponent" from inside a LAF, someone else have to respond to that.

Nope, bad idea. The paint call should be as fast as possible and getComponent() and especially getAllComponents() should be avoided as it has to iterate over all components and find the one with the matching id. If you have less than 50-100 components this might be OK, but for bigger projects it's recommend to create a reference on init and then use this instead. If you have multiple sliderpacks, you can create a JSON object with the id as key:

const var SliderPacks =
{
  "SliderPack1": Content.getComponent("SliderPack1"),
  "SliderPack2": Content.getComponent("SliderPack2")
};

function drawRotarySlider(g, obj)
{
    var numSliders = SliderPacks[obj.parentName].getNumSliders();
}

[DanH]

@Christoph-Hart Thanks, I'm still struggling to understand how to implement this into that code.... :face_with_tears_of_joy:

	// linear interpolate each slider
	// unfortunately you have to hardcode the slider amount
	// (if it's dynamic, use obj.parentName to fetch the parent slider pack)
	var c = Colours.mix(l, r, parseInt(obj.Arpeggiator1) / 16); // this is normally parseInt(obj.id)

	g.setColour(c);

[d.healey]

@DanH Console.print(trace(obj)); Will show you all of the data that is available to you in the obj object.

[DanH]

@d-healey what am I looking for? I was hoping something like this might work

	// linear interpolate each slider
	// unfortunately you have to hardcode the slider amount
	// (if it's dynamic, use obj.parentName to fetch the parent slider pack)
	var c = Colours.mix(l, r, parseInt(obj.parentName) / numSliders); 

	g.setColour(c);