Difference between revisions of "User:Sharan.nyn/GSoC18/Report"
From BRL-CAD
Sharan.nyn (talk | contribs) |
Sharan.nyn (talk | contribs) (→rtcheck and gqa based on the libanalyze API) |
||
(58 intermediate revisions by the same user not shown) | |||
Line 1: | Line 1: | ||
− | == ''' Report : Check Command and Overlaps Tool ''' == | + | == ''' Project Report : Check Command and Overlaps Tool ''' == |
− | The work done for the project can be split into the following parts: | + | * The work done for the project can be split into the following parts: |
− | === libanalyze function -- analyze_overlaps === | + | === libanalyze function -- analyze_overlaps and libged command -- check_overlaps === |
+ | NOTE: This was later replaced with better code, check it out [https://brlcad.org/wiki/User:Sharan.nyn/GSoC18/Report#new_libanalyze_API_and_check_command_that_uses_this_API here] | ||
+ | * The main goal behind adding a check_overlaps command was that libged's rtcheck command acted as a wrapper for the rtcheck program, and ran the rtcheck program with execvp command -- this was not desired as the execution was done in a different process. | ||
+ | * I added the check_overlaps command which had the same options as rtcheck command and the logic behind shooting of rays was done in libanalyze. | ||
+ | * The overlaps were processed on libged side with callback functions. | ||
+ | * The submitted patches were committed by Daniel -- | ||
+ | ** [https://sourceforge.net/p/brlcad/code/70995/ r70995] [https://sourceforge.net/p/brlcad/code/71008/ r71008] [https://sourceforge.net/p/brlcad/code/71042/ r71402] | ||
+ | ** [https://sourceforge.net/p/brlcad/code/71050/ r71050] -- this was later reverted by me ([https://sourceforge.net/p/brlcad/code/71059/ r71059]) | ||
+ | * I was given commit access to the repo :) and I did the following commits: | ||
+ | ** [https://sourceforge.net/p/brlcad/code/71063/ r71063] [https://sourceforge.net/p/brlcad/code/71065/ r71065] [https://sourceforge.net/p/brlcad/code/71066/ r71066] [https://sourceforge.net/p/brlcad/code/71067/ r71067] [https://sourceforge.net/p/brlcad/code/71103/ r71103] [https://sourceforge.net/p/brlcad/code/71108/ r71108] | ||
+ | * Documentation work related to check_overlaps -- [https://sourceforge.net/p/brlcad/patches/497/ #497] | ||
− | === | + | === rtcheck based on libanalyze's analyze_overlaps function === |
+ | NOTE: This was later replaced with better code, check it out [https://brlcad.org/wiki/User:Sharan.nyn/GSoC18/Report#new_libanalyze_API_and_check_command_that_uses_this_API here] | ||
+ | * Since the logic for shooting the rays was in libanalyze function analyze_overlaps. | ||
+ | * My next task was to refactor rtcheck program to use analyze_overlaps. | ||
+ | * [https://sourceforge.net/p/brlcad/patches/495/ #495] was submitted. | ||
− | === | + | === libanalyze function for grid generation === |
+ | * Function that sets the ray starting point and ray directions in a grid form was added. | ||
+ | * This helped reduce the parameters passed to analyze_overlaps -- all grid related functions and data was passed in two variables. | ||
+ | * The following commits were done - | ||
+ | ** [https://sourceforge.net/p/brlcad/code/71117/ r71117] [https://sourceforge.net/p/brlcad/code/71118/ r71118] [https://sourceforge.net/p/brlcad/code/71160/ r71160] | ||
+ | * Later triple grid support and refining was added - | ||
+ | ** [https://sourceforge.net/p/brlcad/code/71200/ r71200] [https://sourceforge.net/p/brlcad/code/71201/ r71201] [https://sourceforge.net/p/brlcad/code/71204/ r71204] | ||
=== overlaps tool based on check.sh === | === overlaps tool based on check.sh === | ||
+ | * One of the main goals of the project was to remove the need of the check.sh because it was a bash script it meant it didn’t work for windows system. | ||
+ | * The same functionality was added with help of a .tcl file. | ||
+ | * The following commits were done - | ||
+ | ** [https://sourceforge.net/p/brlcad/code/71106/ r71106] [https://sourceforge.net/p/brlcad/code/71107/ r71107] [https://sourceforge.net/p/brlcad/code/71149/ r71149] [https://sourceforge.net/p/brlcad/code/71150/ r71150] | ||
+ | * How it looked: [https://brlcad.org/wiki/File:Overlaps_tool1.gif overlaps_tool.gif] | ||
− | + | * I started to work on better object selection to make it like the prototype image I created during the application period ([http://brlcad.org/wiki/File:CheckGUI.png image]) | |
+ | * The following commits were done - | ||
+ | ** [https://sourceforge.net/p/brlcad/code/71151/ r71151] [https://sourceforge.net/p/brlcad/code/71152/ r71152] [https://sourceforge.net/p/brlcad/code/71153/ r71153] [https://sourceforge.net/p/brlcad/code/71154/ r71154] [https://sourceforge.net/p/brlcad/code/71155/ r71155] | ||
+ | * Better object selection: [https://brlcad.org/wiki/File:Overlaps_tool2.gif better_object_sel.gif] | ||
+ | * Final image: | ||
+ | |||
+ | * [[File:Overlaps_tool_final.png|693px]] | ||
+ | |||
+ | * Documentation was done -- [https://sourceforge.net/p/brlcad/code/71498/ r71498] | ||
=== new libanalyze API and check command that uses this API === | === new libanalyze API and check command that uses this API === | ||
+ | * The goal behind this task was to combine the geometry analysis tools -- rtcheck, gqa and glint into one command. | ||
+ | * A libanalyze API was discussed that contained the backend logic of these commands in one place. | ||
+ | * The check command has sub-commands for - overlaps, mass, volume, surface area, gaps, adjacent air, exposed air and unconfined air. | ||
+ | * This task involved many commits. For list of commits please refer to ''week 10 - week 12'' in my daily log available here -[http://brlcad.org/w/index.php?title=User:Sharan.nyn/GSoC18/Log#Week_10 Log] | ||
+ | |||
+ | * Since I added a better command for checking overlaps (''check overlaps [options]''). I removed the old check_overlaps and analyze_overlaps function. | ||
+ | * And adapted the code for the new check overlaps command. For commits refer to week 13's log - [http://brlcad.org/w/index.php?title=User:Sharan.nyn/GSoC18/Log#Week_13 Week-13] | ||
+ | |||
+ | * I also wrote some bash scripts to compare the outputs of check vs rtcheck/gqa -- [https://sourceforge.net/p/brlcad/code/71355/ r71355] | ||
+ | * Documentation and man page for check command was added - [https://sourceforge.net/p/brlcad/code/71374/ r71374] [https://sourceforge.net/p/brlcad/code/71393/ r71393] | ||
+ | |||
+ | === rtcheck and gqa based on the libanalyze API === | ||
+ | * Since the backend logic of gqa and rtcheck lives in libanalyze/api.c. The duplicated code in gqa and rtcheck can be removed. | ||
+ | * rtcheck and gqa were changed to use the libanalyze functions. | ||
+ | * work related to rtcheck and gqa can be found in the public Google Drive Folder linked below | ||
+ | |||
+ | === Files === | ||
+ | This public Google Drive folder contains all the files which I worked on almost exclusively during the coding period - | ||
+ | * [https://drive.google.com/drive/folders/1WueiX-Cg21SpQ4lYmkhO6Qo40fwZSaaH?usp=sharing saran_narayan-gsoc18-brlcad] | ||
+ | |||
+ | === Logs === | ||
+ | * [http://brlcad.org/w/index.php?title=User:Sharan.nyn/GSoC18/Log Daily Logs] | ||
+ | * The log also contains links to all my commits and patches. |
Latest revision as of 08:51, 14 August 2018
Contents
- 1 Project Report : Check Command and Overlaps Tool
- 1.1 libanalyze function -- analyze_overlaps and libged command -- check_overlaps
- 1.2 rtcheck based on libanalyze's analyze_overlaps function
- 1.3 libanalyze function for grid generation
- 1.4 overlaps tool based on check.sh
- 1.5 new libanalyze API and check command that uses this API
- 1.6 rtcheck and gqa based on the libanalyze API
- 1.7 Files
- 1.8 Logs
Project Report : Check Command and Overlaps Tool [edit]
- The work done for the project can be split into the following parts:
libanalyze function -- analyze_overlaps and libged command -- check_overlaps[edit]
NOTE: This was later replaced with better code, check it out here
- The main goal behind adding a check_overlaps command was that libged's rtcheck command acted as a wrapper for the rtcheck program, and ran the rtcheck program with execvp command -- this was not desired as the execution was done in a different process.
- I added the check_overlaps command which had the same options as rtcheck command and the logic behind shooting of rays was done in libanalyze.
- The overlaps were processed on libged side with callback functions.
- The submitted patches were committed by Daniel --
- I was given commit access to the repo :) and I did the following commits:
- Documentation work related to check_overlaps -- #497
rtcheck based on libanalyze's analyze_overlaps function[edit]
NOTE: This was later replaced with better code, check it out here
- Since the logic for shooting the rays was in libanalyze function analyze_overlaps.
- My next task was to refactor rtcheck program to use analyze_overlaps.
- #495 was submitted.
libanalyze function for grid generation[edit]
- Function that sets the ray starting point and ray directions in a grid form was added.
- This helped reduce the parameters passed to analyze_overlaps -- all grid related functions and data was passed in two variables.
- The following commits were done -
- Later triple grid support and refining was added -
overlaps tool based on check.sh[edit]
- One of the main goals of the project was to remove the need of the check.sh because it was a bash script it meant it didn’t work for windows system.
- The same functionality was added with help of a .tcl file.
- The following commits were done -
- How it looked: overlaps_tool.gif
- I started to work on better object selection to make it like the prototype image I created during the application period (image)
- The following commits were done -
- Better object selection: better_object_sel.gif
- Final image:
- Documentation was done -- r71498
new libanalyze API and check command that uses this API[edit]
- The goal behind this task was to combine the geometry analysis tools -- rtcheck, gqa and glint into one command.
- A libanalyze API was discussed that contained the backend logic of these commands in one place.
- The check command has sub-commands for - overlaps, mass, volume, surface area, gaps, adjacent air, exposed air and unconfined air.
- This task involved many commits. For list of commits please refer to week 10 - week 12 in my daily log available here -Log
- Since I added a better command for checking overlaps (check overlaps [options]). I removed the old check_overlaps and analyze_overlaps function.
- And adapted the code for the new check overlaps command. For commits refer to week 13's log - Week-13
- I also wrote some bash scripts to compare the outputs of check vs rtcheck/gqa -- r71355
- Documentation and man page for check command was added - r71374 r71393
rtcheck and gqa based on the libanalyze API[edit]
- Since the backend logic of gqa and rtcheck lives in libanalyze/api.c. The duplicated code in gqa and rtcheck can be removed.
- rtcheck and gqa were changed to use the libanalyze functions.
- work related to rtcheck and gqa can be found in the public Google Drive Folder linked below
Files[edit]
This public Google Drive folder contains all the files which I worked on almost exclusively during the coding period -
Logs[edit]
- Daily Logs
- The log also contains links to all my commits and patches.