hi,

William F Pokorny <ano### [at] anonymousorg> wrote:

(that statistical summary table does look properly "intimidating :-))


> A few minor comments after my first use - version 202106.2.
>
> 1)----------
> You can get rid of the high ambient warning by changing:
>
> #declare tabu__data_finishes_ = array [2] {
>    finish {ambient .1 diffuse .8 specular .1},
>    finish {ambient 0 emission .85}
> };
>
> to
>
> #declare tabu__data_finishes_ = array [2] {
>    finish {ambient .1 emission 0 diffuse .8 specular .1},
>    finish {ambient 0 emission .85}
> };

thanks, will do.


> 2)----------
> Documentation. (Great by the way)

(thank you.  always the most difficult part of a project, I think)


> Where you have references to your google drive for your Foreach() and
> Ruled() variation at the bottom of the documentation, it would be handy
> to have a reference/link to the most current Tabulated() too.
>
> A nit, but, indicating the Tabulated() version to which the
> documentation pertains in the html would be helpful.

yes, thanks, will make note of the version in text.  will consider the reference
link point; am thinking a wiki page with latest version of each include I've
done?


> 3)----------
>
> Not unexpectedly ;-) ..., the macro uses all lower case identifiers, but
> well prefixed ones. For the povr branch, this does force a configuration
> and compile without the identifier case checking to make use of it.

heh, mine was already compiled to allow lowercase.  :-)


> Great work jr & Tabulated() development helpers!

both BE + TdeG deserve those cheers.


regards, jr.

Post a reply to this message