From 5099e41685f8840678d53994aacc57683249d8ee Mon Sep 17 00:00:00 2001 From: shilpiprd Date: Thu, 31 Aug 2023 16:31:55 +0530 Subject: [PATCH 1/2] Pushing Week14 BlogPost --- posts/2023/2023_08_28_Shilpi_Week14.rst | 27 +++++++++++++++++++++++++ 1 file changed, 27 insertions(+) create mode 100644 posts/2023/2023_08_28_Shilpi_Week14.rst diff --git a/posts/2023/2023_08_28_Shilpi_Week14.rst b/posts/2023/2023_08_28_Shilpi_Week14.rst new file mode 100644 index 00000000..1489e443 --- /dev/null +++ b/posts/2023/2023_08_28_Shilpi_Week14.rst @@ -0,0 +1,27 @@ +Writing Tests & Making Documentation: Week 13 +============================================= + +.. post:: August 28, 2023 + :author: Shilpi Prasad + :tags: google + :category: gsoc + + +What I did this week +~~~~~~~~~~~~~~~~~~~~ + +This week I fixed the test for the isotropic source of kurtosis so now it's working for all DTDs. I also created tests for the ``K_micro`` function. Initially while running the test, I got some errors which made me look deeper into the actual function. The error was that I was doing sqrt of some elements when actually I was supposed to calculate square of them. Also I was using a '1/5' factor which was not actually required. On fixing these issues, the overall map image of ``K_micro`` improved significantly. +Previously the multi-voxel test case was failing due to different eigenvalues in the isotropic total diffusion tensors simulations. Removing the eigenvalue assertion made the test pass, as verifying the kt, cvt, and evals values sufficed. +I also provided documentation to some functions in the test file such as ``_perpendicular_directions_temp_`` and ``from_qte_to_cti`` etc. +Also had to change the name of some functions to make it sound more relatable to what they were actually doing. +This week I almost finished with the CTI tutorial. The only thing remaining is to create a fetcher for the data so that all the users can download the data and use it. Currently the path given for the data retrieval is of my local system. +I also added some references and overall improved the wording and information in the tutorial. +I also finished up writing my final work report and get it reviewed by my mentors and then updated it. +Finally, before pushing the file onto the main PR, I cleaned up the code by removing all the extra comments and some unnecessary code and overall made sure that the entire code was following the pep8 standard. + +What is coming up Next Week +~~~~~~~~~~~~~~~~~~~~~~~~~~~ + +Now that almost everything is completed and all the unit tests are passing, I'll just focus on getting the PR merged. +At present, the merging of my PR is on hold as we're actively seeking a pair of reviewers who can conduct a thorough examination of the entire codebase and provide their expert feedback. +Additionally, I'll try to get the QTI simulation merged as well . From 1bdf415afac0ada6f9d30c3a65dc83532d95e04a Mon Sep 17 00:00:00 2001 From: shilpiprd Date: Thu, 31 Aug 2023 16:36:09 +0530 Subject: [PATCH 2/2] Fixing Week Number --- posts/2023/2023_08_28_Shilpi_Week14.rst | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/posts/2023/2023_08_28_Shilpi_Week14.rst b/posts/2023/2023_08_28_Shilpi_Week14.rst index 1489e443..7ee8df6d 100644 --- a/posts/2023/2023_08_28_Shilpi_Week14.rst +++ b/posts/2023/2023_08_28_Shilpi_Week14.rst @@ -1,5 +1,5 @@ -Writing Tests & Making Documentation: Week 13 -============================================= +Doing Final Touch-Ups: Week 14 +============================== .. post:: August 28, 2023 :author: Shilpi Prasad @@ -23,5 +23,6 @@ What is coming up Next Week ~~~~~~~~~~~~~~~~~~~~~~~~~~~ Now that almost everything is completed and all the unit tests are passing, I'll just focus on getting the PR merged. +I will also aim to achieve a 100% coverage report, thereby ensuring that every line of code is thoroughly tested. At present, the merging of my PR is on hold as we're actively seeking a pair of reviewers who can conduct a thorough examination of the entire codebase and provide their expert feedback. Additionally, I'll try to get the QTI simulation merged as well .