Difference between revisions of "User:Clouddrift/GSoC2014/Logs"
From BRL-CAD
< User:Clouddrift | GSoC2014
Clouddrift (talk | contribs) (→Week 1) |
Clouddrift (talk | contribs) (→Community Bounding Period) |
||
Line 3: | Line 3: | ||
'''Get the commit access''' | '''Get the commit access''' | ||
− | + | Submit two patches for nmg part of BRL-CAD. URLs as following. | |
+ | |||
+ | * [https://sourceforge.net/p/brlcad/patches/272/ Unit test for nmg_mk] | ||
+ | |||
+ | * [https://sourceforge.net/p/brlcad/patches/273/ Unit test for nmg_copy] | ||
'''Read the relavant Codes''' | '''Read the relavant Codes''' | ||
+ | |||
+ | Read and be familiar with nmg-related codes in BRL-CAD. | ||
+ | |||
+ | '''Acquired knowledge''' | ||
+ | |||
+ | * Keep code style consistent is important for Open Source Community. | ||
+ | |||
+ | * Comments greatly helps others to understand the function of routines and structs. Codes should be changed before fully tracking the call order of routines and totally understanding the comments of them. | ||
+ | |||
+ | * That's a really good ideas to submit one or two patch to know the coding convention of the community. As a chinese saying goes, sharpening your axe will not delay your job of cutting wood. | ||
== Work Perod == | == Work Perod == |
Revision as of 00:59, 24 May 2014
Community Bounding Period
Get the commit access
Submit two patches for nmg part of BRL-CAD. URLs as following.
Read the relavant Codes
Read and be familiar with nmg-related codes in BRL-CAD.
Acquired knowledge
- Keep code style consistent is important for Open Source Community.
- Comments greatly helps others to understand the function of routines and structs. Codes should be changed before fully tracking the call order of routines and totally understanding the comments of them.
- That's a really good ideas to submit one or two patch to know the coding convention of the community. As a chinese saying goes, sharpening your axe will not delay your job of cutting wood.
Work Perod
Week 1
Monday, May 19
OK.
Tuesday, May 20
OK.
Wednesday, May 21
Get the commit access for BRL-CAD successfully. What a day to celebrate! Now, I begin my work on a branch for NMG reorganization. Remove model and nmgregion struct, Then ready to fix all compilation errors.
Thursday, May 22
Rewrite nmg struct to fit BRL-CAD better. Remove model and nmgregion struct, Then change shell struct as following.
- remove member: l (bu_list);
- remove member: r_p (nmgregion pointer);
- add member: magic (uint32_t);
- add member: manifolds (char pointer);
- add member: maxindex (long);
Thursday, May 23
Fix the rest compilation errors due to fit the new nmg structure. The code files are corrected as following.
- nmg_ck.c;
- nmg_class.c;
- nmg_copy.c;
- nmg_eval.c;
- nmg_mk.c;
- nmg_info.c;
- nmg_plot.c;
- nmg_pr.c;
- nmg_tri.c;