Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • G gsll
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 3
    • Issues 3
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 4
    • Merge requests 4
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • antik
  • gsll
  • Merge requests
  • !2

Open
Created Dec 28, 2016 by Mirko Vukovic@mvukovicReporter
  • Report abuse
Report abuse

fixed incorrect logic in rng testpdf

  • Overview 0
  • Commits 20
  • Changes 11

The test logic is to compare the actual and expected count in bins. This works as long as the actual and expected pdf's are properly normalized.

In the prior code this assumption failed when the upper and lower bin limits (given by gslt-lower-limit and gslt-upper-limit) were such that some of the PDF was left unsampled. Under those conditions, the tests in limits-check would never pass. For some distribution functions the tests took forever to complete because upon a failure test the number of samples would be doubled (up to 50 times, leading to a 10^15 increase in the number of points).

The proposed merge fixes to code so to satisfy this assumption. I also re-factored some code and added documentation.

After the change, all the tests run in a finite amount of time and pass as well.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: x16-rng