Editing User:Bhollister/DevLogAug2015

From BRL-CAD

Warning: You are not logged in. Your IP address will be publicly visible if you make any edits. If you log in or create an account, your edits will be attributed to your username, along with other benefits.

The edit can be undone. Please check the comparison below to verify that this is what you want to do, and then save the changes below to finish undoing the edit.
Latest revision Your text
Line 26: Line 26:
 
** none label objects with symbols (only color change, etc. and NOT faces)
 
** none label objects with symbols (only color change, etc. and NOT faces)
 
** ''ev'' provides some direction
 
** ''ev'' provides some direction
* todo: may have to implement something like ''f_labelvert'' (in mged/overlay.c) but for face index labelling
+
* todo: may have to implement something like ''f_labelvert'' (in mged/overlay.c) but for face index labeling
 
** OR have users specify verts for face selection (probably not preferable)
 
** OR have users specify verts for face selection (probably not preferable)
* updated design docs reflecting recent integrated patches
+
* updated design docs reflected recent integrated patches
  
 
non-brlcad expenditures...
 
non-brlcad expenditures...
  
* spent time in phone conversation negotiating / considering a job offer
+
* spent time in phone conversation negotiating / considering on job offer
  
 
= Wed, August 5, 2015 =
 
= Wed, August 5, 2015 =
 
* implementing new mged cmd called ''labelface'' need for kill F
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/cdbe653c24e26fea050289d6f6d9fa220931927f commit #1] added placeholder for lableface mged cmd
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/db9a7aa52d7d3fc8c8448b99aba2ab9fc7b3aa83 commit #2]  traversing model to collect a list of faces to label. work in progress...
 
  
 
= Thurs, August 6, 2015 =
 
= Thurs, August 6, 2015 =
 
* ''labelface'' impl
 
** working to get list of faceuses from nmg model
 
** working to get center coord of face polygons
 
*** this will allow labels to be displayed at center of faces
 
 
non-brlcad expenditures...
 
 
* spend much of the day deciding between two employment offers
 
* chose post-doctoral research over lecturer position
 
  
 
= Fri, August 7, 2015 =
 
= Fri, August 7, 2015 =
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/35e5931d443219253c223f60e1bcf15a51192723 commit #1]  verified that faceuses are properly gathered in get_face_list. todo: use bu_list instead of array placeholder for faceuse list.
 
* todo: need to find how to calculate barycenter for face verts (there must be a utility function in brlcad)
 
* instead: ''very easy as a simple average of coords of face verts''
 
** where face labels will be placed using ''bn_vlist_3string()'' in ''rt_label_vlist_faces()''
 
  
 
= Sat, August 8, 2015 =
 
= Sat, August 8, 2015 =
 
n/a
 
  
 
= Sun, August 9, 2015 =
 
= Sun, August 9, 2015 =
 
n/a
 
  
 
= Mon, August 10, 2015 '''''Week 12 (of 14)''''' =
 
= Mon, August 10, 2015 '''''Week 12 (of 14)''''' =
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/b230dec94cb5358f0337d8c7f5a2f5b588bc3180  commit #1] able to label faceuses at min pt of their bounding box. faceuse index used for label. todo: need to plot label at center of face. also need to find way to show both front / back facing faceuses. right now they are overdrawn. placeholder array causes seg fault when dereferencing empty faceuses in array.
 
*[https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/9efa36cfadd3e3cc8522f0489c6b4bfcbd3f6224 commit #2] using struct face as replacement to struct faceuse 
 
* todo: use center of face for face index placement
 
* since there are two facesuses per face, best to use index of face for label
 
* see:
 
 
<pre>
 
 
/**
 
* Note: there will always be exactly two faceuse's using a face.  To
 
* find them, go up fu_p for one, then across fumate_p to other.
 
*/
 
struct face {
 
    struct bu_list l; /**< @brief faces in face_g's f_hd list */
 
    struct faceuse *fu_p; /**< @brief Ptr up to one use of this face */
 
    union {
 
uint32_t *magic_p;
 
struct face_g_plane *plane_p;
 
struct face_g_snurb *snurb_p;
 
    } g; /**< @brief geometry */
 
    int flip; /**< @brief !0 ==> flip normal of fg */
 
    /* These might be better stored in a face_a (not faceuse_a!) */
 
    /* These are not stored on disk */
 
    point_t min_pt; /**< @brief minimums of bounding box */
 
    point_t max_pt; /**< @brief maximums of bounding box */
 
    long index; /**< @brief struct # in this model */
 
};
 
 
</pre>
 
  
 
= Tues, August 11, 2015 =
 
= Tues, August 11, 2015 =
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/49f6daaddfd8b6630832c48c7de0f730aa136175 commit #1] now taking avg of min and max bounding extents. works great for label placement.
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/aa0082b0e10a15d256b59ad944bf9313154a7947 commit #2] rt_label_vlist_faces caught in infinite loop. face list element had forw / back points to itself. todo: fix.
 
  
 
= Wed, August 12, 2015 =
 
= Wed, August 12, 2015 =
 
* added [http://brlcad.org/wiki/MGED_CMD_labelface labelface] documentation
 
* added patch [https://sourceforge.net/p/brlcad/patches/403/ 403] for ''labelface'' cmd
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/2a9d3a7077547a6fe72b138ffe3d70467ddcae31 commit #1] fixed problem with adding same face struct to bu_list - which caused self-referential node (forw/back pointers pointing to node itself) in list
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/2f2d1ce261a5798390937ac8d334d5bf00b84fd8 commit #2] added help lookup for 'lableface'
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/5d69a16a456b2fbd607a9e9193b86aca4c8ffb15 commit #3] removed accidental redundant error checking
 
* '''todo: implement 'kill F' next, using the indices provided in display by [http://brlcad.org/wiki/MGED_CMD_labelface labelface]'''
 
  
 
= Thurs, August 13, 2015 =
 
= Thurs, August 13, 2015 =
 
* submitted patch [https://sourceforge.net/p/brlcad/patches/404/ 404] for 'kill F'
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/144539694228226172afa30c573a08688e4a369c commit #1] added boiler-plate for ged_nmg_kill_f cmd 
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/1dc5b185b818bdaa88015a53c6001852aa0cd38f commit #2]  using nmg_kfu(). loops / edges still present after faceuse / face removal. todo: either verify that face is removed by testing with solid face rendering, or remove loops / edges along with face...
 
* Documentation updated: http://brlcad.org/wiki/MGED_CMD_nmg#Subcommands
 
* todo: impl 'make V' as proposed
 
  
 
= Fri, August 14, 2015 =
 
= Fri, August 14, 2015 =
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/bf83734208abbc9f4d25a748580d1d908428b32f commit #1] boiler-plate for move V mged cmd
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/777e9e006d454f851bd7f770c35807a093f32679 commit #2] move V cmd implementation. More or less a variation on kill V.
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/7476b5b7f4e95c6c2ef73c1dc3c0567f5565391e commit #3] added check for cmd argument length. should not exceed two coords.
 
* submitted patch [https://sourceforge.net/p/brlcad/patches/405/ 405] for move V nmg subcommand
 
* updated docs
 
** [[MGED_CMD_nmg| nmg]]
 
** [[MGED_CMD_labelvert|labelvert]] for proposed -i (index) option
 
*** this will allow for CLI specification with make F
 
* updated [[User:Bhollister/Proposal#Project_Schedule| project schedule]] to reflect remaining work for next week, i.e. make F, make V, labelvert -i
 
  
 
= Sat, August 15, 2015 =
 
= Sat, August 15, 2015 =
 
n/a
 
  
 
= Sun, August 16, 2015 =
 
= Sun, August 16, 2015 =
 
n/a
 
  
 
= Mon, August 17, 2015 '''''Week 13 (of 14)''''' =
 
= Mon, August 17, 2015 '''''Week 13 (of 14)''''' =
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/0e1b539601a3167dcce14d98601612815502ae66 commit #1] beginning of vertex index labelvert -i option.
 
* crashing in bn_vlist_3string with MAT_DELTAS_VEC( xlate_to_origin, origin );
 
* todo: fix above issue
 
  
 
= Tues, August 18, 2015 =
 
= Tues, August 18, 2015 =
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/b7d06366f26525d4d28cbc674cf1579dd90b1c25 commit #1] labelvert -i option working. however, code uses new struct vtxlabel to avoid potential lists already used by nmg datastructure (struct vertex). also, there is no freeing of label list. todo: either use struct vertex instead if there is no list in nmg structure using struct vertex, or free list appropriately after usage.
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/05790259d2ecd8347c287494dba8895fb335755e commit #2] struct vertex does not have a l bu_list. do not want to alter nmg types. thus, now freeing struct vtxlabel list after use.
 
* todo: will cleanup above commits (if needed) and submit as patch tomorrow
 
  
 
= Wed, August 19, 2015 =
 
= Wed, August 19, 2015 =
 
* posted patch: [https://sourceforge.net/p/brlcad/patches/412/ 412]
 
* posted patch: [https://sourceforge.net/p/brlcad/patches/409/ 409]
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/c54ea02a09eb78003e5d1dc43b0e4aadb3fafcf7 commit #1] now freeing heap memory for vertex label list
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/2f9576729eb60133ba1ebcfddacc3dba6e0fb7a4 commit #3] using bu_malloc and bu_free now.
 
* make V subcommand:
 
** [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/931eb783599ff956b69ca594d2912e72376e373b commit #2] boiler-plate for make V cmd
 
** [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/59760062ff10d5cca5df14af2693ac6a49a7502f commit #4] getting casting error with return variable from nmg_mvvu()
 
  
 
= Thurs, August 20, 2015 =
 
= Thurs, August 20, 2015 =
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/ed9acc12b1ee48f486ab1eb2d8261ede791e87ae commit #1]  apparently we don't want to create a new loopuse for each new isolated vertex. can only add one vertex per shell. so, todo: use nmg_msv, for each new vertex + shell.
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/a424344f653a9ef0ca4f50f36d7db6137d4f3ffa commit #2]  working and cleaned up ged_nmg_kill_v. now adding a new shell when needed for a new vertex coord.
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/248c05b10d46c007cb208d2517afe79745f2e871 commit #3] added info about move V subcommand.
 
* patch hasn't build properly with merge of latest changes (nmg_make_v.c) to trunk...working on it. todo: submit patch tomorrow??
 
** build dies @ 97 percent for linking ../../../../bin/step-g?
 
** [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/801d4143aa4cf6935163ef8b22a178aa46b4f9dc commit #4]  simplified code to not add into existing shell. this was causing an undef linking error due to header issues with nmg_mvvu().
 
*  'make V' nmg subcommand patch submitted, see...
 
** patch [https://sourceforge.net/p/brlcad/patches/414/ 414] Adds new vertices specified by their coords. Each vertex is added to a new shell in the NMG object.
 
* todo: will see if I can get 'make F' working and submitted by COB tomorrow...
 
* todo: minor updates to mged nmg docs as well.
 
* todo: fix up -i option patch
 
  
 
= Fri, August 21, 2015 '''''FIRM PENCILS DOWN''''' =
 
= Fri, August 21, 2015 '''''FIRM PENCILS DOWN''''' =
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/688c64e1d20b8bbd0d24ee727fcd241a47b8a151 commit #1] moved vtxlabel struct def to rt/vlist.h
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/d058369dc0ec7815364df314bb7d2e2d17b099be commit #2] boiler-plate for ged_nmg_make_F subcommand
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/ff9ea3190265b3aafdf6c16b57ae10a03ad67be3 commit #3] added find verts function stub with nmg traversal
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/4a6bea01bef8d6b84d58111d2547a6c85935f10e commit #4] make F working for three vertices specified by labelvert -i index labels
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/fdd8abb11d46c31d3294a7aaff02a72bfeadf3ec commit #5] first take on variabel command line with F
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/12bb0c06cfa8b8dc28aa25fcfbee6e4329b552db commit #6]  appears to work for multiple F
 
* todo: will submit patch for move F over weekend after more testing...
 
  
 
= Sat, August 22, 2015 =
 
= Sat, August 22, 2015 =
 
* [https://github.com/behollis/brlcad-svn-rev65072-gsoc2015/commit/c2e49d80e0776ea6da4358a345a04f56e0088b90 commit #1] processing mulitple faces on command line for 'make F' subcommand
 
* submitted patch [https://sourceforge.net/p/brlcad/patches/415/ 415] for 'make F' subcommand
 
* finished final evaluation on google-melange
 
* updated [http://brlcad.org/wiki/MGED_CMD_nmg mged nmg subcommands] documentation
 
  
 
= Sun, August 23, 2015 =
 
= Sun, August 23, 2015 =
 
n/a
 
  
 
= Mon, August 24, 2015 '''''Week 14 (of 14)''''' =
 
= Mon, August 24, 2015 '''''Week 14 (of 14)''''' =
 
n/a
 
  
 
= Tues, August 25, 2015 =
 
= Tues, August 25, 2015 =
 
n/a
 
  
 
= Wed, August 26, 2015 =
 
= Wed, August 26, 2015 =
 
n/a
 
  
 
= Thurs, August 27, 2015 =
 
= Thurs, August 27, 2015 =
 
n/a
 
  
 
= Fri, August 28, 2015 '''''FINAL EVALS DUE''''' =
 
= Fri, August 28, 2015 '''''FINAL EVALS DUE''''' =
 
n/a
 

Please note that all contributions to BRL-CAD may be edited, altered, or removed by other contributors. If you do not want your writing to be edited mercilessly, then do not submit it here.
You are also promising us that you wrote this yourself, or copied it from a public domain or similar free resource (see BRL-CAD:Copyrights for details). Do not submit copyrighted work without permission!

To edit this page, please answer the question that appears below (more info):

Cancel Editing help (opens in new window)