)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"bf3eae93a6d64d38f052c7856951d261971565e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9d293206_8848e51e","updated":"2021-11-04 00:48:04.000000000","message":"good catch","commit_id":"2ec971a5ae480b42a8de52456ca8f1156f9df99c"},{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"a43905c06748a14ba1be23026f8f5301b4151115","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ce3a64ba_4101b38a","updated":"2021-11-04 00:46:24.000000000","message":"recheck","commit_id":"2ec971a5ae480b42a8de52456ca8f1156f9df99c"},{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"f9a36d618d43382eef34187044b95fad6b4c4870","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7e3fb865_06dc3004","updated":"2021-11-10 07:26:19.000000000","message":"recheck","commit_id":"54a206ee0ff72aa67b9bde117a8b82115ee0e6a9"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"42f7a123997cf075199765b760ae619fa686a39b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d30d3ac3_776f2b3c","updated":"2021-11-11 12:16:39.000000000","message":"over all lgtm, but one comment in test case.","commit_id":"ac0b081b81c50ee46e2223032b1017b891efc62d"}],"cyborg/accelerator/drivers/fpga/inspur/sysinfo.py":[{"author":{"_account_id":25738,"name":"Xinran WANG","email":"xin-ran.wang@intel.com","username":"Xinran"},"change_message_id":"3c15ceba4b1a5814a04603518073b2057c569335","unresolved":true,"context_lines":[{"line_number":41,"context_line":"    r\"[\\[](?P\u003cvendor_id\u003e[0-9a-fA-F]\""},{"line_number":42,"context_line":"    r\"{4}):(?P\u003cproduct_id\u003e[0-9a-fA-F]{4})].*\")"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"VENDOR_MAPS \u003d {\"1bd4\": \"inspur\"}"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"@cyborg.privsep.sys_admin_pctxt.entrypoint"}],"source_content_type":"text/x-python","patch_set":1,"id":"caa0415f_d2e776d8","line":44,"range":{"start_line":44,"start_character":0,"end_line":44,"end_character":32},"updated":"2021-11-08 07:29:50.000000000","message":"why don\u0027t you use vendor id to filter out the device\u003e?","commit_id":"2ec971a5ae480b42a8de52456ca8f1156f9df99c"},{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"561fd7a8c438333f43f8272b4a1bab88cf80475d","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    r\"[\\[](?P\u003cvendor_id\u003e[0-9a-fA-F]\""},{"line_number":42,"context_line":"    r\"{4}):(?P\u003cproduct_id\u003e[0-9a-fA-F]{4})].*\")"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"VENDOR_MAPS \u003d {\"1bd4\": \"inspur\"}"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"@cyborg.privsep.sys_admin_pctxt.entrypoint"}],"source_content_type":"text/x-python","patch_set":1,"id":"bb855c45_09ef0068","line":44,"range":{"start_line":44,"start_character":0,"end_line":44,"end_character":32},"in_reply_to":"caa0415f_d2e776d8","updated":"2021-11-11 08:29:25.000000000","message":"Done","commit_id":"2ec971a5ae480b42a8de52456ca8f1156f9df99c"},{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"47c18dc55f204a07dce4cb8c01a61e089b28315b","unresolved":true,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"def fpga_tree():"},{"line_number":80,"context_line":"    fpga_list \u003d []"},{"line_number":81,"context_line":"    fpgas \u003d get_pci_devices(VENDOR_ID)"},{"line_number":82,"context_line":"    for fpga in fpgas:"},{"line_number":83,"context_line":"        m \u003d INSPUR_FPGA_INFO_PATTERN.match(fpga)"},{"line_number":84,"context_line":"        if m:"}],"source_content_type":"text/x-python","patch_set":2,"id":"217a2a73_249ec6b2","line":81,"range":{"start_line":81,"start_character":28,"end_line":81,"end_character":37},"updated":"2021-11-10 08:28:54.000000000","message":"we shouldnot remove this INSPUR_FPGA_FLAGS param.","commit_id":"54a206ee0ff72aa67b9bde117a8b82115ee0e6a9"},{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"561fd7a8c438333f43f8272b4a1bab88cf80475d","unresolved":false,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"def fpga_tree():"},{"line_number":80,"context_line":"    fpga_list \u003d []"},{"line_number":81,"context_line":"    fpgas \u003d get_pci_devices(VENDOR_ID)"},{"line_number":82,"context_line":"    for fpga in fpgas:"},{"line_number":83,"context_line":"        m \u003d INSPUR_FPGA_INFO_PATTERN.match(fpga)"},{"line_number":84,"context_line":"        if m:"}],"source_content_type":"text/x-python","patch_set":2,"id":"6082d247_72cda234","line":81,"range":{"start_line":81,"start_character":28,"end_line":81,"end_character":37},"in_reply_to":"217a2a73_249ec6b2","updated":"2021-11-11 08:29:25.000000000","message":"Done","commit_id":"54a206ee0ff72aa67b9bde117a8b82115ee0e6a9"}],"cyborg/tests/unit/accelerator/drivers/fpga/inspur/test_driver.py":[{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"42f7a123997cf075199765b760ae619fa686a39b","unresolved":true,"context_lines":[{"line_number":123,"context_line":"        mock_devices_for_vendor.return_value \u003d fake_pci_output"},{"line_number":124,"context_line":"        pci_devices \u003d sysinfo.get_pci_devices(sysinfo.INSPUR_FPGA_FLAGS,"},{"line_number":125,"context_line":"                                              vendor_id\u003dsysinfo.VENDOR_ID)"},{"line_number":126,"context_line":"        print(pci_devices)"},{"line_number":127,"context_line":"        expected \u003d [(\"0000:86:00.0 Processing accelerators [1200]: \""},{"line_number":128,"context_line":"                     \"Inspur Electronic Information Industry Co., Ltd. \""},{"line_number":129,"context_line":"                     \"Device [1bd4:a115] (rev 04)\")]"}],"source_content_type":"text/x-python","patch_set":4,"id":"e19f3a7b_b0a346e6","line":126,"range":{"start_line":126,"start_character":8,"end_line":126,"end_character":26},"updated":"2021-11-11 12:16:39.000000000","message":"This is your debug code, I would like to remove it.","commit_id":"ac0b081b81c50ee46e2223032b1017b891efc62d"},{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"aecef43bfef597098df046baff64a17c64c0ef92","unresolved":false,"context_lines":[{"line_number":123,"context_line":"        mock_devices_for_vendor.return_value \u003d fake_pci_output"},{"line_number":124,"context_line":"        pci_devices \u003d sysinfo.get_pci_devices(sysinfo.INSPUR_FPGA_FLAGS,"},{"line_number":125,"context_line":"                                              vendor_id\u003dsysinfo.VENDOR_ID)"},{"line_number":126,"context_line":"        print(pci_devices)"},{"line_number":127,"context_line":"        expected \u003d [(\"0000:86:00.0 Processing accelerators [1200]: \""},{"line_number":128,"context_line":"                     \"Inspur Electronic Information Industry Co., Ltd. \""},{"line_number":129,"context_line":"                     \"Device [1bd4:a115] (rev 04)\")]"}],"source_content_type":"text/x-python","patch_set":4,"id":"3473a4c3_3f9b7229","line":126,"range":{"start_line":126,"start_character":8,"end_line":126,"end_character":26},"in_reply_to":"e19f3a7b_b0a346e6","updated":"2021-11-12 02:09:58.000000000","message":"Done","commit_id":"ac0b081b81c50ee46e2223032b1017b891efc62d"}]}