<table><tr><td style="">devanshuagarwal marked 9 inline comments as done.<br />devanshuagarwal added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D21684">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D21684#inline-122090">View Inline</a><span style="color: #4b4d51; font-weight: bold;">sgerlach</span> wrote in <span style="color: #4b4d51; font-weight: bold;">HypothesisTest.cpp:334</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">very similar to TestT. Put everything general for all tests outside the conditional blocks.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Actually,</p>

<p style="padding: 0; margin: 8px;">For now these things are same, but will change later.</p>

<p style="padding: 0; margin: 8px;">In two these cases, I have put blocks of calculating results and setting result model.</p>

<p style="padding: 0; margin: 8px;">This will be customized according to test.</p>

<p style="padding: 0; margin: 8px;">Things which are common like calculating, std, mean, sum and count are kept outside the conditional blocks</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D21684#inline-122096">View Inline</a><span style="color: #4b4d51; font-weight: bold;">sgerlach</span> wrote in <span style="color: #4b4d51; font-weight: bold;">HypothesisTestDock.cpp:282</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">shouldn't this be "test == ttest"? Using switch would make it more readable.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">no ttest and ztest are boolean values. They are not enum values and enums TestT and TestZ are private members of HypothesisTest.cpp and is not intended to make useable to other files.</p>

<p style="padding: 0; margin: 8px;">The rationale is to provide functions for performing each test like there are functions called perfromTwoSampleIndependetTTest() and performTwoSampleIndependentZTest.</p>

<p style="padding: 0; margin: 8px;">This makes it more abstract.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R262 LabPlot</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21684">https://phabricator.kde.org/D21684</a></div></div><br /><div><strong>To: </strong>devanshuagarwal, sgerlach, asemke<br /><strong>Cc: </strong>kde-edu, LabPlot, narvaez, apol<br /></div>