00:54.31 |
*** join/#brlcad teepee_
(~teepee@unaffiliated/teepee) |
01:32.02 |
*** join/#brlcad KimK
(~Kim__@2600:8803:7a85:6d00:31c9:b100:c4bc:40cd) |
02:08.51 |
brlcad |
starseeker: the current attribute system is
defined as a string=>string map, so introspection into the value
is only arguably defined for the standard attributes (for which
this tolerance setting isn't necessary, doesn't make any
sense) |
02:11.50 |
brlcad |
I would think it perfectly safe, perhaps even
desirable, to not have -t have any bearing on attributes, to not
introspect attributes -- just compare and report when they
differ |
02:13.45 |
brlcad |
it's not in mm because of asc -- it's because
that is (intentionally) defined as the base unit for all storage
and in-memory processing, that requested units merely affects
presentation |
02:15.19 |
brlcad |
this is done for a variety of reasons, one
example being that you wouldn't want to re-encode all values just
by running setting different working units. that would be a source
of error accumulation and be nearly impossible to manage
code-wise |
02:15.24 |
brlcad |
there are other reasons. |
02:16.43 |
brlcad |
so for clarity, I think your last point is
spot on -- that we shouldn't do numerical difference filtering on
attributes (at least until a schema defines the value as a floating
point number, e.g., binary attributes) |
02:30.01 |
starseeker |
brlcad: OK, I'll have to check what we do
there - maybe need to do some work besides updating docs, not
sure |
02:35.03 |
starseeker |
yeah, structurally everything boils down to
avs diffing, and I don't distinguish at the lower level between
params and attrs |
02:35.49 |
starseeker |
even for primitives it's a problem, if some of
them have numerical parameters that aren't dimensions |
02:36.47 |
starseeker |
almost begs for a per-primitive functab
function that can intelligently diff two objects of the same
type |
02:38.04 |
starseeker |
urk |
02:38.44 |
starseeker |
doing this right will take more time than I
have available, certainly before the next point release or two -
should we just disable the tolerance feature until we can do it
right? |
02:39.19 |
starseeker |
winces a little at yanking it
immediately after announcing it, but it might be worse to leave it
in a confusing state... |
02:43.07 |
Notify |
03BRL-CAD:starseeker * 68812
brlcad/trunk/TODO: Have man page for brlman now |
02:45.17 |
starseeker |
yeah, I'm remembering now - doing "smart"
primitive diffing was more than I had time to do last time as well,
so I added the numerical filter as a way to at least be able to
cull really small differences out of the results, however
crudely |
02:45.41 |
starseeker |
the unintended consequences of that might not
be worth it though |
03:12.06 |
starseeker |
blinks - I'll be darned, an
MIT licensed cross platform text editor with both graphical and
terminal modes that's written in C and lua: http://foicica.com/textadept/index.html |
04:17.50 |
brlcad |
starseeker: okay, that all makes
sense |
04:21.14 |
brlcad |
I think the only issue is one of how to
explain it in the manual page, and what that means for attrs if
anything at all |
04:22.52 |
brlcad |
I think saying it's unitless will be
unexpected/confusing, inconsistent, and potentially conflicting
down the road ; saying it's "mm" is fully inadequate and wrong for
some fields |
04:25.18 |
brlcad |
the simple fix might be to just state what the
tolerance is intended to describe (for comparing floating point
geometry parameters) and caveating that it currently also applies
to all other numeric fields as well (which should be fine because
integral change is usually far greater than tol) |
04:26.36 |
brlcad |
so they'll have to be really cautious when
working with tol > 1 |
04:27.16 |
brlcad |
I can take a stab at it later and you can let
me know how it reads |
04:27.34 |
brlcad |
textadept looks really cool |
04:28.57 |
brlcad |
also, fyi, I'd rather see a json-style
callback that would let us universally diff any two
objects |
04:29.20 |
brlcad |
there's lots of good examples of json diffing
that could be modified for use here |
04:40.56 |
*** join/#brlcad boj
(~boj@2001:250:3c00:2074:b1e8:ee29:561b:e3e2) |
06:58.49 |
*** join/#brlcad ickby_
(~stefan@x5d847781.dyn.telefonica.de) |
07:22.30 |
*** join/#brlcad ickby
(~stefan@x5d847781.dyn.telefonica.de) |
07:31.15 |
*** join/#brlcad ickby
(~stefan@x5d847781.dyn.telefonica.de) |
08:05.27 |
*** join/#brlcad Caterpillar
(~caterpill@unaffiliated/caterpillar) |
09:53.07 |
*** join/#brlcad ickby_
(~stefan@x5d847781.dyn.telefonica.de) |
09:53.53 |
*** join/#brlcad ries
(~ries@D979C7EF.cm-3-2d.dynamic.ziggo.nl) |
09:56.32 |
*** join/#brlcad ries
(~ries@D979C7EF.cm-3-2d.dynamic.ziggo.nl) |
10:55.07 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.217.123) |
11:10.44 |
*** join/#brlcad ries
(~ries@D979C7EF.cm-3-2d.dynamic.ziggo.nl) |
11:13.05 |
*** join/#brlcad ries
(~ries@D979C7EF.cm-3-2d.dynamic.ziggo.nl) |
11:17.47 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.217.123) |
11:31.05 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.217.123) |
11:48.36 |
*** join/#brlcad kintel
(~kintel@unaffiliated/kintel) |
11:55.24 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.217.123) |
12:40.26 |
*** join/#brlcad teepee_
(~teepee@unaffiliated/teepee) |
12:46.46 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.244.182) |
13:17.03 |
*** join/#brlcad ChanServ
(ChanServ@services.) |
13:17.03 |
*** mode/#brlcad [+o ChanServ]
by tepper.freenode.net |
13:27.06 |
*** join/#brlcad kintel
(~kintel@unaffiliated/kintel) |
13:41.19 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.244.182) |
13:57.38 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.244.182) |
14:19.04 |
*** join/#brlcad ickby_
(~stefan@x5d847442.dyn.telefonica.de) |
14:36.52 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.216.193) |
14:42.59 |
*** join/#brlcad kintel
(~kintel@unaffiliated/kintel) |
14:47.02 |
*** join/#brlcad merzo
(~merzo@195.128.182.31) |
14:52.33 |
*** join/#brlcad merzo
(~merzo@195.128.182.31) |
15:03.35 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.243.86) |
15:41.16 |
*** join/#brlcad ickby_
(~stefan@x5d847442.dyn.telefonica.de) |
15:44.10 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.243.86) |
16:02.24 |
*** join/#brlcad kintel
(~kintel@unaffiliated/kintel) |
16:20.53 |
*** join/#brlcad merzo
(~merzo@195.128.182.31) |
17:02.57 |
*** join/#brlcad m5w
(~matthew@108-203-5-240.lightspeed.rlghnc.sbcglobal.net) |
17:05.24 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.243.86) |
17:10.32 |
*** join/#brlcad kintel
(~kintel@unaffiliated/kintel) |
17:27.55 |
*** join/#brlcad m5w
(~matthew@108-203-5-240.lightspeed.rlghnc.sbcglobal.net) |
17:52.44 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.243.86) |
18:28.45 |
*** join/#brlcad m5w
(~matthew@108-203-5-240.lightspeed.rlghnc.sbcglobal.net) |
18:37.13 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.230.106) |
19:00.38 |
*** join/#brlcad amarjeet
(~Amarjeet@101.211.230.106) |
19:21.06 |
*** join/#brlcad m5w
(~matthew@108-203-5-240.lightspeed.rlghnc.sbcglobal.net) |
19:44.15 |
*** join/#brlcad shubham
(6719e766@gateway/web/freenode/ip.103.25.231.102) |
20:10.43 |
*** join/#brlcad KimK
(~Kim__@2600:8803:7a85:6d00:31c9:b100:c4bc:40cd) |
23:19.46 |
*** join/#brlcad teepee_
(~teepee@unaffiliated/teepee) |