Thread: [brlcad-tracker] [Google Code-in 2012] New comment on Add a command-line option that sets an object
Open Source Solid Modeling CAD
Brought to you by:
brlcad
From: <no-...@go...> - 2012-12-24 18:49:21
|
Hi, javamonn has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Task Claimed I would like to work on this task. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-24 19:21:34
|
Hi, Harmanpreet Singh has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Task Assigned This task has been assigned to javamonn. You have 72 hours to complete this task, good luck! Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-24 20:26:19
|
Hi, javamonn has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: A little confused... So, I have the option added and the color parsed, but I'm unsure as to how to change the color of the rendering. Is there a function I should be calling? Or maybe a way to overwrite the color of the pixels worker.c is producting? I'd appreciate any help you could give me. Thanks, Daniel Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-25 04:31:19
|
Hi, javamonn has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Claim Removed The claim on this task has been removed, someone else can claim it now. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-27 17:38:11
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Setting the color Where is the function that sets the colour of the object? And where is the color specified in "mater sph.r ..." written to?(I'm assuming it's written to some file since that colour is accessible by rt even after mged is closed.) Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-27 17:51:29
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Database To the database, of course. That question was probably stupid. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-27 18:02:25
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Color In function "get_args" in opt.c, where the cmd line options are processed, I see that the operations specified from the command line are executed using the function "rt_do_cmd." To set background color, the string "set background=x/y/z" is passed to rt_do_cmd. Is there any such command/statement that would let me change the color of the current rendered object from rt_do_cmd? Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-28 08:59:29
|
Hi, Sean has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: close questions Your questions are basically asking "where's the code that does this for me?" We never said there was. ;) The task could involve the rt_do_cmd() if you like, but it's enough work to figure out how to set the color (see what some of the ged functions (src/libged/*.c) related to color do in their implementation). Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-28 12:40:07
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: So... This task only involves adding that option support, but not doing anything with it? Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-29 05:52:42
|
Hi, Sean has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Nope You've got it backwards. The task is to figure out how to set a color (in code, on an object, NOT how to read a value from a user), set a temporary color, and show that it works. The task description uses a -O option as an example because the user will obviously need to specify a color somehow. Reading a color from the user that way is trivially easy if our -C option is used as an example. The work, though, is to FIGURE OUT how to set object colors. Try to render all objects as red (255/0/0) regardless of their default/set color, for example. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-29 12:04:46
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Task Claimed I would like to work on this task. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-29 12:37:09
|
Hi, Andrei Popescu has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Task Assigned This task has been assigned to Skriptkid. You have 72 hours to complete this task, good luck! Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-29 16:03:13
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: ged What is the structure "ged" used for? Does it need to be fiiled with certain values before being passed to the GED functions?(Functions such as GED_CHECK_DATABASE_OPEN, GED_DB_PUT_INTERNAL, etc) Also, what does the function RT_CK_COMB do? Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-29 16:35:33
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Check It's not complete, but I just wanted to know if I've understood the option part right. Also, there's a change I've made to a break statement, which I'm pretty sure was an error,(unless it was where it was for a purpose.) Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-29 16:36:02
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Ready for review The work on this task is ready to be reviewed. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-29 18:05:28
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Bad option? Why does that error arise? Do I have to the add the option somewhere else also? Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-29 18:51:16
|
Hi, Sean has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: man getopt You need to add your option to the bu_getopt option string. See the getopt man page. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-29 18:51:46
|
Hi, Sean has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: 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 requirements needed to complete this task, submit your work again and mark the task as complete once you re-submit your work. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-29 22:16:24
|
Hi, Sean has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: ged structure The GED structure represents a geometry database editing context. If you run "ged_draw()" for example, the GED structure contains the information about which .g file to read from. The RT_CK_COMB() is to ensure that the a given combination structure is valid memory-wise. There are macros like that for nearly all structures to validate their memory and detect corruption quickly. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-30 05:29:48
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: opt #define GETOPT_STR\ ".:,:@:a:b:c:d:e:f:g:h:ij:k:l:n:o:p:q:rs:tu:v:w:x:A:BC:D:E:F:G:H:IJ:K:MN:O:P:Q:RST:U:V:WX:!:+:" to #define GETOPT_STR\ ".:,:@:a:b:c:d:e:f:g:h:ij:k:l:n:o:p:q:rs:tu:v:w:x:z:A:BC:D:E:F:G:H:IJ:K:MN:O:P:Q:RST:U:V:WX:!:+:" But the option '-z' or '-zoptarg' still don't work. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-30 05:57:45
|
Hi, Sean has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: getopt isn't the main goal It's a little concerning that you can't figure out how to properly add a new command-line option given there are dozens of other options to follow by example... That's because setting the color will require a fair bit more understanding of code. As for the getopt failure, you found the option string and seem to have added z: correctly. You also have to find where bu_getopt is called and make sure you handle the option there. That sais, again this is entirely secondary to the task -- you could frankly gang off of the -C color parsing first and see if you can change an object color instead of the background color. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-30 15:02:26
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: mater I understand the code, but it's just that I'm not able to completely understand where things are being processed and what certain BRL-CAD functions are doing. bu_getopt is called only in get_args, which is called in main.c. So the processing of flags is happening in opt.c I took a look at mater.c, which processes the mater command and saw how it sets the colour of the object. I could get a fair idea about the logic and what the functions are doing and I added a new function to opt.c, which is called by the -C flag. Problem is, this is still changing the background color. What's wrong, and more importantly what have I got right? Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-30 15:03:17
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Code It's in opt2.patch Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2012-12-30 16:52:15
|
Hi, Sean has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: progress! That's looking like progress! It looks like you're on the right track without a few exceptions. You did hook into the -C option and change the behavior there, so that's good for testing. If the background is still changing, though, you may not be recompiling/testing correctly since you clearly removed the code that called rt_do_cmd() which is what set the background global to the -C color. Put a bu_log() or printf() statement before change_color() (should use consistent spelling) and make sure you're running the right binary when you're testing. As for the change_color() command, that's a good first step looking at what the matter command does. Note, however, that it's not an acceptable final solution as that method involves updating the comb structure and then WRITING IT OUT TO DISK (via GED_DB_PUT_INTERNAL())... you'll need to find a method that doesn't require read/write access to the .g file and doesn't write to disk. Try to get it to work that way first, though, to make sure you're passing parameters correctly. I don't see you verify that argc/argv are actually geometry names and not other parameters, for example. Print argv to make sure it's actually geometry and not a file name or option while testing. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |
From: <no-...@go...> - 2013-01-01 11:01:58
|
Hi, Skriptkid has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202: Extension Need an extension. Sorry. Haven't been able to work on this for sometime. Greetings, The Google Open Source Programs Team --- You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202. |