| 00:10.34 | maths22 | brlcad: fisheye is actually up to date |
| 00:10.55 | maths22 | I also decided to activate crucible on it, in case that happens to be somehow useful |
| 00:11.02 | maths22 | (Crucible is for code review) |
| 00:29.20 | *** join/#brlcad kintel (~kintel@unaffiliated/kintel) | |
| 01:16.14 | starseeker | huh, interesting: http://imageengine.github.io/gaffer/index.htm |
| 01:41.38 | kanzure | 404 |
| 01:41.44 | kanzure | this one works: http://imageengine.github.io/gaffer/index.html |
| 02:37.54 | *** join/#brlcad FreezingCold (~FreezingC@135.0.41.14) | |
| 06:17.02 | brlcad | starseeker: yes, the caller is supposed to catch the failure ... crashing/bombing is literally just an uncaught exception |
| 06:17.07 | brlcad | catch it and all is good |
| 06:18.34 | brlcad | some/most -- nearly all -- of the converters do this, while it's hit or miss in non-converter code, usually just whether the person calling the nmg function knew that exceptions were possible, and that they needed to be caught/handled |
| 06:19.26 | *** join/#brlcad sk2 (~simran@101.56.127.240) | |
| 06:26.54 | brlcad | maths22: iiiinteresting about crucible, I hadn't seen that product |
| 06:27.24 | brlcad | I was just looking into reviewboard and they looked like they're still a bit of a pain in the butt for a post-commit workflow |
| 06:27.40 | brlcad | looks like crucible handles that workflow MUCH better, more like what I was hoping |
| 07:27.31 | *** join/#brlcad deepak (~chatzilla@49.14.115.21) | |
| 09:12.52 | *** join/#brlcad teepee- (bc5c2133@gateway/web/freenode/ip.188.92.33.51) | |
| 09:46.43 | *** join/#brlcad ries (~ries@D979EA84.cm-3-2d.dynamic.ziggo.nl) | |
| 10:15.21 | *** join/#brlcad teepee- (bc5c2133@gateway/web/freenode/ip.188.92.33.51) | |
| 10:20.42 | *** join/#brlcad luca79 (~luca@host158-107-dynamic.15-87-r.retail.telecomitalia.it) | |
| 10:31.52 | *** join/#brlcad mihaineacsu (~textual@92.85.194.225) | |
| 11:19.46 | *** join/#brlcad ries (~ries@D979EA84.cm-3-2d.dynamic.ziggo.nl) | |
| 12:54.53 | *** join/#brlcad kesha (~kesha@14.139.122.114) | |
| 13:04.05 | *** join/#brlcad teepee- (bc5c2133@gateway/web/freenode/ip.188.92.33.51) | |
| 13:15.25 | *** join/#brlcad kesha (~kesha@14.139.122.114) | |
| 13:48.37 | *** join/#brlcad luca79 (~luca@net-93-71-169-149.cust.vodafonedsl.it) | |
| 14:11.46 | Notify | 03BRL-CAD:starseeker * 63461 brlcad/trunk/misc/CMake/FindTCL.cmake: Apply patch #285 from Mihai Neacsu quoting path names in EXEC_PROGRAM arguments. Suggests we need to do the odd pathname test using system installed libs to properly exercise the Find logic... |
| 14:33.10 | *** join/#brlcad kesha_ (~kesha@14.139.122.114) | |
| 14:37.37 | *** join/#brlcad clock (~clock@212.203.58.127) | |
| 14:46.53 | *** join/#brlcad kintel (~kintel@unaffiliated/kintel) | |
| 14:54.40 | Notify | 03BRL-CAD:starseeker * 63462 (brlcad/trunk/BUGS brlcad/trunk/NEWS and 2 others): Apply patch #314 from Stefan Mirea fixing a bug in nirt's handling of commands with extra whitespace before a semicolon. Also added a regression test. |
| 14:55.30 | maths22 | ``Erik: Do you know of a good reference for how to write a rc.d script? |
| 14:56.39 | ``Erik | maths22: for which os? if fbsd, check out /usr/local/etc/rc.d/ |
| 14:58.49 | maths22 | thanks |
| 15:22.26 | Notify | 03BRL-CAD:starseeker * 63463 (brlcad/trunk/BUGS brlcad/trunk/NEWS brlcad/trunk/src/libged/draw.c): Apply patch #313 from Stefan Mirea fixing a crash when running the draw command without arguments and bogus error messages when options such as -C supply their error messages with a space. |
| 15:29.46 | starseeker | brlcad, ``Erik: I could use a second opinion on https://sourceforge.net/p/brlcad/patches/312/ |
| 15:37.02 | brlcad | looking |
| 15:38.34 | brlcad | starseeker: from the description, it looks good to me ... we talked about there here or on the mailing list about removing the global |
| 15:39.08 | brlcad | starseeker: it sounds like at least two of those patches applied without change? |
| 15:39.29 | brlcad | I know stefan had three or four pending |
| 15:41.41 | Notify | 03BRL-CAD:carlmoore * 63464 (brlcad/trunk/src/conv/ply/g-ply.c brlcad/trunk/src/conv/ply/ply-g.c): remove trailing blanks/tabs; do a rewording which includes a spelling fix |
| 15:42.35 | starseeker | yeah, stefan's have looked awesome so far |
| 15:43.45 | starseeker | man, quite an annoying day for sf to develop connectivity issues... |
| 15:45.31 | starseeker | brlcad: my only concern with 312 is to wonder if the nirt tie option should, in the process of deprecation, locally set the environment variable if the option is set. 312 looks like it just removes the option |
| 15:46.00 | brlcad | I don't believe the nirt option was documented |
| 15:46.13 | starseeker | there's a man page patch... |
| 15:46.31 | brlcad | oh, hrm |
| 15:46.52 | brlcad | wonder if that made it into a release |
| 15:46.54 | starseeker | would just figure if I added it to the nirt man page docs... |
| 15:46.56 | starseeker | checks |
| 15:47.41 | brlcad | I think it's minimally impacting in the sense that you can just remove that arg from a usage string and prepend with the env var |
| 15:48.04 | starseeker | doesn't look like it was in 7.24.0... |
| 15:49.09 | Notify | 03BRL-CAD:starseeker * 63465 brlcad/trunk/src/other/tk/CMakeLists.txt: Some indications that freetype may need png, so go ahead and add it to tk's link list. |
| 15:49.57 | starseeker | finally |
| 15:50.03 | starseeker | glowers at sf |
| 15:50.45 | starseeker | ah hah - carl added it in 60633 |
| 15:53.56 | starseeker | ok, phew - doesn't look like it's in the tag |
| 15:54.01 | starseeker | (for 7-24-2) |
| 15:54.41 | brlcad | I think that's what prompted the discussion actually |
| 15:54.54 | starseeker | heh |
| 15:55.06 | brlcad | saw carl's commit and could just see the feature creep growing |
| 15:55.09 | starseeker | ok, so I'll just scrub it out of the man page |
| 15:55.24 | brlcad | for a feature that shouldn't even exist separate |
| 15:55.28 | brlcad | cool |
| 15:55.31 | starseeker | but yeah, I think at least two of stefan's commits have gone in clean |
| 15:55.53 | starseeker | (modulo adding NEWS lines for him) |
| 15:55.53 | brlcad | great, I'll ask if it's interested in commit |
| 15:56.15 | starseeker | is looking forward to the IGES patch - first improvement there in a looong time |
| 16:04.15 | Notify | 03BRL-CAD:starseeker * 63466 (brlcad/trunk/BUGS brlcad/trunk/NEWS and 13 others): Apply patch #312 from Stefan Mirea removing the rt_bot_mintie global variable in favor of the environmental variable LIBRT_BOT_MINTIE. |
| 16:11.11 | Notify | 03BRL-CAD:starseeker * 63467 (brlcad/trunk/BUGS brlcad/trunk/NEWS brlcad/trunk/src/conv/iges/iges.c): Apply patch #310 from Stefan Mirea fixing a crash in iges-g and correcting problems with its output. |
| 16:11.44 | *** join/#brlcad sofat (~sofat@202.164.45.204) | |
| 16:12.02 | sofat | hello maths22 |
| 16:14.31 | brlcad | starseeker: shouldn't PNG_LIBRARY be in FREETYPE_LIBRARIES? |
| 16:15.24 | brlcad | starseeker: the iges patch fixed a bug that was introduced about 2 years ago |
| 16:15.25 | brlcad | basically before the last big release, just went undetected because we have no iges regression |
| 16:15.30 | sofat | brlcad, hello |
| 16:15.48 | sofat | please guide me for next step in website |
| 16:16.53 | brlcad | sofat: did you see my recent posting about "guiding"? |
| 16:17.12 | sofat | no |
| 16:17.19 | sofat | where is |
| 16:17.38 | brlcad | guiding is compelely open-ended and far too many possibilities to be productive |
| 16:17.46 | brlcad | ask me something specific :) |
| 16:17.51 | brlcad | or take initiative |
| 16:18.18 | brlcad | there's a lot going on right now and we need the new website in place before the end of the month, however that happens |
| 16:27.22 | maths22 | brlcad: fisheye is now runing as a service |
| 16:27.32 | maths22 | service fecru start/stop |
| 16:27.40 | maths22 | also now runs as fecru user |
| 16:30.41 | *** part/#brlcad fenn (~fenn@131.252.130.248) | |
| 16:31.16 | starseeker | brlcad: in principle it should, but I don't know if the FindFreetype stuff guarantees it |
| 16:31.55 | starseeker | that whole Tcl/Tk CMake business needs a revisit, but I'm sort of hoping I can ignore it until I can sneak JimTcl and Qt in instead... |
| 16:32.47 | *** join/#brlcad teepee- (bc5c2133@gateway/web/freenode/ip.188.92.33.51) | |
| 16:36.38 | *** join/#brlcad kesha (~kesha@14.139.122.114) | |
| 18:16.36 | brlcad | maths22: awesome, that should seriously help |
| 18:17.15 | brlcad | starseeker: *nod* |
| 18:24.28 | starseeker | brlcad: I'm trying to focus on the code patches, since those are the ones that would run aground of a style switch, but your name is on a lot of them - are there any I should steer clear of? |
| 18:25.51 | starseeker | is guessing most of the simulate patches are pretty much totally out of date now... |
| 18:30.05 | starseeker | brlcad: what do we want to do about patch 28? That's already an oldie... |
| 18:31.27 | *** join/#brlcad kintel (~kintel@unaffiliated/kintel) | |
| 18:35.52 | brlcad | none stick out, just use your best judgement -- if they have any value and are simply out of date, I'd keep them open (we can have a GCI task to update them) |
| 18:36.37 | brlcad | tried the nmg patch a while back and it simply had too many issues (cyclic dependency being one of the biggest) |
| 18:36.48 | brlcad | and then there's the branch |
| 18:37.47 | brlcad | patch 28 is definitely a keeper, it's issue is mainly timing, being a major change and lacking an intermediate step |
| 18:37.58 | brlcad | was waiting for a major release rev |
| 18:39.16 | *** join/#brlcad kesha (~kesha@14.139.122.114) | |
| 18:40.54 | brlcad | curious what your comment(s) are on #88 (nmg) |
| 18:45.01 | Notify | 03BRL-CAD:brlcad * 63468 brlcad/trunk/NEWS: Applied sf patch #310 from Stefan Mirea fixing a crash in iges-g and correcting problems with its output. (rewording because they're importers, not exporters) |
| 19:07.46 | *** join/#brlcad Izakey (~Isaac@41.205.22.13) | |
| 19:56.27 | *** join/#brlcad deepak (~chatzilla@117.207.155.254) | |
| 20:02.48 | starseeker | brlcad: between the branch (which is far more extensive and even it has issues) and the passage of time, I figured the original (#88) was just out of date |
| 20:04.56 | starseeker | it may very well be that it will be a major release before we can properly untangle all of that - given the attempts so far, my thought would be to have people survey our new nurbs/brep code and the older nurbs code to see if there is anything that needs to be moved from the old to the new |
| 20:06.20 | starseeker | Not sure if that's a GCI-able setup or not (pick one feature/function in the nmg nurbs code or the old librt primitive, and identify it's corresponding functionality in opennurbs or libbrep) |
| 20:08.09 | starseeker | if the terminology doesn't match up fairly well it may be a real trick to sort that out |
| 20:11.18 | starseeker | brlcad: has this got the pointer foo right? https://sourceforge.net/p/brlcad/patches/308/ |
| 20:15.14 | starseeker | blinks - a material object patch? |
| 20:17.49 | starseeker | brlcad: is there a backstory on this? I know we've discussed design ideas for this over the years, are they reflected here? https://sourceforge.net/p/brlcad/patches/302/ |
| 20:22.38 | *** join/#brlcad mihaineacsu (~textual@92.85.194.225) | |
| 20:25.05 | brlcad | the guy that just joined and left is behind 308, he was our socis student working on materials |
| 20:26.39 | brlcad | #88 actually wasn't really out of date ... it does the job at least did when I last tested it .. just it introduced a cyclic dependency so it wasn't ideal |
| 20:26.58 | brlcad | otherwise, it is/was progress towards separation (and in direct conflict with the branch work, which has its own issues) |
| 20:29.10 | brlcad | not seeing the connection with the new nurbs/brep code though as 88 was about nmg not bspline |
| 20:30.56 | starseeker | apparently the nmg code has some nurbs related stuff in it |
| 20:31.09 | starseeker | based on the breakout needed for the GSoC project |
| 20:32.08 | starseeker | I recall trying the #88 patch and having some trouble with it (maybe on Windows?) |
| 20:34.20 | starseeker | brlcad: I guess it boils down to this - is there a chance we'll use #88 instead of building off the gsoc nmg work? If so, then the patch isn't out of date |
| 20:35.46 | starseeker | the materials patch doesn't look bad - I guess my though there is to hold that until we've actually shipped 7.26.0 |
| 20:36.35 | starseeker | maybe the thing to do with a couple of these that we want to hang on to is to apply them to an old style tree, reformat, and create new patches |
| 20:38.09 | starseeker | 300 looks like Daniel's baliwick |
| 20:41.18 | brlcad | yeah, all the bspline stuff is intertwined with nmg and should die |
| 20:41.24 | brlcad | ON_Brep rules that roost |
| 20:41.53 | brlcad | probably can kill off all of bspline, but that is worth an hour to audit the old code for algorithms |
| 20:42.17 | starseeker | nods - that's were I was wondering if the students might be able to dig, but I suppose it will take an expert eye |
| 20:42.30 | brlcad | "is there a chance we'll use #88 instead of building off the gsoc nmg work?" <--- don't have an answer |
| 20:42.35 | brlcad | because the branch is still broken |
| 20:42.43 | brlcad | if the branch worked, then no chance |
| 20:42.49 | brlcad | with the branch broken, slim chance |
| 20:43.25 | starseeker | OK, I'll put it back to open then. If there's a way to flag these, I'll start flagging them as "need to reformat" so we can still proceed with the style update |
| 20:43.38 | brlcad | maybe leave a comment for the next person reading it |
| 20:44.07 | brlcad | can you see if there are any "must have fix" patches remaining that you're going to merge or want me to look at? |
| 20:44.38 | brlcad | if there aren't, I think we should snap to RELEASE and I can work on stabilizing, and trunk can proceed with more chaos |
| 20:46.31 | starseeker | I'll do a quick scan |
| 20:46.44 | starseeker | mmm chaos |
| 20:47.26 | starseeker | can you give me your C-pointer-guru take on 308? |
| 20:47.37 | starseeker | https://sourceforge.net/p/brlcad/patches/308/ |
| 20:50.25 | brlcad | huh, at a glance, it looks like it still goes a byte too far |
| 20:54.31 | brlcad | checks out a copy to test |
| 20:59.11 | starseeker | we should probably get in some combination of 275 and 284 - I'm not sure about 281 |
| 20:59.33 | starseeker | I think those are the only ones that jump out at me for "this release" attention |
| 21:22.06 | starseeker | brlcad: I'm thinking patch 275 (vs. 284) but a second opinion would be appropriate |
| 21:26.37 | starseeker | brlcad: didn't the patch from 310 correct iges output from .g, not the other way around? |
| 21:29.06 | Notify | 03BRL-CAD:starseeker * 63469 (brlcad/branches/qtged/BUGS brlcad/branches/qtged/CMakeLists.txt and 61 others): Sync with trunk to r63468 |
| 21:32.30 | starseeker | guesses we'll have to preserve the old bspline i/o routines in dbupgrade somewhere |
| 22:58.58 | brlcad | that would probably belong on the rel8 branch |
| 22:59.17 | brlcad | have you tried syncing it lately? I tried a couple weeks ago and ran into problems |
| 22:59.57 | brlcad | seemed like the merge tracking was incomplete (probabaly hasn't merged since the upgrade) |
| 23:26.25 | Notify | 03BRL-CAD:starseeker * 63470 (brlcad/branches/qtged/src/qbrlcad/CMakeLists.txt brlcad/branches/qtged/src/qbrlcad/cadattributes.cxx and 2 others): Add the specific accordian structure that will be the primary interactive panel on the right side |
| 23:32.37 | starseeker | brlcad: yeah, something's up - I had a problem with the xor removal, iirc |
| 23:33.11 | starseeker | may be simplier to pull the patches and re-create it, at this point |
| 23:44.48 | brlcad | good point |