On Tue, Oct 14, 2014 at 7:23 AM, Christopher Sean Morrison <brl...@mac.com> wrote: > >> What would be the best approach? > > I’d suggest either #1 or #2: > > #1 Move rt_bot_mintie into the rtip structure. With that working and the > global eliminated, you then just have to ensure the value is set when an rtip > is created/initialized (probably within rt_new_rti() which rt_dirbuild() > calls). > > #2 Eliminate rt_bot_mintie global altogether. Make rt_bot_prep() read the > environment variable on-demand (which is called during rt_dirbuild()). Any > user tools that expose a bot_mintie option (e.g., nirt, rt, and archer) will > have to either have that removed (user must use env var) or they’ll have to > get updated to putenv/setenv instead of setting a global. > > Either would be acceptable and #1 is easier, but #2 is also appealing because > it eliminates an unnecessary option (we should automatically be using > whichever is faster or optimize TIE prep so we can eliminate the old method > wholesale).
I'd like to try the second one, but I'm not sure if I fully understand what you meant. Please correct me if I'm wrong: you propose two options: 1) to completely remove any option regarding rt_bot_mintie from the interface, such as File -> Preferences... -> rt_bot_mintie in Archer or the -T flag and the bot_mintie command in nirt. Thereby this parameter will only be available to advanced users through the LIBRT_BOT_MINTIE environment variable. 2) to let the interface as is and only get rid of the rt_bot_mintie global variable and use LIBRT_BOT_MINTIE instead (setting it via putenv/setenv). In both cases, you suggest that any of those applications use the fastest of the two methods (basically XGLUE(rt_bot_prep_, TRI_TYPE)() versus bottie_prep_double()) by default. I don't really know how to automatically check whick one would be faster :( Also, the variable significance is a little unclear to me: alongside its declaration in librt/globals.c it is said to be the "minimum number of bot for TIE". In nirt/nirt.c it is said that "0 means do not use pieces, default is 4294967295". But, as far as I can see in rt_bot_prep(), it looks to be the minimum number of faces that a BoT should have in order to be precomputed with bottie_prep_double(). What does it mean more exactly? Thanks for your help, Stefan ------------------------------------------------------------------------------ Comprehensive Server Monitoring with Site24x7. Monitor 10 servers for $9/Month. Get alerted through email, SMS, voice calls or mobile push notifications. Take corrective actions from your mobile device. http://p.sf.net/sfu/Zoho _______________________________________________ BRL-CAD Developer mailing list brlcad-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/brlcad-devel
