{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/10385","repository_url":"https://api.github.com/repos/RIOT-OS/RIOT","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/10385/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/10385/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/10385/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/10385","id":380627963,"node_id":"MDExOlB1bGxSZXF1ZXN0MjMwNzc5NzYx","number":10385,"title":"tests/puf_sram: add input args to automation script","user":{"login":"PeterKietzmann","id":7765855,"node_id":"MDQ6VXNlcjc3NjU4NTU=","avatar_url":"https://avatars3.githubusercontent.com/u/7765855?v=4","gravatar_id":"","url":"https://api.github.com/users/PeterKietzmann","html_url":"https://github.com/PeterKietzmann","followers_url":"https://api.github.com/users/PeterKietzmann/followers","following_url":"https://api.github.com/users/PeterKietzmann/following{/other_user}","gists_url":"https://api.github.com/users/PeterKietzmann/gists{/gist_id}","starred_url":"https://api.github.com/users/PeterKietzmann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PeterKietzmann/subscriptions","organizations_url":"https://api.github.com/users/PeterKietzmann/orgs","repos_url":"https://api.github.com/users/PeterKietzmann/repos","events_url":"https://api.github.com/users/PeterKietzmann/events{/privacy}","received_events_url":"https://api.github.com/users/PeterKietzmann/received_events","type":"User","site_admin":false},"labels":[{"id":89588546,"node_id":"MDU6TGFiZWw4OTU4ODU0Ng==","url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Area:%20tests","name":"Area: tests","color":"6600ff","default":false},{"id":214816571,"node_id":"MDU6TGFiZWwyMTQ4MTY1NzE=","url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/CI:%20ready%20for%20build","name":"CI: ready for build","color":"66ffcc","default":false},{"id":104179071,"node_id":"MDU6TGFiZWwxMDQxNzkwNzE=","url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Impact:%20minor","name":"Impact: minor","color":"ff0000","default":false},{"id":1075449391,"node_id":"MDU6TGFiZWwxMDc1NDQ5Mzkx","url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Reviewed:%201-fundamentals","name":"Reviewed: 1-fundamentals","color":"00dd00","default":false},{"id":1075450172,"node_id":"MDU6TGFiZWwxMDc1NDUwMTcy","url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Reviewed:%202-code-design","name":"Reviewed: 2-code-design","color":"00dd00","default":false},{"id":23399100,"node_id":"MDU6TGFiZWwyMzM5OTEwMA==","url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Type:%20enhancement","name":"Type: enhancement","color":"ffff66","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-11-14T10:18:25Z","updated_at":"2018-11-14T10:59:25Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/10385","html_url":"https://github.com/RIOT-OS/RIOT/pull/10385","diff_url":"https://github.com/RIOT-OS/RIOT/pull/10385.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/10385.patch"},"body":"### Contribution description\r\n\r\nThis PR adds optional input arguments to the automation script for estimating the minimum entropy of the SRAM based random seed (*tests/puf_sram*). It is needed to change the serial port or baudrate when changing to platforms with different serial parameters.\r\n\r\n### Testing procedure\r\n\r\n- Calling the script without parameters should lead to the same behavior as before\r\n- Calling script help (`example_test.py -h`) should show information about input parameters\r\n```\r\nusage: example_test.py [-h] [-n NUMBER] [-t OFF_TIME] [-p PORT] [-b BAUDRATE]\r\n                       [-d]\r\n\r\noptional arguments:\r\n  -h, --help            show this help message and exit\r\n  -n NUMBER, --number NUMBER\r\n                        Number of iterations, default: 500\r\n  -t OFF_TIME, --off_time OFF_TIME\r\n                        Off time, default: 1 [s]\r\n  -p PORT, --port PORT  Serial port, default: /dev/ttyUSB0\r\n  -b BAUDRATE, --baudrate BAUDRATE\r\n                        Baudrate of the serial port, default: 115200\r\n  -d, --disable_output  Disable verbose output\r\n```\r\n- `example_test.py -n 3 -t 10` should reduce the number of iterations to 3 and the power-off time before restart to 10 seconds\r\n- - `example_test.py -d` disables intermediate status reports but the board reboots anyway (status LEDs turn of repeatedly)\r\n- Changing the `-p` or `-b` parameters doesn't make sense right now, as these parameters are fixed for boards\r\n\r\n### Issues/PRs references\r\nn/a"},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/438623059","html_url":"https://github.com/RIOT-OS/RIOT/pull/10385#issuecomment-438623059","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/10385","id":438623059,"node_id":"MDEyOklzc3VlQ29tbWVudDQzODYyMzA1OQ==","user":{"login":"MrKevinWeiss","id":19396439,"node_id":"MDQ6VXNlcjE5Mzk2NDM5","avatar_url":"https://avatars1.githubusercontent.com/u/19396439?v=4","gravatar_id":"","url":"https://api.github.com/users/MrKevinWeiss","html_url":"https://github.com/MrKevinWeiss","followers_url":"https://api.github.com/users/MrKevinWeiss/followers","following_url":"https://api.github.com/users/MrKevinWeiss/following{/other_user}","gists_url":"https://api.github.com/users/MrKevinWeiss/gists{/gist_id}","starred_url":"https://api.github.com/users/MrKevinWeiss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MrKevinWeiss/subscriptions","organizations_url":"https://api.github.com/users/MrKevinWeiss/orgs","repos_url":"https://api.github.com/users/MrKevinWeiss/repos","events_url":"https://api.github.com/users/MrKevinWeiss/events{/privacy}","received_events_url":"https://api.github.com/users/MrKevinWeiss/received_events","type":"User","site_admin":false},"created_at":"2018-11-14T10:59:25Z","updated_at":"2018-11-14T10:59:25Z","author_association":"CONTRIBUTOR","body":"Ya it makes sense and look well done, maybe DEFAULT_NUMBER can be better named like DEFAULT_POWER_CYCLES or DEFAULT_SEED_GENERATIONS, but all in all looks good.  I will do a test tomorrow when I have some hardware. (unless @MichelRottleuthner beats me)"}}
{"actor":{"display_login":"MrKevinWeiss"}}