Feature/42a inaccurate fdpm amplitude and phase derivative values with respect to mu a and mu s

[hayakawa16]

Draft of PR started so I can check code cleanup.

[hayakawa16]

I have fixed the code coverage percentage to 93.3% and now passes. I think the checks are not passing due to the new issues. I will work on those this week. The code can be tried at this point though. I would like to make sure I didn't miss anything with regards to the derivatives if possible.

[lmalenfant]

The quality gate failed for this branch, there are 4 code smells, I think 2 can be easily fixed, once we do that we can reassess the quality gate to see if it still fails, if it does we might be able to change the parameters until we can clean up the code more.

[hayakawa16]

@lmalenfant, would you possibly be able to fix those code smells? If not, I can do later this week.

[lmalenfant]

@hayakawa16 It passed! I'll start reviewing the PR now :)

[hayakawa16]

Great work @lmalenfant! Thanks.

[lmalenfant]

@hayakawa16 when you get a chance, could you pull the latest and try out the log plots and let me know if this looks good to you.

[hayakawa16]

Hi @lmalenfant, I tried log plots both in x and y, and ticks look good. Are you still noticing something incorrect?

[lmalenfant]

@hayakawa16 I think everything looks good, I was running tests at the same time as running the GUI and I think that may have caused some issues.

I tested several plots yesterday and they all looked good.

[lmalenfant]

Added the new VTS library and now I can get the Phase and Amplitude derivative values.

Phase:

Amplitude:

[sonarcloud[bot]]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
95.1% Coverage on New Code
6.4% Duplication on New Code

See analysis details on SonarCloud