couple of weeks ago accidentaly deleted most of my work. Before I did find a neat way to round edges using hull(), rather than minkowski which is kinda too slow for my taste, I tried to replicate from memory the code but no luck, it was something like:   //**** HandlerBar module ****** idiameter = 32.5; band_dickness = 3; band_width = 20; xdiameter = idiameter + band_dickness; fix_block_high = 15; stop = 2; sideWall = 2 *2; //**** Flash light Module ***** detach_lower_block_length = fix_block_length - stop; detach_lower_block_width = band_width - sideWall; detach_lower_block_high = 3; chanel_width = 3; chanel_neck = 3; detach_upper_block_length = 35; detach_upper_block_width = 25; detach_upper_block_high = 15; roundRadio = 1; screw_offset_length = 8 * 2; //two per side screw_offset_width = band_width; screw_offset_dickness = 3 * 2;  //uper & lower sides //union() //    {     hull()         {         for(i=[0:3])         translate(-[i % 2] * band_width, + i[i % 2] * xdiameter + screw_offset_length, 0], cube(), center = true);    //I get a syntax error on this line, why?         } //*****            I found this other one with two for()s, the problem is does not round the top of cube edges:    hull(){     for(end = [-1,1])     for(side = [-1,1])     translate([end * (length / 2 - orad), side * (width / 2 - orad), 0])         {     cylinder(r = orad, h = height);     sphere(orad);         }     } I will appreciate any help. Thank you for reading,

     translate(-[i % 2] * band_width, + i[i % 2] * xdiameter +

 for(end = [-1,1])
 for(side = [-1,1])
 translate([end * (length / 2 - orad), side * (width / 2 - orad), 0])
     {
 cylinder(r = orad, h = height);
 sphere(orad);
     }
 }

Your brackets () and [] are all mixed up: value=1 //union() // { hull() { for(i=[0:3]) translate([-(i % 2) * band_width, + i*(i % 2) * xdiameter + screw_offset_length, 0], cube(value, center = true); //I get a syntax error on this line, why? 2015-02-22 9:02 GMT+01:00 jose tav <j_tav@yahoo.com>: > couple of weeks ago accidentaly deleted most of my work. > Before I did find a neat way to round edges using hull(), rather than > minkowski which is kinda too slow for my taste, I tried to replicate from > memory the code but no luck, > it was something like: > > //**** HandlerBar module ****** > > idiameter = 32.5; > band_dickness = 3; > band_width = 20; > xdiameter = idiameter + band_dickness; > > fix_block_high = 15; > stop = 2; > sideWall = 2 *2; > > //**** Flash light Module ***** > > detach_lower_block_length = fix_block_length - stop; > detach_lower_block_width = band_width - sideWall; > detach_lower_block_high = 3; > chanel_width = 3; > chanel_neck = 3; > detach_upper_block_length = 35; > detach_upper_block_width = 25; > detach_upper_block_high = 15; > roundRadio = 1; > > screw_offset_length = 8 * 2; //two per side > screw_offset_width = band_width; > screw_offset_dickness = 3 * 2; //uper & lower sides > > > //union() > // { > hull() > { > for(i=[0:3]) > > translate(-[i % 2] * band_width, + i[i % 2] * xdiameter + > screw_offset_length, 0], cube(), center = true); //I get a syntax error > on this line, why? > } > //***** > > I found this other one with two for()s, the problem is does not round the > top of cube edges: > > hull(){ > > for(end = [-1,1]) > for(side = [-1,1]) > translate([end * (length / 2 - orad), side * (width / 2 - orad), 0]) > { > cylinder(r = orad, h = height); > sphere(orad); > } > } > > I will appreciate any help. > Thank you for reading, > > > _______________________________________________ > OpenSCAD mailing list > Discuss@lists.openscad.org > http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org > > -- stempeldergeschichte@googlemail.com <karsten@rohrbach.de> P.S. Falls meine E-Mail kürzer ausfällt als Dir angenehm ist: Ich probiere gerade aus kurze Antworten statt gar keine Antworten zu schreiben. Wenn Du gerne mehr lesen möchtest, dann lass es mich bitte wissen. P.S. In case my e-mail is shorter than you enjoy: I am currently trying short replies instead of no replies at all. Please let me know, if you like to read more. Enjoy!

     translate(-[i % 2] * band_width, + i[i % 2] * xdiameter +

 for(end = [-1,1])
 for(side = [-1,1])
 translate([end * (length / 2 - orad), side * (width / 2 - orad), 0])
     {
 cylinder(r = orad, h = height);
 sphere(orad);
     }
 }

Still mixed up :) But I did only fixed syntax, not sure it does what you need: value=1; //union() // { hull() { for(i=[0:3]) translate([-(i % 2) * band_width, + i*(i % 2) * xdiameter + screw_offset_length, 0]) cube(value, center = true); //I get a syntax error on this line, why? 2015-02-22 11:33 GMT+01:00 Peter Falke <stempeldergeschichte@googlemail.com> : > Your brackets () and [] are all mixed up: > > value=1 > > //union() > > // { > > hull() > > { > > for(i=[0:3]) > > > translate([-(i % 2) * band_width, + i*(i % 2) * xdiameter + > screw_offset_length, 0], cube(value, center = true); //I get a syntax error > on this line, why? > > > 2015-02-22 9:02 GMT+01:00 jose tav <j_tav@yahoo.com>: > >> couple of weeks ago accidentaly deleted most of my work. >> Before I did find a neat way to round edges using hull(), rather than >> minkowski which is kinda too slow for my taste, I tried to replicate from >> memory the code but no luck, >> it was something like: >> >> //**** HandlerBar module ****** >> >> idiameter = 32.5; >> band_dickness = 3; >> band_width = 20; >> xdiameter = idiameter + band_dickness; >> >> fix_block_high = 15; >> stop = 2; >> sideWall = 2 *2; >> >> //**** Flash light Module ***** >> >> detach_lower_block_length = fix_block_length - stop; >> detach_lower_block_width = band_width - sideWall; >> detach_lower_block_high = 3; >> chanel_width = 3; >> chanel_neck = 3; >> detach_upper_block_length = 35; >> detach_upper_block_width = 25; >> detach_upper_block_high = 15; >> roundRadio = 1; >> >> screw_offset_length = 8 * 2; //two per side >> screw_offset_width = band_width; >> screw_offset_dickness = 3 * 2; //uper & lower sides >> >> >> //union() >> // { >> hull() >> { >> for(i=[0:3]) >> >> translate(-[i % 2] * band_width, + i[i % 2] * xdiameter + >> screw_offset_length, 0], cube(), center = true); //I get a syntax error >> on this line, why? >> } >> //***** >> >> I found this other one with two for()s, the problem is does not round >> the top of cube edges: >> >> hull(){ >> >> for(end = [-1,1]) >> for(side = [-1,1]) >> translate([end * (length / 2 - orad), side * (width / 2 - orad), 0]) >> { >> cylinder(r = orad, h = height); >> sphere(orad); >> } >> } >> >> I will appreciate any help. >> Thank you for reading, >> >> >> _______________________________________________ >> OpenSCAD mailing list >> Discuss@lists.openscad.org >> http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org >> >> > > > -- > stempeldergeschichte@googlemail.com <karsten@rohrbach.de> > > P.S. Falls meine E-Mail kürzer ausfällt als Dir angenehm ist: > Ich probiere gerade aus kurze Antworten statt gar keine Antworten zu > schreiben. > Wenn Du gerne mehr lesen möchtest, dann lass es mich bitte wissen. > > P.S. In case my e-mail is shorter than you enjoy: > I am currently trying short replies instead of no replies at all. > Please let me know, if you like to read more. > > Enjoy! > -- stempeldergeschichte@googlemail.com <karsten@rohrbach.de> P.S. Falls meine E-Mail kürzer ausfällt als Dir angenehm ist: Ich probiere gerade aus kurze Antworten statt gar keine Antworten zu schreiben. Wenn Du gerne mehr lesen möchtest, dann lass es mich bitte wissen. P.S. In case my e-mail is shorter than you enjoy: I am currently trying short replies instead of no replies at all. Please let me know, if you like to read more. Enjoy!