| 01:17.01 | *** join/#brlcad caen23 (~caen23@92.81.194.192) | |
| 01:31.31 | *** join/#brlcad kesha_ (~kesha@14.139.122.114) | |
| 02:04.51 | brlcad | Izak_: they are nodes in the red-black tree, a generic data structure for holding anything ... so you could store ray information in a node, but no they're not rays |
| 02:05.18 | brlcad | and Izak_ see what ``Erik said |
| 02:05.24 | brlcad | don't break anything ;) |
| 03:00.34 | Notify | 03BRL-CAD:phoenixyjll * 56684 brlcad/trunk/src/libbrep/boolean.cpp: Add vertexes correctly. |
| 05:51.30 | Notify | 03BRL-CAD:phoenixyjll * 56685 brlcad/trunk/src/libbrep/boolean.cpp: Check m_type: if the event is not a curve event, we don't need to do anything. |
| 05:58.39 | zero_level | ``Erik : many of current utilities allow images to be read from stdin and stored in stdout. How do you think we can implement this provided current feature of icv library. |
| 11:07.32 | *** join/#brlcad infobot (~infobot@rikers.org) | |
| 11:07.32 | *** topic/#brlcad is BRL-CAD || http://brlcad.org || logs: http://ibot.rikers.org/%23brlcad/ || GSoC 2013! http://brlcad.org/wiki/Google_Summer_of_Code | |
| 11:49.59 | *** join/#brlcad Izak__ (~Izak@66-118-151-70.static.sagonet.net) | |
| 11:51.11 | *** join/#brlcad Izak___ (~Izak@195.24.220.16) | |
| 12:35.14 | ``Erik | zero_level: sounds like maybe a live fd in the struct would be handy, maybe with a flags field that has a bit to determine if the struct represents a streamed image or buffered one... |
| 12:35.40 | ``Erik | zero_level: I suppose you could merely read stdin until it ends, then do the op, then write to stdout.. much like how 'sort' operates |
| 12:54.24 | *** join/#brlcad kesha_ (~kesha@14.139.122.114) | |
| 13:15.13 | Notify | 03BRL-CAD:starseeker * 56686 (brlcad/trunk/src/other/stepcode/src/cldai/sdaiModel_contents.h brlcad/trunk/src/other/stepcode/src/cllazyfile/lazyInstMgr.h): Clear a couple warnings |
| 13:24.09 | Notify | 03BRL-CAD:erikgreenwald * 56687 brlcad/trunk/src/libbrep/boolean.cpp: comment out unused variable |
| 14:06.42 | Notify | 03BRL-CAD Wiki:Phoenix * 5959 /wiki/User:Phoenix/GSoc2013/Reports: /* Week 8 */ |
| 15:09.53 | Izak___ | ``Erik: Can i completely remove rt_hrt_??port4() from the hrt.c since they are database version 4 functions ? |
| 15:10.30 | ``Erik | Izak___: yes |
| 15:11.02 | Izak___ | ``Erik: thanks |
| 15:33.50 | Izak___ | ``Erik: What exactly is a trailing blank? hacking file says a white space should follow every semicolon. calmoore's "removing trailing blanks" comment on r56687 is not understood |
| 15:43.25 | ``Erik | that part of the HACKING file is not referring to an "end of statement" semicolon, but the kind found in a "for" loop |
| 15:44.02 | Izak___ | ``Erik: Thanks |
| 15:44.25 | Izak___ | learning how to commit |
| 15:45.46 | ``Erik | I recommend making your first couple commits small... |
| 15:46.37 | Izak___ | okay |
| 15:52.31 | ``Erik | a good quick patch might be to remove the import4/export4 funcs |
| 15:53.37 | Izak___ | yeah that's what I am doing.. |
| 15:55.25 | ``Erik | then maybe do the print() function and commit it, then the export5 and commit, then the import5 and commit... by that point, you should be competently contributing without needing much supervision :D |
| 15:57.26 | Izak___ | ``Erik: Thanks. hope i haven't been a full time job for you ? |
| 15:57.42 | Izak___ | :-) |
| 15:59.43 | ``Erik | nah :) and hopefully soon you'll be providing far more benefit than the time spent helping you get started |
| 16:01.33 | Izak___ | ``Erik: :). cool |
| 16:04.23 | Notify | 03BRL-CAD:phoenixyjll * 56688 brlcad/trunk/src/libbrep/intersect.cpp: Wrong comment - we don't merge at this time. |
| 16:06.25 | Notify | 03BRL-CAD:carlmoore * 56689 brlcad/trunk/include/icv.h: remove trailing blanks/tabs; make minor formatting changes and make correction to NITERP (all of this is in **comments**) |
| 16:20.58 | Notify | 03BRL-CAD:carlmoore * 56690 brlcad/trunk/src/libicv/decimate.c: presumably, 'binterp' and 'ninterp' are correct spellings -?? |
| 16:26.30 | zero_level | carlmoore : Thanks. |
| 16:26.51 | Notify | 03BRL-CAD:carlmoore * 56691 brlcad/trunk/misc/auto-man-page/README.auto-man-page-handling: fix this README file (added missing newline at end, and put in a string used near top, to mark end of the document; and fix a spelling) |
| 16:26.57 | zero_level | carlmoore : Thanks. |
| 16:27.01 | zero_level | carlmoore binterp and ninterp are corrrect. |
| 16:27.13 | ``Erik | he's not on irc |
| 16:27.18 | zero_level | ok. |
| 16:27.24 | zero_level | so shld i mail him ? |
| 16:31.58 | Notify | 03BRL-CAD:carlmoore * 56692 brlcad/trunk/include/brep.h: remove a trailing blank/tab, and fix a spelling in a comment |
| 16:46.03 | *** join/#brlcad caen23 (~caen23@92.81.200.36) | |
| 16:55.07 | Notify | 03BRL-CAD:carlmoore * 56693 (brlcad/trunk/misc/auto-man-page/auto-man-progs.txt brlcad/trunk/src/libicv/decimate.c brlcad/trunk/src/tclscripts/util/expand_int_ranges.tcl): fix spellings (notice my inclusion of 'ninterp' |
| 17:18.40 | brlcad | zero_level: he's not on our mailing list either |
| 17:18.53 | brlcad | so the goal is to double-check so he doesn't spend timing cleaning up changes you make ;) |
| 17:19.53 | brlcad | if he corrects something, note what he changed and make a note to yourself to be sure you check that as you make more changes |
| 18:08.20 | Notify | 03BRL-CAD:iiizzzaaakkk * 56694 brlcad/trunk/src/librt/primitives/hrt/hrt.c: Ensuring consistent bu_log("rt_hrt_xxx: not implemented yet"); in callback functions in src/librt/primitives/hrt/hrt.c |
| 18:13.39 | Notify | 03BRL-CAD Wiki:IIIzzzaaakkk * 5960 /wiki/User:Izak/GSOC_2013_logs: /* August 5th to August 9th */ |
| 18:13.55 | Izak___ | ``Erik: Did you just see that ? |
| 18:14.02 | ``Erik | woohoo, izak's first commit O.o |
| 18:14.18 | ``Erik | (and it looks good, no corrections needed) |
| 18:14.25 | brlcad | woot! |
| 18:14.37 | Izak___ | I am happy! |
| 18:14.40 | brlcad | :) |
| 18:14.44 | brlcad | it's a good day |
| 18:14.53 | Izak___ | yeah sure it is |
| 18:15.07 | ``Erik | now it's time to change some of those bu_log("not implemented yet\n"); into implemented functions |
| 18:15.16 | Izak___ | yes sure |
| 18:15.23 | brlcad | welcome to the team, hopefully many many many more commits to come ;) |
| 18:15.48 | Izak___ | They will keep coming :) |
| 18:15.56 | brlcad | be sure to read the commiter-responsibilities section in the HACKING file now that you have commit access |
| 18:16.07 | brlcad | but basically, try not to break anything and do great work |
| 18:16.17 | brlcad | it's a brief section |
| 18:16.36 | Izak___ | okay |
| 18:16.57 | Izak___ | looking at HACKING file |
| 18:19.55 | *** join/#brlcad kesha_ (~kesha@14.139.122.114) | |
| 18:23.50 | Notify | 03BRL-CAD:starseeker * 56695 brlcad/trunk/src/conv/step/ON_Brep.cpp: Add knots to curves |
| 18:35.00 | Notify | 03BRL-CAD:carlmoore * 56696 (brlcad/trunk/src/conv/g-acad.c brlcad/trunk/src/conv/g-egg.c): remove unneeded break in g-acad, and revise usage statement in g-egg (adding -P) |
| 18:46.03 | brlcad | zero_level: a note that will probably take a half hour or more to fix, we want to be using size_t as much as possible |
| 18:46.21 | brlcad | where something represents a size, like width and height, probably should be a size_t |
| 18:46.32 | brlcad | instead of int/unsigned int/long/unsigned long, etc |
| 18:51.47 | Notify | 03BRL-CAD:brlcad * 56697 brlcad/trunk/src/libicv/decimate.c: check the more general case to future-proof against a signed type. make log messages be less developer-speak. |
| 18:51.56 | zero_level | hi brlcad, ``Erik and others. Pls test pixscale. |
| 18:52.15 | Notify | 03BRL-CAD:brlcad * 56698 brlcad/trunk/src/libicv/decimate.c: ws indent comma cleanup |
| 18:52.17 | zero_level | It turns out there is a bug in loading the image. |
| 18:52.18 | brlcad | also should check your commas... |
| 18:52.49 | brlcad | what sort of bug? |
| 18:53.05 | zero_level | fread: Inappropriate ioctl for device |
| 18:53.40 | zero_level | this must be around line 96 |
| 18:53.48 | zero_level | <PROTECTED> |
| 18:56.30 | brlcad | there's nothing inherently wrong with that line, but if fread fails like that, it means buffp is not right |
| 18:57.25 | brlcad | if you moved that to icv, you would have had to take care of that global |
| 18:58.12 | Notify | 03BRL-CAD:starseeker * 56699 (brlcad/trunk/src/conv/step/ON_Brep.cpp brlcad/trunk/src/conv/step/g-step.cpp): Write to an actual file. |
| 19:05.33 | zero_level | brlcad : moving to icv means ____ ? |
| 19:10.06 | brlcad | added it to icv |
| 19:10.28 | brlcad | or is the code still just sitting in src/util/ and you get that error when running the existing pixscale tool? |
| 19:11.43 | zero_level | :-) |
| 19:11.52 | zero_level | i just add functionalities. |
| 19:12.17 | zero_level | since each code in util folder. |
| 19:12.50 | brlcad | are you saying you just added it to libicv? |
| 19:12.58 | zero_level | no. |
| 19:13.11 | brlcad | then no comprendo :) |
| 19:13.12 | zero_level | binterp and ninterp are already commited. |
| 19:13.21 | brlcad | to libicv |
| 19:13.43 | zero_level | yes. |
| 19:14.54 | brlcad | and? :) |
| 19:15.12 | brlcad | i'm not sure if you're asking something, telling me something, or what ... |
| 19:15.19 | zero_level | I am trying to compare two files from pixscale. |
| 19:15.26 | zero_level | and the function in decimate.c |
| 19:15.36 | zero_level | so just wanted pixscale to work. |
| 19:15.45 | brlcad | it works for me here |
| 19:15.46 | zero_level | brlcad : keep light headed :-) |
| 19:15.57 | brlcad | i am VERY light headed at the moment ;) |
| 19:15.59 | zero_level | I am just discussing |
| 19:16.04 | zero_level | :-) |
| 19:16.27 | zero_level | whcih involves asking, telling and .. ;) |
| 19:16.53 | brlcad | okay, but you're not being very clear .. so may ambiguities (things that are not clear) |
| 19:16.58 | brlcad | s/may/many/ |
| 19:17.14 | zero_level | didint it give the ioctl erro ? |
| 19:17.20 | brlcad | such as "I am trying to compare two files from pixscale." .. two output files or two source files? |
| 19:17.23 | brlcad | two input files? |
| 19:18.25 | zero_level | did pixscale work at your system ? |
| 19:18.30 | brlcad | yes |
| 19:18.36 | brlcad | and I got the ioctl error |
| 19:18.43 | brlcad | warning really |
| 19:18.49 | zero_level | so it worked ? |
| 19:18.50 | brlcad | the index is probably wrong |
| 19:18.54 | brlcad | yep worked just fine |
| 19:18.57 | zero_level | alright. |
| 19:19.10 | brlcad | it's probably reading too much |
| 19:19.24 | brlcad | reading too far, maybe an off-by-one error |
| 19:20.12 | brlcad | but help me understand what you just said -- what are you trying to compare, and what function in decimate.c? |
| 19:20.45 | brlcad | if you have a build, you can test pixscale with this: |
| 19:20.52 | brlcad | bin/pixscale -s 512 -S 1024 share/pix/moss.pix |bin/pix-fb -s1024 -F/dev/X |
| 19:24.02 | brlcad | zero_level: ? |
| 19:24.22 | zero_level | brlcad : ok. |
| 19:24.42 | zero_level | wait. |
| 19:24.45 | brlcad | help me understand what you said earlier -- what are you trying to compare, and what function in decimate.c? |
| 19:25.12 | brlcad | does not wait, he does other things :) |
| 19:25.13 | zero_level | I am testing my implemented functions in decimate.c |
| 19:25.53 | zero_level | so to compare the result i am using pixscale. |
| 19:26.16 | zero_level | brlcad : I hope this has no ambiguity. :-) |
| 19:26.54 | zero_level | also gave build commands in the mean time. Benifit of two screens. |
| 19:33.32 | zero_level | brlcad : I am not sure if it worked correctly at your end. |
| 19:33.47 | zero_level | Did you try pix-png and see what is the output. |
| 19:34.35 | brlcad | zero_level: a common error for new users is to not realize that pix files are headerless/dimensionless |
| 19:34.58 | brlcad | if the image size is not 512x512, you have to tell the pix* tool what the dimensions are |
| 19:35.05 | zero_level | i did |
| 19:35.08 | zero_level | -s1024 |
| 19:35.52 | brlcad | and yes, pix-png worked just fine |
| 19:43.51 | brlcad | you can test my example above to confirm on your end: bin/pixscale -s 512 -S 1024 share/pix/moss.pix | bin/pix-png -s 1024 -o file.png |
| 19:49.00 | *** join/#brlcad kesha_ (~kesha@14.139.122.114) | |
| 19:52.17 | zero_level | apparently i see color mismatch |
| 19:52.37 | zero_level | I believe there is an issue with one color channel. |
| 19:53.28 | zero_level | thats an issue with pixscale i guess. |
| 19:54.10 | zero_level | brlcad : but nevertheless there are these errors. Although it still works. |
| 19:54.20 | zero_level | is looking at a way to fix this. |
| 19:55.22 | *** join/#brlcad kesha__ (~kesha@14.139.122.114) | |
| 19:56.58 | zero_level | brlcad : do you see perfect interpolation or a color mismatch ? |
| 20:08.42 | ``Erik | blindly assuming that the existing tool is correct may be a bad approach... you should try to understand why they're different, then make the determination in which side is 'wrong' and argue the case |
| 20:10.02 | zero_level | ``Erik thats what i am trying to do. |
| 20:10.20 | zero_level | Can you test the command brlcad wrote. |
| 20:10.29 | ``Erik | aight, just making sure :) |
| 20:10.47 | ``Erik | I'm heading out in a couple minutes, I might have some time tomorrow.. |
| 20:10.52 | zero_level | and see if does introduce a color mismatch or interpolates the image perfectly. |
| 20:10.54 | zero_level | alright. |
| 20:11.08 | ``Erik | <-- just has enough time for broad platitudes ;) |
| 20:11.21 | zero_level | alright. |
| 20:25.17 | *** join/#brlcad kesha_ (~kesha@14.139.122.114) | |
| 20:37.38 | *** join/#brlcad Izak__ (~Izak@66-118-151-70.static.sagonet.net) | |
| 20:54.25 | Notify | 03BRL-CAD:starseeker * 56700 (brlcad/trunk/src/other/libgdiam/gdiam.cpp brlcad/trunk/src/other/libgdiam/gdiam.hpp): Back out the near zero logic - causing as much trouble as it's solving |
| 20:56.10 | *** join/#brlcad kesha_ (~kesha@14.139.122.114) | |
| 20:58.07 | Notify | 03BRL-CAD:tbrowder2 * 56701 brlcad/trunk/misc/auto-man-page/auto-man-progs.txt: reorder progs for testing |
| 21:02.19 | Notify | 03BRL-CAD:tbrowder2 * 56702 brlcad/trunk/src/proc-db/wdb_example.c: add usage string; eliminate usage function |
| 21:09.44 | *** join/#brlcad kesha_ (~kesha@14.139.122.114) | |
| 21:11.49 | *** join/#brlcad kesha_ (~kesha@14.139.122.114) | |
| 21:14.34 | *** join/#brlcad kesha__ (~kesha@14.139.122.114) | |
| 21:14.56 | *** join/#brlcad mpictor (~mark@2601:d:b280:b5:d63d:7eff:fe2d:2505) | |
| 21:15.16 | *** join/#brlcad kesha__ (~kesha@14.139.122.114) | |
| 21:27.11 | Notify | 03BRL-CAD:carlmoore * 56703 brlcad/trunk/doc/docbook/system/man1/en/g-egg.xml: revise the g-egg man page |
| 21:28.28 | Notify | 03BRL-CAD:carlmoore * 56704 brlcad/trunk/src/conv/g-egg.c: fix the options |
| 21:29.14 | *** join/#brlcad kesha__ (~kesha@14.139.122.114) | |
| 21:36.32 | Notify | 03BRL-CAD:starseeker * 56705 brlcad/trunk/src/other/libgdiam/gdiam.cpp: Unbelieveable. Simply changing the value being checked here from 1e-9 to 1e-8 causes failures. Need to re-think this. |
| 22:06.29 | Notify | 03BRL-CAD:carlmoore * 56706 (brlcad/trunk/src/conv/euclid/g-euclid.c brlcad/trunk/src/conv/g-acad.c): remove unneeded braces in g-acad.c, and in g-euclid.c make changes to the usage statement and implement h? |
| 22:08.06 | *** join/#brlcad caen23_ (~caen23@92.81.200.36) | |
| 22:43.20 | Notify | 03BRL-CAD:r_weiss * 56707 brlcad/trunk/src/librt/primitives/bot/tie_kdtree.c: Fixed some memory free errors when raytracing with bot-tie. |
| 23:59.04 | brlcad | someone following our more recent work: http://www.geom.uiuc.edu/software/cglist/csg.html |