IRC log for #brlcad on 20140103

00:02.19 *** join/#brlcad merzo (~merzo@71-40-132-95.pool.ukrtel.net)
00:14.49 Notify 03GCI:starseeker * 6441264325066752 Create unit tests for our basic utility library (LIBBU) #3: Task Assigned - This task has been assigned to Johannes Schulte. You have 72 hours to complete this...
00:17.19 Notify 03GCI:starseeker * 5865810849431552 Write draft of BRL-CAD application interface design guidelines: It's fairly free form - You get to propose a set of design criteria - the links are just to give...
00:33.36 Notify 03BRL-CAD:j-schulte * 59264 brlcad/trunk/src/libbu/tests/CMakeLists.txt: Add unit test for date-time.c from libbu (GCI task: http://www.google-melange.com/gci/task/view/google/gci2013/6441264325066752)
00:35.16 Notify 03GCI:Johannes Schulte * 6441264325066752 Create unit tests for our basic utility library (LIBBU) #3: Ready for review - The work on this task is ready to be reviewed.
00:36.22 Notify 03GCI:Johannes Schulte * 6441264325066752 Create unit tests for our basic utility library (LIBBU) #3 http://www.google-melange.com/gci/task/view/google/gci2013/6441264325066752: Questions When...
00:38.08 Notify 03GCI:Sean * 5873716877590528 Create a precise modeling transcript for modeling a simple toy car: Task Assigned - This task has been assigned to Jacob Fine. You have 72 hours to complete this task,...
00:38.13 Notify 03GCI:Sean * 5332162035318784 Write a wiki page on joining the BRL-CAD project: Task Assigned - This task has been assigned to Michael. You have 72 hours to complete this task, good luck!
01:55.15 Notify 03GCI:Michael * 5332162035318784 Write a wiki page on joining the BRL-CAD project: Ready for review - The work on this task is ready to be reviewed.
01:59.57 Notify 03GCI:fernozzle * 5034143381454848 Compile BRL-CAD on Windows using Cygwin: Task Claimed - I would like to work on this task.
02:08.48 brlcad maths22: http://www.google-melange.com/gci/task/view/google/gci2013/5780674933948416
02:08.48 gcibot Update Fisheye installation
02:08.49 gcibot Status: Open
02:08.49 gcibot Mentor(s): Sean
02:16.32 Notify 03GCI:Sean * 5300578120892416 Implement new API function to reduce duplication #3: duplication - Number of lines is irrelevant. Any code duplication warrants refactoring. What's not clear to me...
02:17.46 maths22 brlcad: thanks
02:19.04 maths22 rx
02:19.12 maths22 oops :)
02:19.17 Notify 03GCI:Sean * 5300578120892416 Implement new API function to reduce duplication #3 http://www.google-melange.com/gci/task/view/google/gci2013/5300578120892416: BoT task Note that there is a BoT...
02:23.17 Notify 03GCI:Jacob B * 5315017800941568 Enable a new compiler warning, fix any issues that ensue #3: Ready for review - The work on this task is ready to be reviewed.
02:28.48 Notify 03GCI:Sean * 5300578120892416 Implement new API function to reduce duplication #3 http://www.google-melange.com/gci/task/view/google/gci2013/5300578120892416: bot surface area There's not a BoT...
02:45.27 Notify 03GCI:Sean * 5300578120892416 Implement new API function to reduce duplication #3 http://www.google-melange.com/gci/task/view/google/gci2013/5300578120892416: follow-on A task for BoT mesh...
03:12.11 Notify 03GCI:Sean * 5849146376323072 Model pliers in 3D using BRL-CAD: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to this task to...
03:12.56 Notify 03GCI:Sean * 5849146376323072 Model pliers in 3D using BRL-CAD: overlaps - Looks fantastic, but there are 0.25mm overlaps (the handles iirc).
03:28.42 Notify 03GCI:Sean * 6441264325066752 Create unit tests for our basic utility library (LIBBU) #3: few issues - Few issues: Missing header and footer The #include "stdio.h" isn't right, system headers are...
03:28.43 Notify 03GCI:Sean * 6441264325066752 Create unit tests for our basic utility library (LIBBU) #3: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s)...
03:29.02 Notify 03GCI:Sean * 5034143381454848 Compile BRL-CAD on Windows using Cygwin: Task Assigned - This task has been assigned to fernozzle. You have 72 hours to complete this task, good luck!
03:31.52 Notify 03GCI:Sean * 5332162035318784 Write a wiki page on joining the BRL-CAD project: Missing a lot - Michael, did you check out all of the examples? Particularly the Gnome and Ubuntu examples are really...
03:31.53 Notify 03GCI:Sean * 5332162035318784 Write a wiki page on joining the BRL-CAD project: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to...
03:33.02 Notify 03GCI:Sean * 5660495776317440 Create prototype 2D Drawing: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to this task to satisfy the...
03:36.34 Notify 03GCI:Sean * 5660495776317440 Create prototype 2D Drawing http://www.google-melange.com/gci/task/view/google/gci2013/5660495776317440: not the right task Ilkin, what you submitted has nothing to...
04:27.53 Notify 03GCI:Sean * 5315017800941568 Enable a new compiler warning, fix any issues that ensue #3: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s)...
04:29.53 *** join/#brlcad microchip_sac (~microchip@223.239.198.244)
04:43.49 maths22 brlcad: what needs work? I'm sure something does, but I'm not sure what
04:51.18 brlcad maths22: took me a while to write the reply
04:51.19 Notify 03GCI:Sean * 5315017800941568 Enable a new compiler warning, fix any issues that ensue #3: almost good - Wow, this was a really tricky patch to review... nice work doing your homework, though. It...
04:51.20 brlcad that was a tough one
04:52.11 brlcad I had to pull out documentation on a number of points to make sure things were right/wrong
05:31.57 *** join/#brlcad microchip_sac (~microchip@223.239.198.244)
05:38.16 microchip_sac I'm working on http://www.google-melange.com/gci/task/view/google/gci2013/5240158668455936
05:38.18 gcibot Enable a new compiler warning, fix any issues that ensue #5
05:38.18 gcibot Status: NeedsWork (4 days, 6 hours left)
05:38.18 gcibot Mentor(s): Kesha Shah
05:38.50 microchip_sac and I've fixed the warnings that occur *without* the new flag because of my changes
05:39.11 microchip_sac other than those, is the patch OK?
05:39.23 microchip_sac should I submit a new build log and patch for review now?
05:48.28 brlcad microchip_sac: that is what the last comment said, no?
05:48.43 brlcad just make sure to submit a "make -k" with the flag enabled
05:48.58 brlcad need to get an idea of how many issues remain
05:49.06 brlcad the previous logs looked like they were not -k
06:32.18 *** join/#brlcad kesha (~kesha@14.139.122.114)
06:52.54 Notify 03GCI:Sharan Narayan * 5849146376323072 Model pliers in 3D using BRL-CAD: Ready for review - The work on this task is ready to be reviewed.
06:56.31 *** join/#brlcad microchip_sac (~microchip@223.239.198.244)
06:58.54 Notify 03GCI:Sean * 5849146376323072 Model pliers in 3D using BRL-CAD: Task Closed - Congratulations, this task has been completed successfully.
07:03.20 Notify 03GCI:Sean * 4773374307860480 Fix MGED crash on Mac OS X: more info - Yash, if you followed the steps correctly and cannot reproduce the crash, you can claim the task, submit details on what you...
07:04.55 Notify 03GCI:Sean * 5054849284571136 Doxygen cleanup for LIBWDB http://www.google-melange.com/gci/task/view/google/gci2013/5054849284571136: SVN Information on how to obtain BRL-CAD's sources is...
07:07.25 Notify 03GCI:Sean * 4549761130561536 Create an utility library (LIBBU) API unit test for any file: no follow-up - Arjun, have you given up on this? Your last comment made it sound like you'd fixed the...
07:08.35 Notify 03GCI:Sean * 5675661037404160 Create numerics library (LIBBN) unit test for rand.c http://www.google-melange.com/gci/task/view/google/gci2013/5675661037404160: SVN Information on how to obtain...
07:11.00 Notify 03GCI:Sean * 6501205291302912 Fix at least 8 spelling mistakes in at least 8 different files: nobody? - This task is so close to completion, just was submitted in the wrong format. Is nobody really...
07:16.30 *** join/#brlcad caen23 (~caen23@92.81.162.63)
07:56.56 Notify 03GCI:Sharan Narayan * 6012644171448320 Find, reliably reproduce, and report any bug in MGED: Task Claimed - I would like to work on this task.
07:59.46 Notify 03GCI:Sharan Narayan * 6012644171448320 Find, reliably reproduce, and report any bug in MGED: SourceForge - Where to Sumbit, Should I submit my bug report to sourceforge.net?
08:48.46 *** join/#brlcad microchip_sac (~microchip@223.239.198.244)
09:43.39 Notify 03GCI:Alexandra Covor * 5804117939191808 Design a cover image or banner for our Facebook page (and/or another social media site) #15: Ready for review - The work on this task is ready to be...
09:44.19 Notify 03GCI:Ilkin Musaev * 5660495776317440 Create prototype 2D Drawing: Ready for review - The work on this task is ready to be reviewed.
10:59.45 *** join/#brlcad chick_ (~chick_@41.204.83.2)
10:59.59 chick_ #join #ublab
11:06.14 Notify 03BRL-CAD:j-schulte * 59265 brlcad/trunk/src/libbu/tests/bu_date-time.c: fix r59264
11:06.15 Notify 03GCI:AdityaGulati * 6387756011880448 Create a Motion Typography video for BRL-CAD #4: Task Claimed - I would like to work on this task.
11:07.29 Notify 03GCI:Johannes Schulte * 6441264325066752 Create unit tests for our basic utility library (LIBBU) #3: Ready for review - The work on this task is ready to be reviewed.
11:38.48 *** join/#brlcad microchip_sac (~microchip@223.239.198.244)
13:07.22 Notify 03GCI:Vipul Sharma * 5902558589616128 Design a BRL-CAD Sticker #14: Ready for review - The work on this task is ready to be reviewed.
13:12.02 Notify 03GCI:Vipul Sharma * 5902558589616128 Design a BRL-CAD Sticker #14: Claim Removed - The claim on this task has been removed, someone else can claim it now.
13:12.03 Notify 03GCI:Vipul Sharma * 5902558589616128 Design a BRL-CAD Sticker #14: Task Claimed - I would like to work on this task.
13:12.07 *** join/#brlcad KimK (~Kim__@24.255.223.153)
13:55.28 Notify 03GCI:Andromeda Galaxy * 5895393510424576 Improve the superellipsoid surface area implementation: Ready for review - The work on this task is ready to be reviewed.
14:11.45 Notify 03GCI:Sean * 5902558589616128 Design a BRL-CAD Sticker #14: Task Assigned - This task has been assigned to Vipul Sharma. You have 168 hours to complete this task, good luck!
14:12.05 Notify 03GCI:Sean * 6387756011880448 Create a Motion Typography video for BRL-CAD #4: Task Assigned - This task has been assigned to AdityaGulati. You have 72 hours to complete this task, good luck!
14:12.20 Notify 03GCI:Sean * 6012644171448320 Find, reliably reproduce, and report any bug in MGED: Task Assigned - This task has been assigned to Sharan Narayan. You have 72 hours to complete this task, good luck!
14:12.52 maths22 brlcad: thanks
14:13.14 maths22 I don't know if you noticed in the log, but there were two warnings I could not figure out how to fix
14:13.17 maths22 is that ok?
14:13.56 Notify 03GCI:Sean * 5660495776317440 Create prototype 2D Drawing: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to this task to satisfy the...
14:14.49 *** join/#brlcad kesha (~kesha@49.202.231.149)
14:19.00 Notify 03GCI:Sean * 5660495776317440 Create prototype 2D Drawing: much better - This is much better but your prototype looks incomplete. What are all of those little boxes for? That goes into each of the...
14:19.27 Notify 03GCI:Sean * 6441264325066752 Create unit tests for our basic utility library (LIBBU) #3: Task Closed - Congratulations, this task has been completed successfully.
14:21.17 Notify 03GCI:Sean * 5804117939191808 Design a cover image or banner for our Facebook page (and/or another social media site) #15: Task Closed - Congratulations, this task has been completed successfully.
14:25.34 Notify 03GCI:Sean * 5895393510424576 Improve the superellipsoid surface area implementation: Task Closed - Congratulations, this task has been completed successfully.
14:26.58 *** join/#brlcad microchip_sac (~microchip@223.239.198.74)
14:27.56 *** join/#brlcad kesha_ (~kesha@49.249.17.247)
14:29.13 Notify 03GCI:Sharan Narayan * 6012644171448320 Find, reliably reproduce, and report any bug in MGED: Ready for review - The work on this task is ready to be reviewed.
14:34.01 Notify 03GCI:Andromeda Galaxy * 5895393510424576 Improve the superellipsoid surface area implementation: Performance improvement - Sorry, I forgot to post these numbers in the explanation file: 10 runs of...
14:35.46 Notify 03GCI:Andromeda Galaxy * 5895393510424576 Improve the superellipsoid surface area implementation: One more performance improvement - I had one more idea for another performance improvement after I...
14:36.03 *** join/#brlcad andromeda-galaxy (~user@61.3.164.205)
14:36.11 Notify 03GCI:Sean * 5895393510424576 Improve the superellipsoid surface area implementation: some problems - Andromeda, this looks good but you did introduce a new constant (500000) without documenting...
14:37.16 Notify 03GCI:Sean * 6012644171448320 Find, reliably reproduce, and report any bug in MGED: Task Closed - Congratulations, this task has been completed successfully.
14:37.36 andromeda-galaxy brlcad: Can I talk to you for a minute about those two constants that I changed?
14:37.38 brlcad maths22: I commented on those two
14:37.44 brlcad andromeda-galaxy: sure
14:38.34 maths22 you didn't comment on theme here http://www.google-melange.com/gci/task/view/google/gci2013/5315017800941568
14:38.35 gcibot Enable a new compiler warning, fix any issues that ensue #3
14:38.35 gcibot Status: NeedsWork (5 days, 4 hours left)
14:38.35 gcibot Mentor(s): Daniel Rossberg
14:38.49 andromeda-galaxy brlcad: The reason that I said that in the comments is that many different sets of numbers get within 0.001 of the correct values in my tests, and these values gave the maximum performance for similar accuracy
14:39.12 brlcad maths22: it's the second sentence
14:39.45 andromeda-galaxy When the function was significantly sped up, this changed somewhat, as it became less expensive to start with much higher values, and to recompute more often: the faster convergence now outweighs the larger numbers at the beginning (I think)
14:40.40 maths22 not those;
14:40.57 maths22 you didn't see the other errors because I forgot to redirect stderr
14:41.04 brlcad ahh
14:41.53 maths22 will share errors shortly
14:42.35 brlcad andromeda-galaxy: it's not that your comments weren't understandable
14:42.41 brlcad they're just not useful :)
14:43.16 andromeda-galaxy brlcad: I see --- what I was saying here is just why I think that these are the optimum values, even though one of them is an order of magnitude different
14:43.18 brlcad you could have just as well said "these were the values that seemed to work best"
14:43.29 brlcad it's two orders
14:44.17 andromeda-galaxy brlcad: can you suggest a better way to find parameters that maximize performance?
14:44.35 andromeda-galaxy and accuracy?
14:44.37 brlcad that's the wrong question to be asking
14:44.51 andromeda-galaxy I'm not sure how to go about doing it any more than this...
14:45.38 brlcad let me see if I can show you another example
14:46.05 andromeda-galaxy brlcad: okay, thanks
14:46.40 maths22 http://paste.ubuntu.com/6684923/
14:47.24 andromeda-galaxy brlcad: by the way, the other constant that you mentioned (the +500000) was a slight mistake in the diff, it only needs to be +1; I added a comment just now explaining that the +1 is necessary to store both endpoints of the line of the grid that will be stored in the arrays
14:48.30 brlcad andromeda-galaxy: okay, take a peek at src/libbu/heap.c where there are two constants there in #defines at the top of the file -- the comment tells you what the number means, what changes to it imply, and roughly talk about impact (performance)
14:49.07 maths22 brlcad: those are the two errors
14:49.31 andromeda-galaxy brlcad: okay, thanks --- I will try to rewrite my comments
14:49.55 andromeda-galaxy would you mind taking a look at my final ones before commit?
14:50.56 brlcad andromeda-galaxy: think about it like a variable, don't focus on the value itself -- that might help
14:51.27 andromeda-galaxy brlcad: thank you for the help!
14:51.41 brlcad andromeda-galaxy: like instead of *2 or *4, what would you call that? CONVERGENCE_RATE or STEPPING_SIZE or ...
14:51.55 brlcad then without thinking about a value, document it
14:52.05 brlcad what is it, what does it mean, how does it affect anything
14:52.14 brlcad and this is all in just one or two sentences ;)
14:52.41 brlcad it should just be useful/informative, and you probably won't end up talking about performance much at all (which is fine)
14:53.25 brlcad I might be using a computer that is literally 100x faster than yours -- will that make the values you chose bad values? or will it be fine?
14:53.47 brlcad or if I use a computer that's 100 times slower, ditto
14:54.19 andromeda-galaxy brlcad: I think that these will still be fairly good values, but I'm not completely sure how to prove it...
14:54.46 brlcad how do they affect quality?
14:55.34 brlcad if you changed 1024 to 2048, how does that affect the sampling error of a 1m^3 superell?
14:57.22 brlcad not to lose sight of the path through the forest, you could work for hours to understand your own numbers and that's not the goal
14:57.34 brlcad the goal is to just document what you DO know now
14:58.09 brlcad and to say something useful about them, what will be useful to other devs, which is to explain what those numbers are, what they mean, and how sensitive they are
14:59.13 andromeda-galaxy brlcad: I have two more comments that I think are more useful
14:59.20 brlcad k, go for it ;)
14:59.40 andromeda-galaxy Would you mind looking at them quickly before I commit?
14:59.50 brlcad just commit if you're comfortable with it
14:59.57 andromeda-galaxy brlcad: okay, 1 minute
15:00.12 Notify 03GCI:Ilkin Musaev * 5660495776317440 Create prototype 2D Drawing: Ready for review - The work on this task is ready to be reviewed.
15:01.20 brlcad maths22: I see what you mean -- those two are just aweful bits of code
15:01.25 brlcad awful even
15:04.22 brlcad those are string numbers ... being read in and turned into pointers
15:04.35 brlcad the second is even worse, turning the pointer into a function
15:05.17 Notify 03GCI:Sean * 5660495776317440 Create prototype 2D Drawing: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to this task to satisfy the...
15:06.09 maths22 is it ok then if I leave it?
15:08.41 brlcad well if you leave it, the build will be broken :)
15:09.04 Notify 03GCI:Sean * 5660495776317440 Create prototype 2D Drawing: okay to worse? - Ilkin, your design is digressing. What is effectivity? What is "designed data"? Not sure what would go into that column...
15:09.38 *** join/#brlcad kesha__ (~kesha@49.202.231.158)
15:10.19 andromeda-galaxy brlcad: There was a slight conflict, where someone fixed a misspelling in my comment, so I'll commit in a minute; can you take a quick look at it when I do?
15:11.06 andromeda-galaxy Also, did you see my comments about a couple of possible tasks/things to do after GCI? (one in the _explanations file, one on the website a minute ago)
15:14.42 Notify 03BRL-CAD:peter-sa * 59266 brlcad/trunk/src/librt/primitives/superell/superell.c: This adds a significant performance gain to the superell surface area functions, from GCI task http://www.google-melange.com/gci/task/view/google/gci2013/5895393510424576
15:14.43 gcibot Improve the superellipsoid surface area implementation
15:14.43 gcibot Status: Closed
15:14.43 gcibot Mentor(s): Sean
15:14.51 brlcad andromeda-galaxy: all commits are reviewed/watched by myself and others, no need to ask for that ;)
15:15.03 brlcad that just happens automatically all the time, every commit
15:15.42 andromeda-galaxy brlcad: okay, thanks
15:15.58 andromeda-galaxy I wasn't entirely sure, so I thought it was better to ask
15:16.34 andromeda-galaxy (I'm in general more used to projects with a pre-commit review, so I'm not entirely sure about the procedures for post-commit reviewing)
15:17.14 andromeda-galaxy Do you think that my two other ideas are probably best as post-GCI?
15:18.13 brlcad I must have missed what you're referring to in the explanations file -- you talked about how you improved performance and how you tried a fast sine approximation that sucked
15:18.33 andromeda-galaxy I was actually saying that the two fast sine approximations are really good
15:18.40 andromeda-galaxy it's just that this needs them to be really really good
15:18.54 brlcad (which was pretty cool, but yeah we usually can't use them)
15:19.27 andromeda-galaxy So I was wondering if any of the rendering code, etc. could use a large performance gain at the cost of a fairly small accuracy loss...
15:20.02 brlcad most of what we calculate needs to be really accurate where even double precision is insufficient, but we make it work as best we can because the alternatives (fixed precision, interval arithmetic, etc) are ORDERS of magnitude slower
15:20.05 *** join/#brlcad kesha__ (~kesha@49.249.17.238)
15:20.25 andromeda-galaxy brlcad: okay
15:21.28 andromeda-galaxy And I just realized that several of the trig computations in the surf_area computation can be factored out in a way that I think it is fairly unlikely that the compilers automatically do, as it is nested several layers of function calls deep, and it is only a small part of an expression that happens to not be reliant on the innermost loop
15:21.57 andromeda-galaxy which is why I was wondering if you thought it might be reasonable to add one more of these performance improvement ones...
15:23.50 brlcad if it's not going to give an order of magnitude performance improvement, and it sounds like it's not likely, then it's probably "good enough" for now until we have hard quality metrics to improve upon
15:24.41 brlcad it's also not a good idea to guess what the compiler is or is not capable of when it comes to performance
15:24.45 brlcad you'll often be surprised
15:24.52 brlcad you got to profile
15:25.05 andromeda-galaxy brlcad: I did do a profile
15:25.20 andromeda-galaxy nearly all of the time is spent in the sin/cos functions as far as I can tell
15:25.24 brlcad excellent
15:25.50 andromeda-galaxy though the improvement I just submitted makes that a bit better (I think 80-some percent to 60-some)
15:27.49 brlcad so then optimizing would mean either 1) improving the performance of sin/cos (not likely, and you tried that), 2) calling sin/cos less (possible, but requires a lot of work to track the impact), or 3) using a different approach that avoids the problem altogether (navel gazing)
15:28.54 brlcad you could also leverage data coherence and parallelism, though both are sort of advanced topics to do well
15:29.34 andromeda-galaxy brlcad: I was indeed thinking of parallelism; also, I think that the optimization that I was just mentioning would significantly reduce calls to sin/cos
15:33.56 brlcad with most of our work, the emphasis is first on getting it implemented not just "good enough" but provably accurate
15:33.59 maths22 I will try to fix it then
15:34.14 maths22 do you have any suggestions?
15:34.35 maths22 (otherwise I can just stash the value in a local variable and then cast it)
15:34.40 maths22 that should make GCC happy
15:34.41 brlcad since this is already an approximation approach, performance is not really the primary concern -- accuracy is the concern
15:36.05 brlcad it doesn't matter if something takes an hour to compute if it's ultimately wrong or has bugs for some cases or has too much error or an indeterminate amount of error
15:36.26 andromeda-galaxy brlcad: I see
15:36.35 brlcad maths22: stashing may be the way to go for this
15:36.52 brlcad the "fix" is to eliminate that pointer trickery altogether
15:39.45 andromeda-galaxy brlcad: do you have something specific in mind about trying to test the accuracy more, or should I try to find another task for GCI and then talk to you about it more after gci?
15:55.19 Notify 03GCI:Michael * 5332162035318784 Write a wiki page on joining the BRL-CAD project: Ready for review - The work on this task is ready to be reviewed.
15:59.11 Notify 03GCI:Andromeda Galaxy * 5803108621877248 Enable a new compiler warning, fix any issues that ensue: Task Claimed - I would like to work on this task.
15:59.52 *** join/#brlcad sac_microchip (~microchip@223.239.199.15)
16:01.52 Notify 03GCI:Mandeep Kaur * 5803108621877248 Enable a new compiler warning, fix any issues that ensue: Task Assigned - This task has been assigned to Andromeda Galaxy. You have 144 hours to complete this...
16:03.13 maths22 I will do that now
16:08.15 Notify 03GCI:Sean * 5332162035318784 Write a wiki page on joining the BRL-CAD project: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to...
16:10.10 Notify 03GCI:Sean * 5332162035318784 Write a wiki page on joining the BRL-CAD project: you didn't change anything? - Michael, it looks like you didn't change anything. Someone added a contact section, but...
16:12.31 Notify 03GCI:Michael * 5332162035318784 Write a wiki page on joining the BRL-CAD project: ??? - Can you give me an example of structure? I have a lot of the structure that GNOME has.
16:16.09 Notify 03GCI:Melange * 4932878790033408 Fix at least 8 spelling mistakes in at least 8 different files #7: Initial Deadline passed - Melange has detected that the initial deadline has passed and it has set...
16:19.24 maths22 done; rebuilding
16:31.11 Notify 03GCI:Michael * 5332162035318784 Write a wiki page on joining the BRL-CAD project: Ready for review - The work on this task is ready to be reviewed.
16:31.17 Notify 03GCI:Michael * 5332162035318784 Write a wiki page on joining the BRL-CAD project: What else should I do?? - What I am trying to say is that there is no accessibilty page on the BRL-CAD site. That...
16:32.14 *** part/#brlcad sac_microchip (~microchip@223.239.199.15)
16:32.37 *** join/#brlcad microchip_sac (~microchip@223.239.199.15)
16:33.10 microchip_sac I'm doing GCI task http://www.google-melange.com/gci/task/view/google/gci2013/5240158668455936
16:33.12 gcibot Enable a new compiler warning, fix any issues that ensue #5
16:33.12 gcibot Status: NeedsWork (3 days, 19 hours left)
16:33.12 gcibot Mentor(s): Kesha Shah
16:33.43 microchip_sac and I'm getting some warnings without the extra flag, which I'm trying to fix
16:33.51 microchip_sac (as my mentor said in the comments)
16:34.40 microchip_sac I need to fix a warning in src/mged/edsol.c, involving calls to rt_arb_calc_points
16:34.49 microchip_sac brlcad: could you help me?
16:35.01 brlcad ~ask
16:35.02 infobot Questions in the channel should be specific, informative, complete, concise, and on-topic. Don't ask if you can ask a question first. Don't ask if a person is there; just ask what you intended to ask them. Better questions more frequently yield better answers. We are all here voluntarily or against our will.
16:35.11 brlcad "yes"
16:35.13 brlcad :)
16:35.47 maths22 commit 59266 broke build: http://brlcad.org:8085/browse/BRLCAD-BUIL-513/log
16:35.50 microchip_sac brlcad: could you tell me how to deal with the pointers to arrays?
16:37.09 Notify 03GCI:Ilkin Musaev * 5660495776317440 Create prototype 2D Drawing: Ready for review - The work on this task is ready to be reviewed.
16:40.34 maths22 could misc/pastebinit be installed on the server?
16:43.56 Notify 03GCI:Jacob B * 5315017800941568 Enable a new compiler warning, fix any issues that ensue #3: Ready for review - The work on this task is ready to be reviewed.
16:47.59 brlcad maths22: can pastebinit be configured to anything other than pastebin.com? we can't use that one
16:48.24 maths22 yes
16:48.34 maths22 reference coming shortly...
16:48.46 maths22 alos, what's wrong with pastebin.com
16:49.01 microchip_sac brlcad: how do I deal with the pointers to arrays?
16:49.11 maths22 see: http://www.moleberry.com/man/p/pastebinit.1.html
16:50.10 maths22 ping
16:50.19 maths22 oops!
16:50.26 brlcad microchip_sac: you need to give me more information than that
16:50.42 microchip_sac file src/mged/edsol.c
16:51.00 microchip_sac line # 6882, for example
16:51.25 microchip_sac the call to rt_arb_calc_points involves es_peqn
16:51.25 maths22 microchip_sac: what flag are you using?
16:51.28 brlcad maths22: pastebin.com is blacklisted/blocked by many of our corporate contributor environments
16:51.34 maths22 that makes sense
16:51.46 microchip_sac but es_peqn is not of the correct type
16:51.53 brlcad they have too much of a history of hosting malware, cracks, ssns, etc
16:52.05 microchip_sac see declaration of rt_arb_calc_points
16:52.19 microchip_sac maths22: none, just the default
16:52.45 maths22 line 6882 is blank
16:52.57 maths22 Are you using the latest svn checkout?
16:53.09 microchip_sac oops, sorry, I have local edits
16:53.33 microchip_sac just look for rt_arb_calc_points
16:54.02 maths22 that makes sense; can you post the build error to a pastebin (like paste.ubuntu.com)?
16:54.27 Notify 03BRL-CAD:brlcad * 59267 brlcad/trunk/src/librt/primitives/superell/superell.c: index cannot be used as a variable name due to shadowing, use idx instead
16:55.14 microchip_sac maths22: I tried fixing it by declaring a separate array of the right type
16:55.26 microchip_sac and then memcpy-ing the array
16:55.47 microchip_sac it works, but is it correct?
16:55.59 brlcad microchip_sac: what's the problem just declaring it as a pointer to an array?
16:56.07 maths22 comitt 59264 also broke build-see http://brlcad.org:8085/browse/BRLCAD-BUIL-511/log
16:56.24 brlcad is it still broken?
16:56.53 microchip_sac brlcad: rt_arb_calc_points expects (const plane_t *[4])
16:56.54 brlcad running a full build now
16:56.57 maths22 compiling right now ...
16:57.19 brlcad microchip_sac: that doesn't look like what it takes to me
16:57.29 brlcad the function definition is different
16:57.42 microchip_sac it takes more than that, that's the part which gives a warning
16:59.05 microchip_sac I tried fixing it by
16:59.14 microchip_sac <PROTECTED>
16:59.27 microchip_sac <PROTECTED>
16:59.34 brlcad back up
16:59.36 brlcad the declaration must match the definition
16:59.38 microchip_sac <PROTECTED>
16:59.39 brlcad what is the definition?
17:00.30 microchip_sac int rt_arb_calc_points(struct rt_arb_internal *arb, int cgtype, const plane_t planes[6], const struct bn_tol *tol);
17:00.44 microchip_sac that's the same for both
17:00.49 microchip_sac decl and defn
17:01.02 brlcad good
17:01.22 brlcad so then that's what makes you saying "brlcad: rt_arb_calc_points expects (const plane_t *[4])" make no sense. :)
17:01.27 brlcad it does not expect that
17:01.43 microchip_sac gcc says that
17:01.52 brlcad it expects "const plane_t planes[6]" which is very different
17:01.54 microchip_sac it doesn't make any sense!
17:02.18 microchip_sac the '4' was my mistake, should have been '6'
17:02.51 microchip_sac ok, I fixed it by the stuff described above
17:03.01 microchip_sac is it good practices, though?
17:03.13 brlcad problems cannot be debugged/discussed when you have mistakes like that in the discussion :)
17:03.26 brlcad programming requires precision 100% of the time
17:04.14 brlcad "I tried fixing it by ..." and "I fixed it by the stuff described above" don't make sense
17:04.19 brlcad did you fix it or did you not fix it?
17:04.38 microchip_sac I fixed it, it builds without errors
17:04.39 brlcad Yoda says "Do or Do Not, there is no try"
17:04.45 microchip_sac but is it good practices?
17:05.22 brlcad what is the type mismatch?
17:05.29 microchip_sac I mean, there's a lot of stuff on the Web about the uselessness
17:05.40 brlcad it really depends
17:05.48 microchip_sac and complications of 'array pointers'
17:05.54 brlcad you need to understand how it's being used, what's being used
17:06.09 brlcad what is the type mismatch?
17:06.13 microchip_sac OK, here goes a precise description:
17:06.27 Notify 03BRL-CAD:carlmoore * 59268 (brlcad/trunk/include/ged.h brlcad/trunk/src/libged/brep.c): fix spelling and remove trailing blanks/tabs
17:06.35 maths22 still broken: http://brlcad.org:8085/browse/BRLCAD-BUIL-514/log
17:06.49 microchip_sac the original variable: fastf_t es_peqn[7][4];
17:07.23 microchip_sac what rt_arb_calc_points expects: struct rt_arb_internal *arb, int cgtype, const plane_t planes[6], const struct bn_tol *tol
17:07.59 microchip_sac (I believe plane_t is a type of fastf_t, because it works!)
17:08.42 microchip_sac so what I did: const fastf_t es_peqn_arg[7][4];
17:09.05 microchip_sac and then: memcpy(es_peqn_arg, es_peqn, sizeof(es_peqn));
17:09.21 microchip_sac passing es_peqn_arg to rt_arb_calc_points
17:09.51 microchip_sac this is made to work by trial and error
17:10.06 microchip_sac is it *designed* to work as well?
17:12.12 microchip_sac brlcad: precise enough? :)
17:14.32 maths22 jenkins is currently broken but using lots of cpu periodically
17:17.57 brlcad microchip_sac: much more precise, should have started there with your question ;)
17:18.12 brlcad plane_t is an array of fastf_t
17:19.03 brlcad you can see the type declaration in include/vmath.h, but it's a fastf_t[4]
17:20.18 microchip_sac that's probably where my 'oops-6-and-not-4' came from
17:20.52 Notify 03BRL-CAD:brlcad * 59269 brlcad/trunk/src/libbu/tests/bu_date-time.c: some of these constants are too big, make sure they instatiate as the correct type, using long long
17:20.54 brlcad so calc points takes 6 of those
17:21.01 maths22 brlcad: thanks
17:21.23 maths22 we should proxy fisheye through apache
17:21.44 microchip_sac calc points takes 6 of fastf_t[4]
17:21.45 brlcad which seems like a problem since es_peqn is 7 of them
17:22.10 brlcad maths22: you know how to do that?
17:22.36 brlcad you don't have access to the apache config, but if you can write down steps to make it happen I can make the necessary edits
17:22.53 brlcad unless it can be all done through .htaccess (which I'd doubt)
17:23.09 maths22 I will give you a snippet of apache config
17:26.07 maths22 brlcad: http://paste.ubuntu.com/6685806/
17:26.49 brlcad http://www.google-melange.com/gci/task/view/google/gci2013/5887726993801216
17:26.50 gcibot Proxy Fisheye interface through Apache
17:26.50 gcibot Status: Open
17:26.50 gcibot Mentor(s): Sean
17:27.18 maths22 I can claim that once you review my compiler flag one
17:27.22 maths22 no hurry
17:30.12 brlcad added the snippet
17:30.34 Notify 03GCI:Shardul Chiplunkar * 5240158668455936 Enable a new compiler warning, fix any issues that ensue #5: Ready for review - The work on this task is ready to be reviewed.
17:30.53 microchip_sac brlcad: could you please see ^
17:31.08 brlcad microchip_sac: so what is the 7th?
17:31.17 brlcad that's at the heart of the mismatch
17:31.36 brlcad (well really the problem is the const, but still the sizes should match)
17:32.37 brlcad [Fri Jan 03 12:31:51 2014] [error] [client 71.206.34.125] File does not exist: /usr/web/brlcad.org/static, referer: http://brlcad.org/fisheye/
17:34.58 microchip_sac brlcad: the 7th doesn't make sense, because common sense tells me there are *six* planes
17:35.00 brlcad starseeker: don't know if this is of interest, http://thewalters.org/events/eventdetails.aspx?e=2738
17:35.13 microchip_sac and es_peqn keeps track of all of them and an extra
17:35.16 brlcad microchip_sac: does the callee use all 7?
17:35.32 brlcad figure out what it uses them for
17:35.52 brlcad if the 7th is accessed/used, then that might indicate a problem
17:36.03 brlcad (which is why it's useful to enable warnings like this, to find problems)
17:36.25 brlcad not just an exercise to keep devs busy fighting types ;)
17:37.50 *** join/#brlcad kesha__ (~kesha@49.248.244.12)
17:38.03 *** join/#brlcad FreezingCold (~FreezingC@135.0.41.14)
17:38.28 microchip_sac brlcad: the callee accesses indexes of es_peqn based on user selections from a menu
17:38.36 microchip_sac (or so I think)
17:40.30 microchip_sac OK, not I think, but I see -- a comment specifically says es_peqn[something] = plane of interest from es_menu
17:43.28 Notify 03GCI:Rishi Sharma * 5822710852616192 Create a New Year Greeting Card for BRL-CAD.: Ready for review - The work on this task is ready to be reviewed.
17:44.47 microchip_sac brlcad: got it!
17:45.16 microchip_sac if es_peqn[7] ever is accessed, it means that
17:45.31 microchip_sac the user has selected the 'go back to main menu' option
17:46.28 microchip_sac I don't see why they needed an extra indice for that, though; probably to avoid es_peqn[es_menu] giving an error
17:47.22 microchip_sac could you review http://www.google-melange.com/gci/task/view/google/gci2013/5240158668455936 now?
17:47.24 gcibot Enable a new compiler warning, fix any issues that ensue #5
17:47.25 gcibot Status: NeedsReview (3 days, 18 hours left)
17:47.25 gcibot Mentor(s): Kesha Shah
17:47.32 microchip_sac I think the patch is all right
17:48.12 microchip_sac except for the es_peqn issue, which I don't think will cause trouble
17:48.35 microchip_sac the seventh index quits the program, I guess
17:50.50 microchip_sac brlcad: nope, it *does* quit for the seventh index
17:54.35 Notify 03GCI:Toshita * 5269834744987648 Design a cover image or banner for our Facebook page (and/or another social media site) #13: Ready for review - The work on this task is ready to be reviewed.
17:55.51 microchip_sac brlcad?
18:00.23 brlcad much better understanding
18:00.42 brlcad the task will get reviewed in the next round of reviews, asap
18:00.56 microchip_sac is it correct, however? :)
18:01.01 brlcad you can get started on another task in the meantime or double-check your work to make sure there are no problems
18:01.10 brlcad is it? :)
18:01.19 microchip_sac I think it is
18:01.37 Notify 03GCI:Johannes Schulte * 5901237887172608 Design new API function to reduce duplication #3: Task Claimed - I would like to work on this task.
18:01.39 microchip_sac but of course, I'm thinking about my own work
18:01.47 microchip_sac :)
18:02.45 brlcad I'd probably look into making it not have a 7th element myself, having an off-by-one array is just asking for problems
18:03.05 brlcad having to copy the array is correct, but a hack
18:03.11 brlcad a hack because of a preceding hack
18:03.15 brlcad better to remove all the hacks :)
18:03.36 microchip_sac if the seventh element is *about to* be accessed then the program quits
18:03.54 microchip_sac we better write that down somewhere :)
18:04.15 microchip_sac because of a switch just before the actual access
18:05.03 Notify 03GCI:Mandeep Kaur * 5901237887172608 Design new API function to reduce duplication #3: Task Assigned - This task has been assigned to Johannes Schulte. You have 72 hours to complete this task, good...
18:14.23 Notify 03GCI:Mandeep Kaur * 5269834744987648 Design a cover image or banner for our Facebook page (and/or another social media site) #13: Needs Improvement - You have to be more creative. Have a look at...
18:14.43 Notify 03GCI:Mandeep Kaur * 5269834744987648 Design a cover image or banner for our Facebook page (and/or another social media site) #13: Task Needs More Work - One of the mentors has sent this task back...
18:33.03 maths22 brlcad: fisheye is now running at http://brlcad.org/fisheye/
18:47.42 Notify 03GCI:Johannes Schulte * 5901237887172608 Design new API function to reduce duplication #3: Ready for review - The work on this task is ready to be reviewed.
18:51.40 *** join/#brlcad jschulte (~johannes@p4FDFB4C6.dip0.t-ipconnect.de)
18:51.53 Notify 03GCI:Johannes Schulte * 5901237887172608 Design new API function to reduce duplication #3 http://www.google-melange.com/gci/task/view/google/gci2013/5901237887172608: None I did implementation...
19:11.14 Notify 03GCI:Armen * 6501205291302912 Fix at least 8 spelling mistakes in at least 8 different files: Task Claimed - I would like to work on this task.
19:12.34 Notify 03GCI:Mandeep Kaur * 6501205291302912 Fix at least 8 spelling mistakes in at least 8 different files: Task Assigned - This task has been assigned to Armen. You have 72 hours to complete this task,...
19:34.02 Notify 03GCI:kenneth dela harlley * 5232363101487104 Post and promote typography video: Ready for review - The work on this task is ready to be reviewed.
19:36.15 *** join/#brlcad Ch3ck (~Ch3ck@195.24.220.16)
20:04.38 *** join/#brlcad Ch3ck (~Ch3ck@195.24.220.16)
20:14.30 maths22 brlcad: change the last line from ProxyPass /fisheye http://brlcad.org:8060/fisheye
20:14.41 maths22 to ProxyPass /fisheye http://localhost:8060/fisheye
20:28.57 starseeker brlcad: cool!
20:36.28 starseeker brlcad: this sounds like it might be up your alley: https://www.eventbrite.com/e/art-bytes-at-the-walters-tickets-9534980383
21:04.37 Notify 03GCI:Michael * 5332162035318784 Write a wiki page on joining the BRL-CAD project: None - Does it look good??
21:15.33 Notify 03GCI:Sean * 5660495776317440 Create prototype 2D Drawing: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to this task to satisfy the...
21:23.39 Notify 03GCI:Sean * 5660495776317440 Create prototype 2D Drawing: branding and example - Ilkin, this is getting a little better but I don't think there's enough time to achieve the task objectives at this...
21:38.01 Notify 03GCI:Toshita * 5269834744987648 Design a cover image or banner for our Facebook page (and/or another social media site) #13: Additions - I have added in some features to the cover and I am working...
21:50.38 Notify 03GCI:Sean * 5332162035318784 Write a wiki page on joining the BRL-CAD project: don't understand - There are three examples in the references section including the gnome, ubuntu, and freebsd pages....
21:50.43 Notify 03GCI:Sean * 5332162035318784 Write a wiki page on joining the BRL-CAD project: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to...
22:03.54 Notify 03GCI:Sean * 5822710852616192 Create a New Year Greeting Card for BRL-CAD.: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to this...
22:03.55 Notify 03GCI:Sean * 5822710852616192 Create a New Year Greeting Card for BRL-CAD.: cite your sources - Rishi, this still doesn't look like a card design we could use. It looks like you searched the web...
22:19.54 Notify 03GCI:Sean * 5901237887172608 Design new API function to reduce duplication #3: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to...
22:19.55 Notify 03GCI:Sean * 5901237887172608 Design new API function to reduce duplication #3: not digging it - Johannes, this doesn't feel like a good API addition as-is. The new function fails to stand on it's...
22:20.29 Notify 03GCI:Sean * 5232363101487104 Post and promote typography video: Task Needs More Work - One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to this task to...
22:21.49 Notify 03GCI:Sean * 5232363101487104 Post and promote typography video: Erm... title?? - That's not much of a Youtube title and there's no description whatsoever. That's not very helpful. It needs to say...
22:32.36 Notify 03GCI:Michael * 5332162035318784 Write a wiki page on joining the BRL-CAD project: Ready for review - The work on this task is ready to be reviewed.
22:36.31 Notify 03GCI:kenneth dela harlley * 5232363101487104 Post and promote typography video: Ready for review - The work on this task is ready to be reviewed.
22:45.04 Notify 03GCI:fernozzle * 5034143381454848 Compile BRL-CAD on Windows using Cygwin: Claim Removed - The claim on this task has been removed, someone else can claim it now.
23:03.13 Notify 03BRL-CAD:j-schulte * 59270 (brlcad/trunk/src/librt/primitives/arbn/arbn.c brlcad/trunk/src/librt/primitives/table.c): Add volume function for arbn to librt
23:04.18 Notify 03GCI:Johannes Schulte * 5901237887172608 Design new API function to reduce duplication #3: Ready for review - The work on this task is ready to be reviewed.
23:05.38 Notify 03GCI:Johannes Schulte * 5901237887172608 Design new API function to reduce duplication #3: None - As you may have seen, I implemented a volume function for arbn, so when this task is finished, a...

Generated by irclog2html.pl Modified by Tim Riker to work with infobot.