{"action":"opened","issue":{"url":"https://api.github.com/repos/AravisProject/aravis/issues/199","repository_url":"https://api.github.com/repos/AravisProject/aravis","labels_url":"https://api.github.com/repos/AravisProject/aravis/issues/199/labels{/name}","comments_url":"https://api.github.com/repos/AravisProject/aravis/issues/199/comments","events_url":"https://api.github.com/repos/AravisProject/aravis/issues/199/events","html_url":"https://github.com/AravisProject/aravis/issues/199","id":368999502,"node_id":"MDU6SXNzdWUzNjg5OTk1MDI=","number":199,"title":"auto packet size does not consider possible value changes by device","user":{"login":"TIS-Edgar","id":6525011,"node_id":"MDQ6VXNlcjY1MjUwMTE=","avatar_url":"https://avatars3.githubusercontent.com/u/6525011?v=4","gravatar_id":"","url":"https://api.github.com/users/TIS-Edgar","html_url":"https://github.com/TIS-Edgar","followers_url":"https://api.github.com/users/TIS-Edgar/followers","following_url":"https://api.github.com/users/TIS-Edgar/following{/other_user}","gists_url":"https://api.github.com/users/TIS-Edgar/gists{/gist_id}","starred_url":"https://api.github.com/users/TIS-Edgar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TIS-Edgar/subscriptions","organizations_url":"https://api.github.com/users/TIS-Edgar/orgs","repos_url":"https://api.github.com/users/TIS-Edgar/repos","events_url":"https://api.github.com/users/TIS-Edgar/events{/privacy}","received_events_url":"https://api.github.com/users/TIS-Edgar/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-10-11T08:05:42Z","updated_at":"2018-10-11T08:05:42Z","closed_at":null,"author_association":"NONE","body":"Hi Emmanuel,\r\n\r\nWe had some problems with `arv_gv_device_auto_packet_size`.\r\nWe noticed that aravis regularly determined a packet size that is far under the allowed maximum.\r\nFor example with a MTU of 9000 aravis would still determine a packet size of 3708 bytes although jumbo frames go through everywhere.\r\n\r\nFrom our understanding the function needs to read the GevSCPSPacketSize back after setting it as the device is allowed to round the value as it has to be divisible by 4.\r\n\r\n```\r\narv_device_set_integer_feature_value (device, \"GevSCPSPacketSize\", current_size);\r\ncurrent_size = arv_device_get_integer_feature_value(device, \"GevSCPSPacketSize\");\r\n```\r\nThis seems to fix our issues, but I am unsure if there is something I am overlooking or if you want additional checks, etc.\r\n"}}
{"actor":{"display_login":"TIS-Edgar"}}