-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Labels
peer reviewFeedback from peer review of the repoFeedback from peer review of the repo
Description
Peer review from @tbslater
Verification and Validation
- Typo…
- How easy is this to do? And does it depend on the software used? --> Can't comment on ease, but have add suggestion of R software that can do this.
- I guess the issue with this is the comparison model might be wrong! Even so, it would force you to double check your model is the right one! --> Have add this note
Tests
Super clear 😊
- Just spotted a small typo…
Mathematical Proof of Correctness
-
Is there a general rule on how much difference you are willing to tolerate (here you have used 15%)?
-
Might benefit from a sentence stating that all tests have passed!
Quality Assurance
No comments 😊
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
peer reviewFeedback from peer review of the repoFeedback from peer review of the repo
Type
Projects
Status
Done