Difference between revisions of "User:Bhollister/DevLogJuly2015"

From BRL-CAD
(→‎Mon, July 20, 2015: Start of Week 9 (of 14))
(→‎Tues, July 21, 2015)
Line 178: Line 178:
  
 
= Tues, July 21, 2015 =
 
= Tues, July 21, 2015 =
 +
 +
* since arbs have vertex labelling
 +
** may want to rethink (at least) how ''kill V'' subcommand
 +
** vertex numbering '''already''' works with nmg, but there are some scaling issues
 +
** i.e. vertex labels are not located right on vertices but too far from them
 +
* new create / kill subcommands should be:
 +
** consistent with modal behavior of mged command line
 +
** i.e. only in "edit" mode do we use create / kill nmg model parts
 +
* todo: need to investigate further libged/edit.c
 +
* assuming we color faces / verts diff colors:
 +
** code proposal: check if in "edit" mode, then:
 +
*** either add new function and call it, i.e. something like 'ged_draw_guts_edit_mode()'
 +
*** or add to already large ged_draw_guts() with conditional logic
 +
*** there is only a single ''struct _ged_client_data'' used for entire model
 +
*** potentially needs to be revamped, since we need it to change per model ''part''
 +
*** warning: could be extensive and therefore may have to use ''existing'' labelling system
 +
* found this at the end of /include/ged.h. should be useful.
 +
<pre>
 +
/***************************************
 +
* Conceptual Documentation for LIBGED *
 +
***************************************
 +
*
 +
* Below are developer notes for a data structure layout that this
 +
* library is being migrated towards.  This is not necessarily the
 +
* current status of the library, but rather a high-level concept for
 +
* how the data might be organized down the road for the core data
 +
* structures available for application and extension management.
 +
*
 +
* struct ged {
 +
*  dbip
 +
*  views * >-----.
 +
*  result()      |
 +
* }              |
 +
*                |
 +
* struct view { <-'
 +
*  geometry * >------.
 +
*  update()          |
 +
* }                  |
 +
*                    |
 +
* struct geometry { <-'
 +
*  display lists
 +
*  directory *
 +
*  update()
 +
* }
 +
*
 +
*/
 +
</pre>
  
 
= Wed, July 22, 2015 =
 
= Wed, July 22, 2015 =

Revision as of 19:23, 21 July 2015


Wed, July 1, 2015: Week 6 (of 14)

  • commit #1: filling out vertex info with structures to avoid errors with incomplete data structures
  • getting crash in nmg_mvu on line 365: BU_LIST_APPEND(...); need to investigate.
  • use case for above: 'make modelname nmg', then 'nmg cmface modelname coords'
  • rechecked patch https://sourceforge.net/p/brlcad/patches/387/
  • verified by compiling with STRICT enabled. posted build.log to sourceforge.
  • problems with compilation apparently not associated with changes. builds fine with STRICT disabled.

Thurs, July 2, 2015

  • Appointment with retinal specialist from 9:30am to Noon PST.
  • Not be able to work today due to pupil dilation.

Fri, July 3, 2015

  • Midterm summary located at link
  • still investigating nmg_mvu issue for ged_nmg_cmface subcommand:
    • likely that the vertexuse is being improperly treated for loop creation.
    • should there be a vertexuse present in the model for the new loop (prior to nmg_mvu call from nmg_cmface)??

Sat, July 4, 2015

  • n/a

Sun, July 5, 2015

  • n/a

Mon, July 6, 2015: Start of Week 7 (of 14)

  • commit #1 fixed issue with missing bu_list in vertex struct
    • while commit #1 provides a "working" call to nmg_cface with three verts, the face is not visible in the render window??
    • ged_nmg_cface appears to make the proper calls...
    • once problem is found with rendered model, need to extend for n verts (should be trivial)
  • commit #2 attempt to fix issue with ged_nmg_cmface crash in nmg_info/nmg_findeu(); in loop

Tues, July 7, 2015

non-brlcad time expenditures...

  • spent some time applying for positions

Wed, July 8, 2015

  • investigated put / get database commands.
    • should illuminate sequence of calls needed by subcommands for nmg api calls
    • posted related questions in irc
  • todo: use info from above to fix mged calls implemented in libged/ged_nmg_cmface.c
  • commit #1: null commit to synchronize to github from nonlocal repos

non-brlcad time expenditures...

  • oil changed for car; 3 hour wait!
  • wrote essay, etc. for instructor position application

Thurs, July 9, 2015

  • investigated get / put nmg object creation...
    • need to follow the code from this sequence of calls using 'put' CLI:
      • ged_put()
      • rt_nmg_make()
      • rt_nmg_adjust()
    • above sequence of calls creates an nmg object and adds verts / faces
  • todo: revamp *ged_cmface* using preceding process.
  • note: apparently using nmgmodel.c was not the best example of manifold nmg object creation!
  • todo: once ged_cmface() properly impl and submitted as patch, can move on the spec for high-level subcommand list; perhaps vert / face removal subcommand CLI routines are the easy to start with??

non-brlcad time expenditures...

  • VDA 2016 template and reformat of paper for submission
  • 10 job applications submitted

Fri, July 10, 2015

  • commit #1 using method from rt_nmg_adjust. still some issues with calls to nmg_cmface. soon to be resolved.
  • appears that vert structs are not being filled out correctly and thus crash in cmface; need to test with known "working" args

non-brlcad time expenditures...

  • applications
  • car in garage again for brake pads
  • car will be in garage again next week for air bag recall (ford mustang)

Sat, July 11, 2015

n/a

Sun, July 12, 2015

n/a

Mon, July 13, 2015: Start of Week 8 (of 14)

  • commit #1 cmface now working. using rt_nmg_adjust as model for impl.
  • commit #2 added variable number of verts for CLI.
  • commit #3 check for minimum number of verts for manifold face
  • cmface patch 390 submitted

Tues, July 14, 2015

  • mm (make nmg object model) subcommand patch 391 submitted
  • updated patch 390 with build fix and code cleanup
  • commit #1 cleaned up loop var. removed unused fastf_t declaration
  • found potential build problems when BRLCAD_ENABLE_STRICT=YES. noted in patch 390 reply (opennurbs lib)
  • todo:
    • impl face kill routine as subcommand
    • design remaining subcommands (will be higher-level than internal nmg api)
    • post usage instructions for simple box creation use case using subcommand 'cmface'
    • will need to determine which routines that have been submitted in patch 387 that need to be removed

Wed, July 15, 2015

  • re-tested with 'strict' build options - works!
  • updated schedule to include new designs
  • created MGED_CMD_nmg design doc
  • proposed new subcommands:
sface
Selects a face by highlighting it. Subsequent commands for manipulation or removal act on the selected face. By repeating this call, the currently selected face is advanced to the next face in the NMG object of the first encountered shell and region. If an (optional) point is specified, the subcommand finds the closest face to the point and selects that face instead.
deselect
Deselects all prior selected parts of the nmg object.
kface
Removes the geometry and faceuse of the currently selected face.
  • todo: need to get mentor(s) advice and okay before implementing proposed new subcommands
  • todo: submit patch to remove calls to low-level api that are not needed
  • todo: update top-level command 'nmg' to reflect help / changes

Thurs, July 16, 2015

  • investigated ways to draw geometry different colors
    • traced ged_draw, etc.
    • this will be needed in order to highlight selected face via 'sface' subcommand
  • updated 'nmg' command design page

non-brlcad expenditures...

  • applied for more positions
  • answered questions from interviewer (email)

Fri, July 17, 2015

  • read / posted a response to mailing list about command design
  • investigated design options (see my response)
  • investigated implications of various implementations (see my response)
  • todo: polish declarative syntax for 'creation' subcommands per Sean's suggestions
  • added 'nmg' to mged cmds

non-brlcad expenditures...

  • researched university locations for jobs
  • found out that I have jury duty in early august - need to postpone if possible

Sat, July 18, 2015

n/a

Sun, July 19, 2015

n/a

Mon, July 20, 2015: Start of Week 9 (of 14)

  • updated proposed subcommands
    • need to find way of marking model parts
    • need to allow users to select desired model parts through model labels in subcommand (see previous link for design ideas)
  • investigated color setting with mged
    • only found info on comb_color which doesn't appear relevant
    • need to find simple way to set real-time geometry window colors (down to an opengl state function possibly?? and trace that back up to the higher-level call)
    • libged/draw.c, _ged_drawtrees() sets the wireframe color...need to leverage this functionality for labelling / marking geometry!

non-brlcad expenditures...

  • interview scheduled for 10 AM PST (Skype)

Tues, July 21, 2015

  • since arbs have vertex labelling
    • may want to rethink (at least) how kill V subcommand
    • vertex numbering already works with nmg, but there are some scaling issues
    • i.e. vertex labels are not located right on vertices but too far from them
  • new create / kill subcommands should be:
    • consistent with modal behavior of mged command line
    • i.e. only in "edit" mode do we use create / kill nmg model parts
  • todo: need to investigate further libged/edit.c
  • assuming we color faces / verts diff colors:
    • code proposal: check if in "edit" mode, then:
      • either add new function and call it, i.e. something like 'ged_draw_guts_edit_mode()'
      • or add to already large ged_draw_guts() with conditional logic
      • there is only a single struct _ged_client_data used for entire model
      • potentially needs to be revamped, since we need it to change per model part
      • warning: could be extensive and therefore may have to use existing labelling system
  • found this at the end of /include/ged.h. should be useful.
/***************************************
 * Conceptual Documentation for LIBGED *
 ***************************************
 *
 * Below are developer notes for a data structure layout that this
 * library is being migrated towards.  This is not necessarily the
 * current status of the library, but rather a high-level concept for
 * how the data might be organized down the road for the core data
 * structures available for application and extension management.
 *
 * struct ged {
 *   dbip
 *   views * >-----.
 *   result()      |
 * }               |
 *                 |
 * struct view { <-'
 *   geometry * >------.
 *   update()          |
 * }                   |
 *                     |
 * struct geometry { <-'
 *   display lists
 *   directory *
 *   update()
 * }
 *
 */

Wed, July 22, 2015

Thurs, July 23, 2015

Fri, July 24, 2015

Sat, July 25, 2015

Sun, July 26, 2015

Mon, July 27, 2015: Start of Week 10 (of 14)

Tues, July 28, 2015

Wed, July 29, 2015

Thurs, July 30, 2015

Fri, July 31, 2015

Retrieved from "https://brlcad.org/w/index.php?title=User:Bhollister/DevLogJuly2015&oldid=9066"