hyppo icon indicating copy to clipboard operation
hyppo copied to clipboard

add more robust unit tests to validate accuracy with other simulations

Open sampan501 opened this issue 3 years ago • 42 comments

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context (e.g. screenshots)

sampan501 avatar Mar 10 '21 14:03 sampan501

tests against sinusodial sims, general nonlinear sims, etc.

sampan501 avatar Mar 10 '21 14:03 sampan501

@sampan501 I am very new to open source.Could you please walk me through and tell me what exactly needs to be done?

pragyasrivastava0805 avatar Aug 24 '21 15:08 pragyasrivastava0805

Hello, thank you for looking into this. If you look at the unit tests in the package, currently it only verifies if the test statistic and p-value are the same for a single simulation, whether Type I error is controlled, and if inputs are proper shapes. I think something like this would be good (similar code to here): 1D linear (we have), a nonlinear simulation, and independence (we have); 5D versions of all of them. Does that make sense?

sampan501 avatar Aug 24 '21 23:08 sampan501

Hi can I work on this too ?

svshivapuja avatar Aug 29 '21 07:08 svshivapuja

sure!

sampan501 avatar Aug 29 '21 14:08 sampan501

Hi @sampan501 , since no prs have been submitted in a while for this issues, can I give it a shot?

scfenton6 avatar Oct 12 '21 10:10 scfenton6

Are @svshivapuja or @pragyasrivastava0805 currently working on this?

sampan501 avatar Oct 12 '21 13:10 sampan501

Sorry, I was caught up on other work. You can go ahead and pass on the work. Sorry again..

svshivapuja avatar Oct 12 '21 13:10 svshivapuja