{"action":"created","issue":{"url":"https://api.github.com/repos/usnistgov/SP800-90B_EntropyAssessment/issues/41","repository_url":"https://api.github.com/repos/usnistgov/SP800-90B_EntropyAssessment","labels_url":"https://api.github.com/repos/usnistgov/SP800-90B_EntropyAssessment/issues/41/labels{/name}","comments_url":"https://api.github.com/repos/usnistgov/SP800-90B_EntropyAssessment/issues/41/comments","events_url":"https://api.github.com/repos/usnistgov/SP800-90B_EntropyAssessment/issues/41/events","html_url":"https://github.com/usnistgov/SP800-90B_EntropyAssessment/pull/41","id":278615502,"number":41,"title":"Modified to use packed data, enhance verbose output, increase efficie…","user":{"login":"zipnemud","id":12663736,"avatar_url":"https://avatars0.githubusercontent.com/u/12663736?v=4","gravatar_id":"","url":"https://api.github.com/users/zipnemud","html_url":"https://github.com/zipnemud","followers_url":"https://api.github.com/users/zipnemud/followers","following_url":"https://api.github.com/users/zipnemud/following{/other_user}","gists_url":"https://api.github.com/users/zipnemud/gists{/gist_id}","starred_url":"https://api.github.com/users/zipnemud/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zipnemud/subscriptions","organizations_url":"https://api.github.com/users/zipnemud/orgs","repos_url":"https://api.github.com/users/zipnemud/repos","events_url":"https://api.github.com/users/zipnemud/events{/privacy}","received_events_url":"https://api.github.com/users/zipnemud/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2017-12-01T22:27:49Z","updated_at":"2017-12-16T19:53:30Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/usnistgov/SP800-90B_EntropyAssessment/pulls/41","html_url":"https://github.com/usnistgov/SP800-90B_EntropyAssessment/pull/41","diff_url":"https://github.com/usnistgov/SP800-90B_EntropyAssessment/pull/41.diff","patch_url":"https://github.com/usnistgov/SP800-90B_EntropyAssessment/pull/41.patch"},"body":"…ncy of permutation check\r\n\r\nLots of changes here.\r\n\r\nI increased the verbose output in permutation_tests.py.  It's now possible to monitor what is happening as the permutation tests are run.\r\n\r\nI changed the tests in permutation_tests.py so that when the pass criteria are met, the test ends.  I changed the tests in permutation_tests.py so that when a test has passed, it no longer is computed.  These two changes mean that the permutation tests are *much* faster.\r\n\r\nI increased the output of restart.py under verbose.\r\n\r\nI altered the row and column frequency selection in restart.py so that the results are saved in case future analysis is desired.\r\n\r\nI changed to_data in util90b.py so that packed data is used.  This allows bit sizes of greater than 8 bits, but their use is curtailed because the rest of the programs don't deal well with the larger size.  I did get a 9 bit sample size to pass the restart tests.\r\n\r\nI added two new variables to the arguments list in the parser in util90b.py.  They are a start_position and read_amount.  The start_position is where to start reading random data in the file, and the read_amount is how many bytes to read.  This allows a larger file to be processed in chunks, so that restart can still be run on the output.\r\n\r\nI modified iid_main.py, noniid_main.py, and restart.py to utilize the start_position and read_amount parameters."},"comment":{"url":"https://api.github.com/repos/usnistgov/SP800-90B_EntropyAssessment/issues/comments/352207051","html_url":"https://github.com/usnistgov/SP800-90B_EntropyAssessment/pull/41#issuecomment-352207051","issue_url":"https://api.github.com/repos/usnistgov/SP800-90B_EntropyAssessment/issues/41","id":352207051,"user":{"login":"zipnemud","id":12663736,"avatar_url":"https://avatars0.githubusercontent.com/u/12663736?v=4","gravatar_id":"","url":"https://api.github.com/users/zipnemud","html_url":"https://github.com/zipnemud","followers_url":"https://api.github.com/users/zipnemud/followers","following_url":"https://api.github.com/users/zipnemud/following{/other_user}","gists_url":"https://api.github.com/users/zipnemud/gists{/gist_id}","starred_url":"https://api.github.com/users/zipnemud/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zipnemud/subscriptions","organizations_url":"https://api.github.com/users/zipnemud/orgs","repos_url":"https://api.github.com/users/zipnemud/repos","events_url":"https://api.github.com/users/zipnemud/events{/privacy}","received_events_url":"https://api.github.com/users/zipnemud/received_events","type":"User","site_admin":false},"created_at":"2017-12-16T19:53:30Z","updated_at":"2017-12-16T19:53:30Z","author_association":"NONE","body":"Interesting.  Thanks for the explanation.  I had just assumed that iid was a synonym for random.  I can see that your arguments are true, but why would we care?  What good is knowing an entropy estimate for non random generators?  If I want non random data, I can always create it to order from random data.  And I wouldn't want to feed non random data into any kind of PRNG.  It's not like true RNGs aren't available, https://en.wikipedia.org/wiki/Comparison_of_hardware_random_number_generators.  I must be missing something, because you obviously went to a lot of effort to build this, and you wouldn't have unless there was a purpose.\r\n\r\nAt least I now understand why to_dataset functions the way it does in the original utility.  I wondered why it was designed that way if you were testing RNGs, but it is clear it has to be for your purposes."}}
{"actor":{"display_login":"zipnemud"}}