| 00:44.23 | *** join/#brlcad Al_Da_Best (~Al_Da_Bes@elvyn-248-109.halls.student.lut.ac.uk) | |
| 00:44.23 | *** join/#brlcad piksi (piksi@pi-xi.net) | |
| 00:44.23 | *** join/#brlcad Stattrav_ (~Stattrav@61.12.114.82) | |
| 00:44.23 | *** join/#brlcad brlcad (~sean@BZ.BZFLAG.BZ) | |
| 00:44.23 | *** join/#brlcad KimK (~Kim__@209.248.147.2.nw.nuvox.net) | |
| 00:44.23 | *** join/#brlcad crdueck (~cdk@d173-238-127-19.home4.cgocable.net) | |
| 00:44.23 | *** join/#brlcad bhinesley (~bhinesley@108.220.113.189) | |
| 00:44.23 | *** join/#brlcad yiyus (1242712427@je.je.je) | |
| 00:44.23 | *** join/#brlcad ChanServ (ChanServ@services.) | |
| 00:44.23 | *** mode/#brlcad [+oo brlcad ChanServ] by wolfe.freenode.net | |
| 01:38.06 | *** join/#brlcad xth1 (~thiago@187.106.51.2) | |
| 05:30.41 | *** join/#brlcad dtidrow (~dtidrow@c-68-84-167-135.hsd1.mi.comcast.net) | |
| 06:09.42 | *** join/#brlcad ksuzee (~ksu@46.149.81.166) | |
| 06:15.38 | *** join/#brlcad ksuzee (~ksu@46.149.81.166) | |
| 06:31.23 | ksuzee | hello, Sean!! Could you look in file util/lowp.c? It seems to be a typo there |
| 06:31.34 | ksuzee | I even don't know |
| 06:33.23 | ksuzee | there're 2 similar part of code one by one |
| 06:33.34 | ksuzee | I'm not sure about it |
| 06:49.07 | *** join/#brlcad d_rossberg (~rossberg@BZ.BZFLAG.BZ) | |
| 07:56.55 | *** join/#brlcad stas (~stas@82.208.133.12) | |
| 08:35.20 | *** join/#brlcad andrei (~andrei@5-12-88-164.residential.rdsnet.ro) | |
| 08:35.33 | andrei | Hello! |
| 08:49.49 | brlcad | hi andrei |
| 08:50.12 | brlcad | ksuzee: can you be more specific? |
| 08:50.47 | ksuzee | yes, in row 110 readval = read(infd3, in3, scanbytes); |
| 08:51.09 | ksuzee | 6 rows |
| 08:51.26 | ksuzee | and after that there's the same block |
| 08:51.34 | andrei | brlcad: should the size parameter be mandatory? should I use the curent size as implicit one if no 3rd paramter given? |
| 08:52.45 | ksuzee | Simian found it |
| 08:53.09 | ksuzee | and I'm not sure what to do with it |
| 08:56.18 | brlcad | those aren't "quite" the same blocks ksuzee -- reading from two different infd's and in's |
| 08:56.30 | brlcad | but sure, those six lines could be turned into a function |
| 08:57.27 | brlcad | called three times with different params |
| 08:57.48 | brlcad | andrei: don't know what context you're referring to |
| 08:57.52 | brlcad | what size parameter? |
| 08:58.09 | brlcad | oooh, tpkg |
| 08:58.19 | brlcad | why would you make it mandatory? |
| 08:58.33 | ksuzee | ok, I understand |
| 08:58.35 | ksuzee | but |
| 08:58.53 | ksuzee | in first block there're infd1, in1 |
| 08:59.10 | ksuzee | but other two are with infd3, in3, |
| 08:59.19 | brlcad | yes? |
| 08:59.27 | ksuzee | yes |
| 08:59.40 | brlcad | but, what's your point? :) |
| 08:59.47 | brlcad | yes they are different |
| 09:00.00 | brlcad | so the function you make will have different values being passed to it |
| 09:00.22 | brlcad | make sure you test before and after ;) |
| 09:00.38 | brlcad | lowp is fortunately a very simple filter |
| 09:00.54 | andrei | then if it's not mandatory , I will only update the "help" output |
| 09:01.26 | brlcad | andrei: I would expect the size parameter to be some -b # or -s # or similar command-line option |
| 09:01.36 | brlcad | -b probably |
| 09:01.49 | ksuzee | ok, thanks I'll make the patch with this functiom |
| 09:06.25 | ksuzee | em, Sean=) Mb, I misunderstand a bit)) But look: we have assignmet of in1, in2, in3 and infd1, infd2 and infd 3 |
| 09:06.45 | ksuzee | but the second "read" gets in3 and infd3 |
| 09:06.50 | *** join/#brlcad anuragmurty (~anurag@14.139.128.11) | |
| 09:07.06 | ksuzee | I think if should get in2 and infd2 |
| 09:08.39 | ksuzee | I see about function, but I don't understand about it |
| 09:09.58 | ksuzee | "it" means second situstion) |
| 09:10.05 | ksuzee | *situation |
| 09:11.17 | andrei | brlcad, looking in the cmakelists from libbu, I added BRLCAD_ADDEXEC(tpkg tpkg.c libpkg NO_INSTALL) to the libpkg one |
| 09:11.33 | andrei | I do get a testpkg binary , not sure from where, but it doesn't seem to be the same file. |
| 09:11.45 | andrei | in /usr/brlcad/bin |
| 09:14.05 | andrei | is there something wrong ? |
| 09:24.50 | andrei | shouldn't I get a tpkg binary in /usr/brlcad/bin |
| 09:36.38 | andrei | (I think I fixed the cmake part) |
| 09:39.41 | *** join/#brlcad kane__ (~kane@e182051111.adsl.alicedsl.de) | |
| 09:48.41 | andrei | No, it does compile succesfully, but I don't get the expected binary. |
| 09:49.41 | andrei | this is my CMake |
| 09:49.43 | andrei | http://slexy.org/view/s29UVlIwGi |
| 09:56.38 | *** join/#brlcad andrei_ (andrei@5-12-88-164.residential.rdsnet.ro) | |
| 11:23.02 | *** join/#brlcad kane__ (~kane@e181161183.adsl.alicedsl.de) | |
| 12:11.34 | *** part/#brlcad kane__ (~kane@e181161183.adsl.alicedsl.de) | |
| 12:31.13 | *** join/#brlcad ksuzee (~ksu@46.149.81.166) | |
| 13:13.10 | ``Erik | andrei_: you're explicitely telling it that tpkg should be built and NOT installed... which is correct, it should create ${BUILD_DIR}/bin/tpkg, but not ${INSTALL_DIR}/bin/tpkg |
| 13:19.01 | CIA-65 | BRL-CAD: 03phoenixyjll * r50683 10/brlcad/trunk/src/librt/primitives/revolve/revolve_brep.cpp: Deal with non-full revolution of revolve. Create two surfaces in that case. |
| 13:22.42 | CIA-65 | BRL-CAD: 03Phoenix 07http://brlcad.org * r3731 10/wiki/User:Phoenix/GSoc2012/Reports: /* Week 1 */ |
| 13:30.23 | *** join/#brlcad kane__ (~kane@e181161183.adsl.alicedsl.de) | |
| 14:53.08 | *** part/#brlcad anuragmurty (~anurag@14.139.128.11) | |
| 14:53.19 | *** join/#brlcad anuragmurty (~anurag@14.139.128.11) | |
| 15:20.30 | brlcad | andrei_: question answered? |
| 15:21.46 | *** join/#brlcad kane__ (~kane@e181161183.adsl.alicedsl.de) | |
| 15:22.32 | brlcad | anuragmurty: note that the comments on your unit test patch apply to all code, like when you upload your current progress that shoots a ray (which you should do today) |
| 15:22.56 | brlcad | consistency is king ;) |
| 15:33.09 | CIA-65 | BRL-CAD: 03d_rossberg * r50684 10/brlcad/trunk/src/libbu/ (CMakeLists.txt Makefile.am test_bitv.c): apply sourceforge parch 3527405 (test_bitv4.patch) from anrgmrty: "Unit test for testing of src/libbu/bitv.c has been written with appropriate changes to CMakelists.txt and Makefile.am" |
| 15:44.20 | *** part/#brlcad kane__ (~kane@e181161183.adsl.alicedsl.de) | |
| 16:15.33 | *** join/#brlcad Stattrav_ (~Stattrav@61.12.114.82) | |
| 16:17.18 | CIA-65 | BRL-CAD: 03tbrowder2 * r50685 10/brlcad/trunk/include/bu.h: correct referenced function name |
| 16:25.31 | CIA-65 | BRL-CAD: 03tbrowder2 * r50686 10/brlcad/trunk/src/libbu/vls.c: add local helper function for bu_vls_vprintf |
| 16:31.28 | anuragmurty | brlcad: ok i will keep that in mind |
| 16:31.30 | anuragmurty | :-) |
| 16:31.36 | anuragmurty | food. again |
| 16:42.37 | CIA-65 | BRL-CAD: 03tbrowder2 * r50687 10/brlcad/trunk/src/libbu/vls.c: add two more helper functions for bu_vls_vprintf; comment out temporarily to prevent unused warning/error |
| 16:45.46 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 16:51.53 | CIA-65 | BRL-CAD: 03starseeker * r50688 10/brlcad/trunk/misc/CMake/BRLCAD_CheckFunctions.cmake: remove debugging message |
| 16:55.34 | CIA-65 | BRL-CAD: 03starseeker * r50689 10/brlcad/trunk/src/librt/primitives/revolve/revolve_brep.cpp: tol is used now |
| 16:59.11 | CIA-65 | BRL-CAD: 03tbrowder2 * r50690 10/brlcad/trunk/src/libbu/vls.c: move S conversion specifier to more appropriate place, modify comment to reflect correct alternative solution |
| 17:00.53 | *** join/#brlcad anuragmurty1 (~anurag@14.139.128.12) | |
| 17:15.11 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 17:31.30 | CIA-65 | BRL-CAD: 0314.139.128.12 07http://brlcad.org * r3732 10/wiki/User:Anuragmurty: /* community bonding period */ |
| 17:31.56 | CIA-65 | BRL-CAD: 03starseeker * r50691 10/brlcad/trunk/src/libbu/test_bitv.c: Use bu functions - might want to run make regress-repository when working on this file... |
| 18:17.53 | CIA-65 | BRL-CAD: 03brlcad * r50692 10/brlcad/trunk/src/libbu/vls.c: move %S back down. it wasn't some long string format. it was like %V is now, used everywhere in our code for printing our bu_vls strings. |
| 18:30.08 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 18:36.43 | *** join/#brlcad jbschw (~jbschw@ool-4355ee10.dyn.optonline.net) | |
| 18:37.46 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 18:40.09 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 18:42.53 | CIA-65 | BRL-CAD: 03tbrowder2 * r50693 10/brlcad/trunk/src/libbu/test_vls.c: add more tests; provide for known failures during development |
| 18:58.37 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 19:03.36 | CIA-65 | BRL-CAD: 03starseeker * r50694 10/brlcad/trunk/src/other/tk/CMakeLists.txt: This might be why TK_ENABLE_XFT was incorrectly ON, but needs testing. |
| 19:17.35 | *** join/#brlcad merzo (~merzo@165-122-133-95.pool.ukrtel.net) | |
| 19:19.54 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 19:20.49 | *** join/#brlcad anuragmurty1 (~anurag@14.139.128.12) | |
| 19:21.00 | *** join/#brlcad DarkCalf (~DarkCalf@173.231.40.99) | |
| 19:21.47 | *** join/#brlcad anuragmurty2 (~anurag@14.139.128.12) | |
| 19:23.38 | *** join/#brlcad anuragmurty1 (~anurag@14.139.128.12) | |
| 19:31.35 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 19:37.28 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 19:42.24 | *** join/#brlcad cristina (~quassel@188.24.71.175) | |
| 19:42.55 | *** join/#brlcad cristina (~quassel@unaffiliated/cristina) | |
| 19:47.22 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 19:56.28 | *** join/#brlcad andrei (~andrei@5-12-88-164.residential.rdsnet.ro) | |
| 20:18.21 | andrei | hello |
| 20:18.54 | brlcad | howdy andrei |
| 20:22.28 | *** join/#brlcad Al_Da_Best (~Al_Da_Bes@elvyn-248-109.halls.student.lut.ac.uk) | |
| 20:25.34 | brlcad | Al_Da_Best: ! |
| 20:26.35 | brlcad | andrei: was your question answered? |
| 20:26.49 | andrei | yes |
| 20:27.03 | brlcad | excellent |
| 20:27.10 | brlcad | patch? :) |
| 20:28.15 | andrei | I m figuring out how to process the command-line |
| 20:28.45 | andrei | as in [-b] which is equivalent to simply giving th number, or -b 2500 ( for example) |
| 20:29.03 | Al_Da_Best | Hi there |
| 20:29.18 | CIA-65 | BRL-CAD: 03brlcad * r50695 10/brlcad/trunk/AUTHORS: add gsoc coder anurag murty for his sf patch 3527405 (Unit test has been written for src/libbu/bitv.c) that got applied by rossberg in r50684 |
| 20:29.36 | brlcad | andrei: okay, but you already have one valid patch there with the build system change |
| 20:29.50 | Al_Da_Best | Sorry for lack of anything. Reinstalled windows yesterday and just finished getting everything in order. Currently sorting out a fedora vm for brl-cad dev. That's nearly ready |
| 20:29.57 | brlcad | you could submit that (the -b change would be another) |
| 20:31.15 | brlcad | patches are supposed to be just a 'single' change anyways, so they should be two commits/patches regardless |
| 20:33.39 | brlcad | Al_Da_Best: so the problem is that a week has now passed - if it weren't the first week, that sort of week-long disappearance without talking would have put you on GSoC probation at risk of getting failed |
| 20:34.09 | Al_Da_Best | I understand. I'll be putting things up this weekend certainly |
| 20:34.29 | brlcad | you're already WAY behind having not established commit access before the coding window anyways, and now a week behind coding too |
| 20:34.42 | brlcad | remember this isn't a homework assignment |
| 20:35.10 | Al_Da_Best | Yea |
| 20:35.49 | brlcad | I know things happen, but please do be careful and put the time in on your project |
| 20:38.03 | Al_Da_Best | I'll be doing my best to catch up this weekend |
| 20:38.28 | brlcad | is there anything you can do today? |
| 20:39.24 | Al_Da_Best | I'm working now, getting it compiled on my vm then I'll start working on a patch(es) for commit |
| 20:39.37 | andrei | brlcad, I m confused. what you said was directed at me , at Al or both? |
| 20:40.02 | andrei | ah, I didn't see the first reply. |
| 20:40.19 | brlcad | andrei: "patch? :) was directed at you.. (for the build system change) |
| 20:40.38 | brlcad | along with a couple followup comments |
| 20:41.07 | brlcad | Al_Da_Best: is there anything salvageable with the patch you first submitted? |
| 20:41.29 | brlcad | that would seem to be something easy to do today, clean that up and resubmit it |
| 20:41.59 | brlcad | or should that just be closed out? |
| 20:42.39 | brlcad | (it must be correct/exact and verified) |
| 20:42.49 | Al_Da_Best | Well I can fix that up, however I wasn't sure if that was needed or not |
| 20:43.40 | brlcad | my recollection is that volume wasn't right, area was an approximation, and I dont' recall centroid |
| 20:44.05 | Al_Da_Best | Those issues were fixed shortly after |
| 20:44.29 | brlcad | right but then the patch also overlapped another patch that implemented one or two of the three, I forget |
| 20:44.35 | Al_Da_Best | Yeah it did |
| 20:44.43 | brlcad | so it has to be cross-checked with whatever the current state of the code is |
| 20:45.04 | brlcad | I think it was volume that was applied, so the other two could be applied if they can be fixed |
| 20:45.46 | Al_Da_Best | I'll check |
| 20:51.38 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 20:52.17 | andrei | seems I haven't used svn checkout for quite a while ... |
| 20:53.28 | brlcad | andrei: problem? |
| 20:53.40 | brlcad | make sure it's not the whole repo ;) |
| 20:53.41 | brlcad | ~cadsvn |
| 20:53.41 | ibot | To obtain BRL-CAD from Subversion: svn checkout https://brlcad.svn.sourceforge.net/svnroot/brlcad/brlcad/trunk brlcad |
| 20:54.56 | brlcad | andrei: next week after your exams, I'd like to work with you to apply the redblack unit test |
| 20:55.07 | CIA-65 | BRL-CAD: 03Cprecup 07http://brlcad.org * r3733 10/wiki/User:Cprecup/GSoC2012_progress: Adaptagrams vs GOBLIN comparison + GOBLIN compilation (24-25/05/2012) |
| 20:55.10 | brlcad | that's particularly useful, but needed some cleanup iirc |
| 20:55.26 | andrei | I m not sure how much time it should take |
| 20:55.29 | brlcad | ~cristina++ |
| 20:55.41 | andrei | I was willing to try working on it after this one, I mean now |
| 20:55.54 | CIA-65 | BRL-CAD: 03Anuragmurty 07http://brlcad.org * r3734 10/wiki/User:Anuragmurty: /* community bonding period */ |
| 20:56.02 | cristina | good morning, brlcad! |
| 20:56.48 | brlcad | good morning :) |
| 20:57.02 | brlcad | andrei: cool |
| 20:57.38 | cristina | brlcad: what should I understand from ~cristina++? :) |
| 20:57.39 | brlcad | that patch got several people's attention, that's a bit of code with some history to it |
| 20:57.47 | brlcad | ~karma cristina |
| 20:57.47 | ibot | cristina has karma of 1 |
| 20:58.07 | brlcad | cristina: oh, nothing really :) just means you're awesome |
| 20:58.40 | ``Erik | 1/clear |
| 20:58.53 | brlcad | 2/clear |
| 20:59.12 | cristina | brlcad: :)) this karma thing is funny |
| 20:59.19 | ``Erik | 3/exec sudo rm -rf / |
| 20:59.25 | ``Erik | :D |
| 21:00.31 | brlcad | anuragmurty: thanks for the patch |
| 21:01.01 | anuragmurty | welcome! |
| 21:01.15 | brlcad | hopefully quick to review and no problems :) |
| 21:05.42 | anuragmurty | i hope so too.. |
| 21:06.50 | anuragmurty | brlcad: i have a question regarding tests that might call bu_bomb during execution |
| 21:09.48 | andrei | brlcad, so I just upload the .diff? |
| 21:10.43 | ``Erik | if your svn tree is up to date (svn update), do svn diff > mypatch.patch and look at it to verify it includes everything needed and just what's needed, then upload that |
| 21:17.58 | brlcad | anuragmurty: and someone might have an answer, but you'd have to actually ask the question instead of waiting (for something, i do not know) :) |
| 21:18.33 | andrei | brlcad : Done. |
| 21:19.54 | anuragmurty | brlcad: if i want to give some test cases that might call a function which bombs, does that mean i need to make changes in bu_bomb?? i mean, so that it behaves differently when the calling function is a test.. |
| 21:22.59 | brlcad | andrei: awesome, thanks |
| 21:23.42 | brlcad | anuragmurty: you absolutely should not make changes to bu_bomb() .. as daniel noted earlier, you work with and test the API as-is |
| 21:24.25 | brlcad | you just need to make sure it's not failing when it shouldn't and is failing when it should |
| 21:24.48 | brlcad | in code, you actually can catch a bu_bomb() and proceed, if the blather is problematic |
| 21:25.09 | anuragmurty | okay! .. because bomb is not the same as getting an error message.. your test stops midway.. so i was just wondering.. thank you!! |
| 21:25.59 | brlcad | see BU_SETJMP() examples throughout the code |
| 21:26.22 | brlcad | it's an old C method of exception handling, similar to try/catch in c++ |
| 21:27.19 | anuragmurty | <PROTECTED> |
| 21:31.44 | andrei | the test_rbtree issues shouldn't take more than one or two hours to correct |
| 21:32.03 | andrei | I ll move to it in a few minutes |
| 21:44.32 | brlcad | if even that -- mostly style/formatting problems |
| 21:44.38 | brlcad | and a question about validation iirc |
| 21:49.19 | *** part/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 23:35.54 | starseeker | cristina: does http://bzflag.bz/~starseeker/goblin-2.8b30-cmake.tar.gz not have what you need? |
| 23:41.11 | cristina | starseeker: I get the same error when running make |
| 23:41.51 | cristina | can you tell me where did you get this source? Mine had just Makefile and Makefile.conf without CMakeLists.txt |
| 23:42.47 | brlcad | in case it comes up a couple years from now and the knowledge is lost in biomass, http://brlcad.org/gallery/s/renderings/GSI_Images/BRDM.png.html has a very cool joint articulation file |
