Page 1 of 1

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Sun Apr 22, 2012 1:30 pm
by Tommie
I don't have an earlier version installed, but since the patch is just a few lines long, someone using an older controlP5 can probably test it easily.

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Sun Apr 22, 2012 1:53 pm
by copterrichie
I just tried compiling the Patch with Processing 1.5.1 and it will not compile. :(

Error message: "This Function hideLabels() does not exist."

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Sun Apr 22, 2012 2:05 pm
by Hamburger
Ok. This is serrious.
We have an incompatiblity issue to deal with in the future from now on till forever.

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Sun Apr 22, 2012 2:35 pm
by Tommie
Why not simply require the newer version of controlP5 at some point in the future?

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Sun Apr 22, 2012 4:24 pm
by Alexinparis
Hi,
I tried the latest version of ControlP5. There is the incompatibility you mentioned. We could handle this by upgrading everything to the last solution, not a problem.

But there is one point I'm not happy with:
The Numberbox is the new version has now a small triangle inside and the text value is not aligned as before.
It's not a good point if we want to optimize GUI space. I would like to keep Numberbox items thanks to their scrolling ability.
I didn't manage to remove the triangles via object attributes, so until a better solution I will stay with former versions.

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Mon Apr 23, 2012 9:23 am
by Hamburger
Tommie wrote:Why not simply require the newer version of controlP5 at some point in the future?

sure, we could do that, but it is not to the point I was poking at. Whichever one of the two options we choose, we will be faced with error/support requests from people with the other s/w version and non-compilable GUI code.
About the upgrade, Alex has made his point. Fine with me.

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Mon Apr 23, 2012 2:43 pm
by copterrichie
IMO, most people don't bother compiling their own version of the GUI, so it is really a non issue in my opinion.

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Tue May 15, 2012 10:36 am
by Katch
Just thought I'd post and say I ran into this problem today - I was only a few google searches away from submitting an error/support request when I found this post. Tommie's patch sorted it for me.

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Sat Jun 02, 2012 12:55 am
by LenzGr
FWIW, all the sudden this happens to me as well, after pulling the latest trunk (running ControlP5 version 0.6.12): "The function hideLabels() does not exist"

This is the offending line:

Code: Select all

    checkbox[i].hideLabels();   


Do I need to upgrade the library in order to build the GUI?

Update: Yes, updating to the latest version of ControlP5 (0.7.5) fixed the compile error.

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Fri Jun 08, 2012 2:21 pm
by Hamburger
Alexinparis wrote:Hi,
I tried the latest version of ControlP5. There is the incompatibility you mentioned. We could handle this by upgrading everything to the last solution, not a problem.
[...]
I didn't manage to remove the triangles via object attributes, so until a better solution I will stay with former versions.


so we have (been) moved to the newer ControlP5 library. I hope we do not receive too many requests for support on older versions of the GUI.

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Fri Jun 08, 2012 2:32 pm
by fiendie
Hamburger wrote:so we have (been) moved to the newer ControlP5 library. I hope we do not receive too many requests for support on older versions of the GUI.

Do you run a call center? :)
Just document it somewhere and be done with it.
It's an open source project. Try getting support from one of the Linux Kernel devs ;)

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Fri Jun 08, 2012 7:39 pm
by kos
the mwi-gui-component / ControlP5 fork used by multiwii can be found here -> https://github.com/treymarc/mwi-gui-com ... ster/build

Re: [Patch] MultiwiiConf fails to compile with current contr

Posted: Fri Jun 29, 2012 11:22 pm
by hdias
I had the same problem and solved it with the following code (I hope that this code is right...):

MultiWiiConf_2_0
Processing-1.5.1
controlP5-0.7.5

Code: Select all

//    hideLabel(checkbox1[i].addItem(i + "1",1));hideLabel(checkbox1[i].addItem(i + "2",2));hideLabel(checkbox1[i].addItem(i + "3",3));
//    hideLabel(checkbox1[i].addItem(i + "4",4));hideLabel(checkbox1[i].addItem(i + "5",5));hideLabel(checkbox1[i].addItem(i + "6",6));

    checkbox1[i].addItem(i + "1",1);
    checkbox1[i].addItem(i + "2",2);
    checkbox1[i].addItem(i + "3",3);
    checkbox1[i].addItem(i + "4",4);
    checkbox1[i].addItem(i + "5",5);
    checkbox1[i].addItem(i + "6",6);
    checkbox1[i].hideLabels();

    checkbox2[i] =  controlP5.addCheckBox("cb_"+i,xBox+170,yBox+20+13*i);
    checkbox2[i].setColorActive(color(255));checkbox2[i].setColorBackground(color(120));
    checkbox2[i].setItemsPerRow(6);checkbox2[i].setSpacingColumn(10);
    checkbox2[i].setLabel("");
//    hideLabel(checkbox2[i].addItem(i + "1_",1));hideLabel(checkbox2[i].addItem(i + "2_",2));hideLabel(checkbox2[i].addItem(i + "3_",3));
//    hideLabel(checkbox2[i].addItem(i + "4_",4));hideLabel(checkbox2[i].addItem(i + "5_",5));hideLabel(checkbox2[i].addItem(i + "6_",6));

    checkbox2[i].addItem(i + "1_",1);
    checkbox2[i].addItem(i + "2_",2);
    checkbox2[i].addItem(i + "3_",3);
    checkbox2[i].addItem(i + "4_",4);
    checkbox2[i].addItem(i + "5_",5);
    checkbox2[i].addItem(i + "6_",6);
    checkbox2[i].hideLabels();