discuss@lists.openscad.org

OpenSCAD general discussion Mailing-list

View all threads

Issues with latest master.

NH
nop head
Thu, Nov 29, 2018 3:32 PM

I get these warnings, which are probably genuine errors, but without a
filename, very hard to find.

OpenSCAD 2018.11.29.ci630

WARNING: Ignoring unknown variable '$back', line 117.

WARNING: Ignoring unknown variable '$back', line 66.

WARNING: Ignoring unknown variable '$back', line 66.

WARNING: Ignoring unknown variable '$bom', line 25.

ARNING: Unable to convert translate([2.63, undef, 0]) parameter to a vec3
or vec2 of numbers, line 159

WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or
vec2 of numbers, line 56

WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or
vec2 of numbers, line 56

WARNING: Unable to convert translate([undef, 4.63, 0]) parameter to a vec3
or vec2 of numbers, line 163

WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or
vec2 of numbers, line 56

WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or
vec2 of numbers, line 56

WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or
vec2 of numbers, line 56

WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or
vec2 of numbers, line 56

WARNING: Unable to convert translate([]) parameter to a vec3 or vec2 of
numbers, line 56

WARNING: Unable to convert translate([]) parameter to a vec3 or vec2 of
numbers, line 56

WARNING: Unable to convert translate([]) parameter to a vec3 or vec2 of
numbers, line 56

WARNING: Unable to convert translate([]) parameter to a vec3 or vec2 of
numbers, line 56

Are all warnings supposed to show the file now, or is that still work in
progress?

Another thing I noticed is the used file cache has way too many files. I
think files used with both absolute and relative file paths get two cache
entries. I don't know if this is a new bug, or has always been the case. It
happens with a library that uses some of its own files with a relative path
but the whole library is used with an absolute path (that comes from the
OPENSCADLIBS environment variable) in the project .

I get these warnings, which are probably genuine errors, but without a filename, very hard to find. OpenSCAD 2018.11.29.ci630 WARNING: Ignoring unknown variable '$back', line 117. WARNING: Ignoring unknown variable '$back', line 66. WARNING: Ignoring unknown variable '$back', line 66. WARNING: Ignoring unknown variable '$bom', line 25. ARNING: Unable to convert translate([2.63, undef, 0]) parameter to a vec3 or vec2 of numbers, line 159 WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or vec2 of numbers, line 56 WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or vec2 of numbers, line 56 WARNING: Unable to convert translate([undef, 4.63, 0]) parameter to a vec3 or vec2 of numbers, line 163 WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or vec2 of numbers, line 56 WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or vec2 of numbers, line 56 WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or vec2 of numbers, line 56 WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or vec2 of numbers, line 56 WARNING: Unable to convert translate([]) parameter to a vec3 or vec2 of numbers, line 56 WARNING: Unable to convert translate([]) parameter to a vec3 or vec2 of numbers, line 56 WARNING: Unable to convert translate([]) parameter to a vec3 or vec2 of numbers, line 56 WARNING: Unable to convert translate([]) parameter to a vec3 or vec2 of numbers, line 56 Are all warnings supposed to show the file now, or is that still work in progress? Another thing I noticed is the used file cache has way too many files. I think files used with both absolute and relative file paths get two cache entries. I don't know if this is a new bug, or has always been the case. It happens with a library that uses some of its own files with a relative path but the whole library is used with an absolute path (that comes from the OPENSCADLIBS environment variable) in the project .
MK
Marius Kintel
Thu, Nov 29, 2018 4:06 PM

On Nov 29, 2018, at 10:32 AM, nop head nop.head@gmail.com wrote:

Are all warnings supposed to show the file now, or is that still work in progress?

At least some of that is an open PR: https://github.com/openscad/openscad/pull/2593

Another thing I noticed is the used file cache has way too many files. I think files used with both absolute and relative file paths get two cache entries.

I had a similar concern last night, see my comment here: https://github.com/openscad/openscad/pull/2593#discussion_r237359938
Not sure if we may need to backtrack through some of the warning stuff to figure this out..

-Marius

> On Nov 29, 2018, at 10:32 AM, nop head <nop.head@gmail.com> wrote: > Are all warnings supposed to show the file now, or is that still work in progress? > At least some of that is an open PR: https://github.com/openscad/openscad/pull/2593 > Another thing I noticed is the used file cache has way too many files. I think files used with both absolute and relative file paths get two cache entries. I had a similar concern last night, see my comment here: https://github.com/openscad/openscad/pull/2593#discussion_r237359938 Not sure if we may need to backtrack through some of the warning stuff to figure this out.. -Marius
NH
nop head
Thu, Nov 29, 2018 5:09 PM

Not sure why you would want to put the main file path into the cache key.
In MDI mode I expect the used files in my library to be cached once, not
once for each open window. Is there a reason why the cached files can't be
shared between different windows?

On Thu, 29 Nov 2018 at 16:06, Marius Kintel marius@kintel.net wrote:

On Nov 29, 2018, at 10:32 AM, nop head nop.head@gmail.com wrote:

Are all warnings supposed to show the file now, or is that still work in

progress?

At least some of that is an open PR:
https://github.com/openscad/openscad/pull/2593

Another thing I noticed is the used file cache has way too many files. I

think files used with both absolute and relative file paths get two cache
entries.

I had a similar concern last night, see my comment here:
https://github.com/openscad/openscad/pull/2593#discussion_r237359938
Not sure if we may need to backtrack through some of the warning stuff to
figure this out..

-Marius


OpenSCAD mailing list
Discuss@lists.openscad.org
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org

Not sure why you would want to put the main file path into the cache key. In MDI mode I expect the used files in my library to be cached once, not once for each open window. Is there a reason why the cached files can't be shared between different windows? On Thu, 29 Nov 2018 at 16:06, Marius Kintel <marius@kintel.net> wrote: > > > On Nov 29, 2018, at 10:32 AM, nop head <nop.head@gmail.com> wrote: > > > Are all warnings supposed to show the file now, or is that still work in > progress? > > > At least some of that is an open PR: > https://github.com/openscad/openscad/pull/2593 > > > Another thing I noticed is the used file cache has way too many files. I > think files used with both absolute and relative file paths get two cache > entries. > > I had a similar concern last night, see my comment here: > https://github.com/openscad/openscad/pull/2593#discussion_r237359938 > Not sure if we may need to backtrack through some of the warning stuff to > figure this out.. > > -Marius > > > _______________________________________________ > OpenSCAD mailing list > Discuss@lists.openscad.org > http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org >
MK
Marius Kintel
Thu, Nov 29, 2018 5:29 PM

On Nov 29, 2018, at 12:09 PM, nop head nop.head@gmail.com wrote:

Not sure why you would want to put the main file path into the cache key. In MDI mode I expect the used files in my library to be cached once, not once for each open window. Is there a reason why the cached files can't be shared between different windows?

Exactly, that was the cause of my concern. If the compiled file is depending on the main file, this won’t fly, so we should probably find a way of decoupling that.
Let’s wait for Michael to comment and see where we can take it from there.

-Marius

> On Nov 29, 2018, at 12:09 PM, nop head <nop.head@gmail.com> wrote: > > Not sure why you would want to put the main file path into the cache key. In MDI mode I expect the used files in my library to be cached once, not once for each open window. Is there a reason why the cached files can't be shared between different windows? > Exactly, that was the cause of my concern. If the compiled file is depending on the main file, this won’t fly, so we should probably find a way of decoupling that. Let’s wait for Michael to comment and see where we can take it from there. -Marius
M
MichaelAtOz
Thu, Nov 29, 2018 8:15 PM

Was 'ARNING:' a cut & paste problem or in the console representing a code
typo?


Admin - email* me if you need anything, or if I've done something stupid...

  • click on my MichaelAtOz label, there is a link to email me.

Unless specifically shown otherwise above, my contribution is in the Public Domain; to the extent possible under law, I have waived all copyright and related or neighbouring rights to this work. Obviously inclusion of works of previous authors is not included in the above.

The TPP is no simple “trade agreement.”  Fight it! http://www.ourfairdeal.org/  time is running out!

Sent from: http://forum.openscad.org/

Was 'ARNING:' a cut & paste problem or in the console representing a code typo? ----- Admin - email* me if you need anything, or if I've done something stupid... * click on my MichaelAtOz label, there is a link to email me. Unless specifically shown otherwise above, my contribution is in the Public Domain; to the extent possible under law, I have waived all copyright and related or neighbouring rights to this work. Obviously inclusion of works of previous authors is not included in the above. The TPP is no simple “trade agreement.” Fight it! http://www.ourfairdeal.org/ time is running out! -- Sent from: http://forum.openscad.org/
NH
nop head
Thu, Nov 29, 2018 9:38 PM

Yes must have been an error on my part. WARNING is always spelt correctly.

On Thu, 29 Nov 2018 at 20:16, MichaelAtOz oz.at.michael@gmail.com wrote:

Was 'ARNING:' a cut & paste problem or in the console representing a code
typo?


Admin - email* me if you need anything, or if I've done something stupid...

  • click on my MichaelAtOz label, there is a link to email me.

Unless specifically shown otherwise above, my contribution is in the
Public Domain; to the extent possible under law, I have waived all
copyright and related or neighbouring rights to this work. Obviously
inclusion of works of previous authors is not included in the above.

The TPP is no simple “trade agreement.”  Fight it!
http://www.ourfairdeal.org/  time is running out!

Sent from: http://forum.openscad.org/


OpenSCAD mailing list
Discuss@lists.openscad.org
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org

Yes must have been an error on my part. WARNING is always spelt correctly. On Thu, 29 Nov 2018 at 20:16, MichaelAtOz <oz.at.michael@gmail.com> wrote: > Was 'ARNING:' a cut & paste problem or in the console representing a code > typo? > > > > ----- > Admin - email* me if you need anything, or if I've done something stupid... > > * click on my MichaelAtOz label, there is a link to email me. > > Unless specifically shown otherwise above, my contribution is in the > Public Domain; to the extent possible under law, I have waived all > copyright and related or neighbouring rights to this work. Obviously > inclusion of works of previous authors is not included in the above. > > The TPP is no simple “trade agreement.” Fight it! > http://www.ourfairdeal.org/ time is running out! > -- > Sent from: http://forum.openscad.org/ > > _______________________________________________ > OpenSCAD mailing list > Discuss@lists.openscad.org > http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org >
NH
nop head
Fri, Nov 30, 2018 11:52 AM

Another issue is false is no longer converted to 0 when used in a vector
parameter to translate.

WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or
vec2 of numbers, line 56

The undef is actually false. I thought true and false were converted to 1 ,
0 when a number was required, as C.

On Thu, 29 Nov 2018 at 21:38, nop head nop.head@gmail.com wrote:

Yes must have been an error on my part. WARNING is always spelt correctly.

On Thu, 29 Nov 2018 at 20:16, MichaelAtOz oz.at.michael@gmail.com wrote:

Was 'ARNING:' a cut & paste problem or in the console representing a code
typo?


Admin - email* me if you need anything, or if I've done something
stupid...

  • click on my MichaelAtOz label, there is a link to email me.

Unless specifically shown otherwise above, my contribution is in the
Public Domain; to the extent possible under law, I have waived all
copyright and related or neighbouring rights to this work. Obviously
inclusion of works of previous authors is not included in the above.

The TPP is no simple “trade agreement.”  Fight it!
http://www.ourfairdeal.org/  time is running out!

Sent from: http://forum.openscad.org/


OpenSCAD mailing list
Discuss@lists.openscad.org
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org

Another issue is false is no longer converted to 0 when used in a vector parameter to translate. WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or vec2 of numbers, line 56 The undef is actually false. I thought true and false were converted to 1 , 0 when a number was required, as C. On Thu, 29 Nov 2018 at 21:38, nop head <nop.head@gmail.com> wrote: > Yes must have been an error on my part. WARNING is always spelt correctly. > > On Thu, 29 Nov 2018 at 20:16, MichaelAtOz <oz.at.michael@gmail.com> wrote: > >> Was 'ARNING:' a cut & paste problem or in the console representing a code >> typo? >> >> >> >> ----- >> Admin - email* me if you need anything, or if I've done something >> stupid... >> >> * click on my MichaelAtOz label, there is a link to email me. >> >> Unless specifically shown otherwise above, my contribution is in the >> Public Domain; to the extent possible under law, I have waived all >> copyright and related or neighbouring rights to this work. Obviously >> inclusion of works of previous authors is not included in the above. >> >> The TPP is no simple “trade agreement.” Fight it! >> http://www.ourfairdeal.org/ time is running out! >> -- >> Sent from: http://forum.openscad.org/ >> >> _______________________________________________ >> OpenSCAD mailing list >> Discuss@lists.openscad.org >> http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org >> >
NH
nop head
Fri, Nov 30, 2018 11:58 AM

BTW, it was actually a mistake that I was passing false but it is
misleading to say it is undef.

On Fri, 30 Nov 2018 at 11:52, nop head nop.head@gmail.com wrote:

Another issue is false is no longer converted to 0 when used in a vector
parameter to translate.

WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or
vec2 of numbers, line 56

The undef is actually false. I thought true and false were converted to 1
, 0 when a number was required, as C.

On Thu, 29 Nov 2018 at 21:38, nop head nop.head@gmail.com wrote:

Yes must have been an error on my part. WARNING is always spelt correctly.

On Thu, 29 Nov 2018 at 20:16, MichaelAtOz oz.at.michael@gmail.com
wrote:

Was 'ARNING:' a cut & paste problem or in the console representing a code
typo?


Admin - email* me if you need anything, or if I've done something
stupid...

  • click on my MichaelAtOz label, there is a link to email me.

Unless specifically shown otherwise above, my contribution is in the
Public Domain; to the extent possible under law, I have waived all
copyright and related or neighbouring rights to this work. Obviously
inclusion of works of previous authors is not included in the above.

The TPP is no simple “trade agreement.”  Fight it!
http://www.ourfairdeal.org/  time is running out!

Sent from: http://forum.openscad.org/


OpenSCAD mailing list
Discuss@lists.openscad.org
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org

BTW, it was actually a mistake that I was passing false but it is misleading to say it is undef. On Fri, 30 Nov 2018 at 11:52, nop head <nop.head@gmail.com> wrote: > Another issue is false is no longer converted to 0 when used in a vector > parameter to translate. > > WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or > vec2 of numbers, line 56 > > The undef is actually false. I thought true and false were converted to 1 > , 0 when a number was required, as C. > > On Thu, 29 Nov 2018 at 21:38, nop head <nop.head@gmail.com> wrote: > >> Yes must have been an error on my part. WARNING is always spelt correctly. >> >> On Thu, 29 Nov 2018 at 20:16, MichaelAtOz <oz.at.michael@gmail.com> >> wrote: >> >>> Was 'ARNING:' a cut & paste problem or in the console representing a code >>> typo? >>> >>> >>> >>> ----- >>> Admin - email* me if you need anything, or if I've done something >>> stupid... >>> >>> * click on my MichaelAtOz label, there is a link to email me. >>> >>> Unless specifically shown otherwise above, my contribution is in the >>> Public Domain; to the extent possible under law, I have waived all >>> copyright and related or neighbouring rights to this work. Obviously >>> inclusion of works of previous authors is not included in the above. >>> >>> The TPP is no simple “trade agreement.” Fight it! >>> http://www.ourfairdeal.org/ time is running out! >>> -- >>> Sent from: http://forum.openscad.org/ >>> >>> _______________________________________________ >>> OpenSCAD mailing list >>> Discuss@lists.openscad.org >>> http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org >>> >>
MK
Marius Kintel
Fri, Nov 30, 2018 1:48 PM

On Nov 30, 2018, at 6:52 AM, nop head nop.head@gmail.com wrote:

Another issue is false is no longer converted to 0 when used in a vector parameter to translate.

WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or vec2 of numbers, line 56

The undef is actually false. I thought true and false were converted to 1 , 0 when a number was required, as C.

This is worthy of a github ticket.

-Marius

> On Nov 30, 2018, at 6:52 AM, nop head <nop.head@gmail.com> wrote: > > Another issue is false is no longer converted to 0 when used in a vector parameter to translate. > > WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 or vec2 of numbers, line 56 > > The undef is actually false. I thought true and false were converted to 1 , 0 when a number was required, as C. > This is worthy of a github ticket. -Marius
NH
nop head
Fri, Nov 30, 2018 3:09 PM

Actually it is my fault again. The undef is generated by false * 15.
Passing an actual true or false gives a correct error message assuming true
and false are not valid as numbers.

echo(version());
echo(false * 5, true + 1);
translate([true, false, false * 15]);

ECHO: [2018, 11, 29]

ECHO: undef, undef

WARNING: Unable to convert translate([true, false, undef]) parameter to a
vec3 or vec2 of numbers, line 5

On Fri, 30 Nov 2018 at 13:49, Marius Kintel marius@kintel.net wrote:

On Nov 30, 2018, at 6:52 AM, nop head nop.head@gmail.com wrote:

Another issue is false is no longer converted to 0 when used in a vector

parameter to translate.

WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3

or vec2 of numbers, line 56

The undef is actually false. I thought true and false were converted to

1 , 0 when a number was required, as C.

This is worthy of a github ticket.

-Marius


OpenSCAD mailing list
Discuss@lists.openscad.org
http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org

Actually it is my fault again. The undef is generated by false * 15. Passing an actual true or false gives a correct error message assuming true and false are not valid as numbers. echo(version()); echo(false * 5, true + 1); translate([true, false, false * 15]); ECHO: [2018, 11, 29] ECHO: undef, undef WARNING: Unable to convert translate([true, false, undef]) parameter to a vec3 or vec2 of numbers, line 5 On Fri, 30 Nov 2018 at 13:49, Marius Kintel <marius@kintel.net> wrote: > > > On Nov 30, 2018, at 6:52 AM, nop head <nop.head@gmail.com> wrote: > > > > Another issue is false is no longer converted to 0 when used in a vector > parameter to translate. > > > > WARNING: Unable to convert translate([0, 0, undef]) parameter to a vec3 > or vec2 of numbers, line 56 > > > > The undef is actually false. I thought true and false were converted to > 1 , 0 when a number was required, as C. > > > This is worthy of a github ticket. > > -Marius > > > _______________________________________________ > OpenSCAD mailing list > Discuss@lists.openscad.org > http://lists.openscad.org/mailman/listinfo/discuss_lists.openscad.org >