Closed hayakawa16 closed 10 months ago
@hayakawa16 The quality gate passed!
There are 11 new code smells, we can't fix them all (some are cognitive complexity with the factory code) but some of them can be fixed.
Thanks Lisa! I will await dcuccia's review before proceeding.
Just wanted to post that I took an infile that I ran with master on 10/12/23 and reran it using this branch. The results are identical! It was a sanity check suggested by lmalenfant and gives us more confidence in our code modifications.
dcuccia, thank you for your review! I can easily modify the unit test names.
I like your ideas for future work. It always bothered me how many times a MC simulation is executed during the course of running the unit tests. If this could be optimized somehow, that would be great! In addition to your list, in our discussion on this issue we uncovered the need to analyze PopulateFromEnumerable and possibly update application to be column-major. Creating place holder issues for these features sounds good. Would you like to create them?
Kudos, SonarCloud Quality Gate passed!
Creating a draft PR so it goes through all of the checks. Will add reviewers next week.