Installing Samples Dialog....

Lindon

OK I made a completely new blank project

added a single set of samples in a single sample map
made the ch1 file
made the hr1 file
put a keyboard on the UI (so it might play)
put this in the extensions

HISE_SAMPLE_DIALOG_SHOW_INSTALL_BUTTON=1
HISE_SAMPLE_DIALOG_SHOW_LOCATE_BUTTON=0

compiled it:
ran it , and got this:

So its failing to run the sample install process here too - so its nothing tho do with my (complicated) project..

Lindon

@d-healey OK ..

Anyone else want to test this version on Windows?

In fact anyone tested it on MacOS?

d.healey

I just tested on Windows and it works. So I think something went wrong when you built HISE. You can test by seeing if some of the scripting commands I added work for you, for example effect.isBypassed();

HiseSnippet 794.3oc0V0saSCCE1tsdhVXSLDBAhah1UsRipTXafzDhR+CUAaqiNl3tIOG2Uq4ZGbbFTgfq48iWDdC.6jrlTnqLpDHVtpm+b+Nmy24X2SIIzf.oB.KdvXeJ.dCT+wB8vlCwLAnaK.bEzN3.MU4DqpwXebP.0C.g4egUArXAPz22dVCLGKHzTU.vgRFg9J1HlNUau5ujw4cvdzCXix38F06RjhlRtLzfm7HWfOlbJ9D5tXqa4P.3Rs8XZopuFqoAFeZH8F2en78hX+OjEvNlSsB0.8MGTrZPygLtWuyy0...VnWZlmONyuMZGlGah9zJvMiL3jFQ1Z.L2zPpvTPp1EAoNRtm8.t.3Ay.uBwvaUTehh4qSsXw10QcElly.rormEVw9BxceHpoz3gPWcD9TZGkQXRDk2x0ccmMccqr8xkVtjo3GncNCqb5v3FWp47TmHxP0Sn51CFPI5xqkXZs3PLmcfjSq5qXBc4DaUYAmCkxUpr8fPAQyjBGoXWoltmnbkRerTwRepjyOaZvfYZylAJImSUyzrkEolWfkEgiNlpV2ja7P5DGMk1o6cKc45cj3BZFGkhtBldOeZhbZ2s1L5tfjNh4WuoaKrFaa3I5L94SUZlENvVzyLSOws+hnVzfS0Rey7yuvMLLToWHGqml1ZmOSLXpGSwOrb.Q.SON676e.W1clb4YlsWR3tJpGSSFNa7laF30T09ai2jMCKihY+ofs.pya+GuFHY+7sP8j7w9CkBFIddKBPEQICeyYAM.zQQeWHUPRqhe4q02OKEvT9yrQ9t02ODyyV0q+45MX9RNVco5H4ubkgeyB54QatCZRR4bEhqa6LWcv68PIMcGaw9pCtWAs++Af80xPMSbxNXsh8A.Dsa3n9lm3PnFfJDTt4OBAyYuWHV10JaATepvKR36luDi0rxvDi0N2HXDlnjGQhuuyN+bsHMFLIhd7TQyq3LxN0.Q2AZhC4V0ELx7vliHD6BsGXv9ri4gKPLOZAhYiEHlMWfX1ZAh4wKPLOYtwXeS4yC0xQwbSihdsidL.D1VfMrrHZJ3G.88bHB

Lindon

@d-healey said in Installing Samples Dialog....:

HiseSnippet 794.3oc0V0saSCCE1tsdhVXSLDBAhah1UsRipTXafzDhR+CUAaqiNl3tIOG2Uq4ZGbbFTgfq48iWDdC.6jrlTnqLpDHVtpm+b+Nmy24X2SIIzf.oB.KdvXeJ.dCT+wB8vlCwLAnaK.bEzN3.MU4DqpwXebP.0C.g4egUArXAPz22dVCLGKHzTU.vgRFg9J1HlNUau5ujw4cvdzCXix38F06RjhlRtLzfm7HWfOlbJ9D5tXqa4P.3Rs8XZopuFqoAFeZH8F2en78hX+OjEvNlSsB0.8MGTrZPygLtWuyy0...VnWZlmONyuMZGlGah9zJvMiL3jFQ1Z.L2zPpvTPp1EAoNRtm8.t.3Ay.uBwvaUTehh4qSsXw10QcElly.rormEVw9BxceHpoz3gPWcD9TZGkQXRDk2x0ccmMccqr8xkVtjo3GncNCqb5v3FWp47TmHxP0Sn51CFPI5xqkXZs3PLmcfjSq5qXBc4DaUYAmCkxUpr8fPAQyjBGoXWoltmnbkRerTwRepjyOaZvfYZylAJImSUyzrkEolWfkEgiNlpV2ja7P5DGMk1o6cKc45cj3BZFGkhtBldOeZhbZ2s1L5tfjNh4WuoaKrFaa3I5L94SUZlENvVzyLSOws+hnVzfS0Rey7yuvMLLToWHGqml1ZmOSLXpGSwOrb.Q.SON676e.W1clb4YlsWR3tJpGSSFNa7laF30T09ai2jMCKihY+ofs.pya+GuFHY+7sP8j7w9CkBFIddKBPEQICeyYAM.zQQeWHUPRqhe4q02OKEvT9yrQ9t02ODyyV0q+45MX9RNVco5H4ubkgeyB54QatCZRR4bEhqa6LWcv68PIMcGaw9pCtWAs++Af80xPMSbxNXsh8A.Dsa3n9lm3PnFfJDTt4OBAyYuWHV10JaATepvKR36luDi0rxvDi0N2HXDlnjGQhuuyN+bsHMFLIhd7TQyq3LxN0.Q2AZhC4V0ELx7vliHD6BsGXv9ri4gKPLOZAhYiEHlMWfX1ZAh4wKPLOYtwXeS4yC0xQwbSihdsidL.D1VfMrrHZJ3G.88bHB

well that's working perfectly...

Lindon

@d-healey deoes this work for you;

HiseSnippet 1318.3oc4X0saaaCElJNrqwMcasnWraFfPwtnEnKPxwwNd6h57iSWPSZLrcaGvtnfQh1lHzjBTTIwqnWr2n8HzGo7FrcnjrEsiaVVvZWwluJm+H+34vy4iJsUx.ZbrTgbVo23HJxYUb2wB8vcFRXBz96hbtGdeQrlv48nwZpxyGs83HRbLMD43T5YF2bVYYT5uKd51DNQDPKTgPuRxBnGvFwzEZa274LNeORHsGajk2UatefTrijKS.HUB6ghHAmPFPeAw31RXjysZExzRUWMQSiQNKusLbb2gxyDY9+JVL6XN0H3i5BKTl58j7PChMZQ6LjwCaO4nGiPN31EIhRYIhGfOjExlpuHg70oFbKhvNe3rzUAOea34c8gmiE7VNCd2C2MPwhzEVLX6NPwBJR8IPIvFVY9hbdOdGI3fPu1HxIz8Tfvz.dTMOum3tgm2i+w9Ih.MSJbkhWH0ziDO5wkea4UJ+txtyape+EZyrMJImSUKzropqtp.ejHYzwT0SbOkvSnScDN9ylewe37qc4OH6Ta4nTrufoOJhJ9PYcTdpB9qWt+tDMwTHx0A9EQUZlABN6ROEtgmUVVAuKM9DsLJ02QQRgYEbVUmemJqPrGWRzLwfdL.pLnB8UXaU9nyKZI7aNdpPyAMOiEpGNUwE+VygT1fgVsV+RybLlAnx3mSGerjnBQGOXRi0p3pU1rZ8ZMVeSODSSGMwvcvUWuVEu5072vRekrHZTsQs5UZrARSOWauTfgM78gPRyRKwcvu8tkcceHryu1.2G9Ct9UWy6IoJ2kEGwIiOJPSNk9hzpLXuOgGSyb3.4YPfftFYx+DKSzuR8LE6jDqkidlhDMjEDOav6R6SR35shhnDkYRDXVqRxstMGll.qVGHUKAKdqUqQ5uM2zqdCu59Yt0SNX.mdnLjN6ha56g4.BAkavSlxWFSeEM.tRZAnhcDrtGmn6pGymewZ2ZRk4RFfa2JcwNUYpgVhPK.T6tkeGLK8RyFfITxvDXamcrkYVctAnWal4Clg.hXldr8r7OJyxttv8d31LcvvEi2kV.dgtyO13MmY3t3V86Ck7BvtLdue9iOMfMT91LnbebWshRFACN5RFEAiaSwSYbtz+Hj0QWax51JJLFKrK6Wsb68M2NAxWpY096MS2psFISDyrW4HuCE5f0ldvBi2tYmNOSIShlOpKdZ5kkdJn8FRE1FNRPg7t19LtiRFG2GNNoqUrso1IpAlbUglNzSop3Y0ASsx6BMLPNNFFhXpoVyBNjnUrys8NcW75QRq2vKXRk8mStxbxqOmb04j2XN4ZyIWuPNKgdHIxbyXU7VL0VZMjnhuDY5Wb8HSgkpMAN7VdtMavzsISi8U4oI7TPY8XsE7Tma8Y8Kw9zMrsz0Ct+EOVbkIXrKyTcZINEZPAMFLdebNY4DsyNr3PoPFMTJXAy1N.WuGLfprw9BOPYWxJz7flcnbJI1pe96Zd.S.7zy1k+2JW3e8yEWU85avYv00Lgy8+Fjjk9eOI4Uk59xbJR2z2Z8YdcdUb5PcWCG7+VPsiLw78IS32v.IXWf3OfZSEtjgdMS1yHa.TWpHLU3Ofe4F8mvaZL5OwHJXxRAms7+N8SJbJQEFpiz29wAFYt8U5X34EFHAbceRP4HRfR9lfrOT0f0ampAPoH8+RwJ3CMxt9nzOdEhC6slGZDvU8lf.Siv2CY3EGSkaPLqeCho5MHlMtAwT6FDS8aPLadkwXdlvVIvGMlMjDTztU1EKmVSuXUB8mM0uAJA

Lindon

...hang on just to make sure I'm not making a silly silly mistake:

I'm putting these directives:

HISE_SAMPLE_DIALOG_SHOW_INSTALL_BUTTON=1
HISE_SAMPLE_DIALOG_SHOW_LOCATE_BUTTON=0

here:

thats right yes?

d.healey

@Lindon said in Installing Samples Dialog....:

@d-healey deoes this work for you;

I just see a keyboard, is that what I'm supposed to see?

@Lindon said in Installing Samples Dialog....:

thats right yes?

Yes that's correct.

When you open the .jucer file after exporting you should see this (except for Windows/MacOS in your case).

Lindon

@d-healey said in Installing Samples Dialog....:

@Lindon said in Installing Samples Dialog....:
@d-healey  deoes this work for you;
I just see a keyboard, is that what I'm supposed to see?

Well you are supposed to load up some sample data into the sampler..anything will do...and then you are supposed to see the install Samples button..

@Lindon said in Installing Samples Dialog....:

thats right yes?

Yes that's correct.

When you open the .jucer file after exporting you should see this (except for Windows/MacOS in your case).

I will check this - re-downlaoding and rebuilding your HIS version now.

Lindon

oh dear...no good news...

re-downloaded your build.

I rebuilt HISE
I deleted the contents of the Build folder in my project
I deleted the folder for the project under ../Use/AppData/...

I ran the "export standalone app" again...

This is what my autogenerated projucer file looks like:

I ran my compiled stand alone app...

it asked for my serial number - I gave it....

and it went straight into the UI - completely failing to show me any Sample Dialog....

I have no idea what else I can do now...sadly this is a MASSIVE issue for my client base....
@Christoph-Hart !!!!!!

d.healey

@Lindon Does it work if you remove the serial code stuff?

Lindon

@d-healey well it also didnt work for my very very simple example, with no serial work, I posted the snippet for so I cant see thats the problem - but I will try - tomorrow.

d.healey

It will be good if someone else can test.

Lindon

@d-healey OK I took out all the authorisation code...

..no difference it still fails to do ANY sample set up, now it just pops thru to the UI and cant find any ch files ...

I think perhaps I have been misleading you - the sample set up is supposed to pop BEFORE the authorisation....first thing.

Lindon

OK a morning (so far) of tedious compiling to test every scenario, in every case I deleted all the config data from AppData and deleted the entire contents of the binaries folder in the project here's the results:

HISE_SAMPLE_DIALOG_SHOW_INSTALL_BUTTON=1
HISE_SAMPLE_DIALOG_SHOW_LOCATE_BUTTON=1

-- shows both buttons

HISE_SAMPLE_DIALOG_SHOW_INSTALL_BUTTON=0
HISE_SAMPLE_DIALOG_SHOW_LOCATE_BUTTON=1

--show just the locate button

HISE_SAMPLE_DIALOG_SHOW_INSTALL_BUTTON=1
HISE_SAMPLE_DIALOG_SHOW_LOCATE_BUTTON=0

-- shows nothing

HISE_SAMPLE_DIALOG_SHOW_LOCATE_BUTTON=0

-- shows nothing !!!

HISE_SAMPLE_DIALOG_SHOW_LOCATE_BUTTON=1

-- shows both buttons

HISE_SAMPLE_DIALOG_SHOW_INSTALL_BUTTON=1

-- shows both buttons

HISE_SAMPLE_DIALOG_SHOW_INSTALL_BUTTON=0

-- shows the locate button

So if you explicit set the locate button to OFF it shows you nothing

So I'm begining to think its something in this logic:

#if HISE_SAMPLE_DIALOG_SHOW_INSTALL_BUTTON
		installSampleButton->setVisible(true);
		installSampleButton->setBounds(b.removeFromTop(32));
#else
		installSampleButton->setVisible(false);
#endif

#if HISE_SAMPLE_DIALOG_SHOW_LOCATE_BUTTON
		resolveSamplesButton->setVisible(true);
		resolveSamplesButton->setBounds(b.removeFromBottom(32));
#else
		resolveSamplesButton->setVisible(false);
#endif

which sorta says to me that if you dont set either then nothing would show....and thats not the case (cause I just checked..)

d.healey

The behaviour you've described is exactly the same as it was before my fix, no? If so that would lead me to believe that you are compiling your plugin against a different version of the source code, how many copies of the source do you have on your system?

For your testing are you using debug builds? They are much faster to build so take out a bit of the tedium of testing.

Lindon

@d-healey said in Installing Samples Dialog....:

The behaviour you've described is exactly the same as it was before my fix, no? If so that would lead me to believe that you are compiling your plugin against a different version of the source code, how many copies of the source do you have on your system?

For your testing are you using debug builds? They are much faster to build so take out a bit of the tedium of testing.

Perhaps I am - how would I check other than to look at the path in the preferences?

I may have built the wrong version of HISE - but I did the "normal" things - opened projucer - pointed it at the standalone project INSIDE the folder structure for Hise-develop - compiled from there....

In any case we checked that your effect.isBypassed(0 worked properly - to assure ourselves I was using the correct HISE ... and I was/am...

Lindon

@d-healey -- right I went and removed all your logic from FrontendBar.cpp and replaced it with:

	if (currentState[SamplesNotInstalled])
	{
		resolveLicenseButton->setVisible(false);
		registerProductButton->setVisible(false);

		auto b = getLocalBounds().withSizeKeepingCentre(200, 50);

		installSampleButton->setVisible(true);
		installSampleButton->setBounds(b.removeFromTop(32));
		resolveSamplesButton->setVisible(false);

and a similar thing for the text:

	case DeactiveOverlay::SamplesNotInstalled:
        return "Please click below to install the samples from the downloaded archive.";
		break;

and compiled that... then opened it and compiled the project: I get this:

So I think we can say I'm using the correct HISE

d.healey

@Lindon said in Installing Samples Dialog....:

In any case we checked that your effect.isBypassed(0 worked properly - to assure ourselves I was using the correct HISE ... and I was/am...

That confirmed that you used my version of the source code to build HISE, but not that you used it to build your project.

So I think we can say I'm using the correct HISE

Yup, so something weird is happening. Need more testers :)

@d-healey -- right I went and removed all your logic from FrontendBar.cpp and replaced it with:

Are you sure it was my logic and not Christoph's? The changes I made were fairly minor and could be mistaken for the original code. (although what you posted before was definitely mine).

Lindon

@d-healey agreed more testing would be good - but I think this area need s a re-evaluation - its a bunch of confusing double negative flags (probably my fault as much as any one):

I suggest we dump these two flags:

HISE_SAMPLE_DIALOG_SHOW_INSTALL_BUTTON=1
HISE_SAMPLE_DIALOG_SHOW_LOCATE_BUTTON=0

to be replaced by:

HISE_SAMPLE_DIALOG_INSTALL_ONLY

(you get ONLY the install button and text)

and

HISE_SAMPLE_DIALOG_LOCATE_ONLY

(you get ONLY the locate button and text)

if as a developer you put both in then - firstly : silly you and secondly you get what you get

if you put in neither you get both buttons.

This leads to this logic in FrontendBar.cpp

	case DeactiveOverlay::SamplesNotInstalled:

#if HISE_SAMPLE_DIALOG_INSTALL_ONLY
        return "Please click below to install the samples from the downloaded archive.";
		break;
#elif HISE_SAMPLE_DIALOG_LOCATE_ONLY
		return "Please click below to point to the location where you've already installed the samples.";
		break;
#else
		return "Please click below to install the samples from the downloaded archive or point to the location where you've already installed the samples.";
		break;

#endif

..and similar for the button displays

d.healey

@Lindon

but I think this area need s a re-evaluation - its a bunch of confusing double negative flags (probably my fault as much as any one):

True, but it's still working for me and not for you, I don't think changing the flags would fix that.

Do you have multiple copies of the source code on your machine?

Installing Samples Dialog....

17

1.7k

11.8k

102.4k