)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35584,"name":"Vasudeo Nimbekar","display_name":"Vasudeo Nimbekar","email":"vasudeo.nimbekar.opendev@gmail.com","username":"devnimbekar"},"change_message_id":"2d422f05dc9dafa0072aeaf603ae149efbacd6b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2f9e49bb_842e393b","updated":"2023-02-27 06:29:49.000000000","message":"recheck","commit_id":"de4b9282258b823b7cc731d21b15a36abc4265e7"},{"author":{"_account_id":35584,"name":"Vasudeo Nimbekar","display_name":"Vasudeo Nimbekar","email":"vasudeo.nimbekar.opendev@gmail.com","username":"devnimbekar"},"change_message_id":"95671a445b184c5879597a512deb79dde95ea51f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"73159135_91a6231b","updated":"2023-04-19 07:25:24.000000000","message":"recheck","commit_id":"36d236a525a8259cc4974dbe3e74274fc0d612a9"},{"author":{"_account_id":35584,"name":"Vasudeo Nimbekar","display_name":"Vasudeo Nimbekar","email":"vasudeo.nimbekar.opendev@gmail.com","username":"devnimbekar"},"change_message_id":"9d8bc4bb7ac85391cda82d40dd4bfcebf3c971be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"0cab2424_7aeaee7f","updated":"2023-04-21 10:06:57.000000000","message":"recheck","commit_id":"9fc0ad85bc72861b8d8eab76dbc6e6cfc821b353"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"709492278f13629b2773bb3792c8fb4c923eabf5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"39a4cab5_82b945e5","updated":"2023-05-29 15:01:30.000000000","message":"recheck","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"13825eaf22ab0ad7e9d512b360f2ebcfcd6a3132","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"050201ac_1bd8a7bc","updated":"2023-06-25 21:58:00.000000000","message":"recheck","commit_id":"2ad9713bedc0287eb115a865a4e468885f7798fb"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"bf35dfeebead4db453bed7478a465c6909452959","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"0383ddfe_1ed95d21","updated":"2023-08-22 11:19:33.000000000","message":"Dear reviewers, please look at this patch 😞","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"65d9c076bb4fd27a8ecae8fc9257d47cb46f8a2a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":22,"id":"63f8e31a_550ab238","updated":"2023-08-22 11:49:42.000000000","message":"I added few comments.\nI am off my computer this week and the coming week, so I\u0027ll me able to review more in september","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"e1d9f3a0f2dcab9f063964eaa1c5f7c78ba125d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"6f07e8c6_cd0c1639","updated":"2023-06-26 23:45:23.000000000","message":"recheck","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"2a148d486d573957e5cc49b945c4639e6c18f06b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"7cb01fea_bffd6690","updated":"2023-06-26 10:48:02.000000000","message":"recheck","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"73f50e4b8ece1c92ad4553b944a83b9c2501b525","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"a937438a_f70c3043","updated":"2023-06-27 18:46:47.000000000","message":"recheck","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"bc6146a8ff446767791a9c2aa9aa11280008e66d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"aad84958_5ada42b2","updated":"2023-06-26 19:24:51.000000000","message":"recheck","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"81c490f168ccbb010e47d923b8a79771824aac9a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"bf200ce0_bcd86d04","updated":"2023-06-27 08:33:51.000000000","message":"recheck","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"59486b491814cc3214ef68b1fbf44195ece9b49b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"d74f9d13_043725a6","updated":"2023-06-27 11:06:43.000000000","message":"recheck","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"81607086d1c64b249f159f439c60688c54662fe3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"799c215c_f0af405e","in_reply_to":"63f8e31a_550ab238","updated":"2024-03-14 09:31:06.000000000","message":"Done","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"d9986f420b8969bf40754962ab9813cefa278ca3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"9671f2a1_69c55e13","in_reply_to":"d74f9d13_043725a6","updated":"2023-06-27 19:32:17.000000000","message":"the tox unit mysql are failing due to a race condition it seems.\nMaybe we can fix that in another patchset to avoid false failure?","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"d06c05db68e1bdd3ccd5900ee1b8d348cba76d5d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"945905ba_b596fe58","updated":"2023-10-05 16:23:25.000000000","message":"recheck","commit_id":"c13a5368e895c501dc13abca235f6183fbcb3bc3"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"65845511442564a6aaf1a3a9fd4d809993b13391","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"e3e36cd0_3bc65d5e","updated":"2023-10-05 14:57:55.000000000","message":"recheck","commit_id":"c13a5368e895c501dc13abca235f6183fbcb3bc3"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"ed21a8dbf598404c79060056af00256de781b569","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"2a296b9c_f7e57977","updated":"2024-03-14 12:03:57.000000000","message":"I think we had enough time to review this 🤓","commit_id":"f2cbe1c59d8a83f86d9321d5d8a4626ffe034364"}],"mistral/config.py":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"01fe359937c192dbac8055465bc2ed3710695788","unresolved":true,"context_lines":[{"line_number":573,"context_line":"        \u0027hash_version_keys\u0027,"},{"line_number":574,"context_line":"        default\u003dTrue,"},{"line_number":575,"context_line":"        help\u003d("},{"line_number":576,"context_line":"            \u0027If this value is set to True then the Mistral will use \u0027"},{"line_number":577,"context_line":"            \u0027md5 hashing for version keys to ensure this keys will be \u0027"},{"line_number":578,"context_line":"            \u0027the same size.\u0027"},{"line_number":579,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":19,"id":"65d1e959_f5f80fd9","line":576,"updated":"2023-05-31 11:56:40.000000000","message":"nit: then Mistral will use","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"81607086d1c64b249f159f439c60688c54662fe3","unresolved":false,"context_lines":[{"line_number":573,"context_line":"        \u0027hash_version_keys\u0027,"},{"line_number":574,"context_line":"        default\u003dTrue,"},{"line_number":575,"context_line":"        help\u003d("},{"line_number":576,"context_line":"            \u0027If this value is set to True then the Mistral will use \u0027"},{"line_number":577,"context_line":"            \u0027md5 hashing for version keys to ensure this keys will be \u0027"},{"line_number":578,"context_line":"            \u0027the same size.\u0027"},{"line_number":579,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":19,"id":"2fa724c0_42413e59","line":576,"in_reply_to":"65d1e959_f5f80fd9","updated":"2024-03-14 09:31:06.000000000","message":"Done","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"01fe359937c192dbac8055465bc2ed3710695788","unresolved":true,"context_lines":[{"line_number":576,"context_line":"            \u0027If this value is set to True then the Mistral will use \u0027"},{"line_number":577,"context_line":"            \u0027md5 hashing for version keys to ensure this keys will be \u0027"},{"line_number":578,"context_line":"            \u0027the same size.\u0027"},{"line_number":579,"context_line":"        )"},{"line_number":580,"context_line":"    )"},{"line_number":581,"context_line":"]"},{"line_number":582,"context_line":""}],"source_content_type":"text/x-python","patch_set":19,"id":"0cf01a17_5d87a7c7","line":579,"updated":"2023-05-31 11:56:40.000000000","message":"A more global comment about this option, is it really needed?\nWhat if we hardcode and force using hashing?\nIn other words, what is the downside of using hashing versus not using it?","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"0b060bd34331e4e6c5e43c1691218f54ca892a7b","unresolved":true,"context_lines":[{"line_number":576,"context_line":"            \u0027If this value is set to True then the Mistral will use \u0027"},{"line_number":577,"context_line":"            \u0027md5 hashing for version keys to ensure this keys will be \u0027"},{"line_number":578,"context_line":"            \u0027the same size.\u0027"},{"line_number":579,"context_line":"        )"},{"line_number":580,"context_line":"    )"},{"line_number":581,"context_line":"]"},{"line_number":582,"context_line":""}],"source_content_type":"text/x-python","patch_set":19,"id":"51f51997_fc2488b8","line":579,"in_reply_to":"0cf01a17_5d87a7c7","updated":"2023-06-25 22:11:19.000000000","message":"hi! thanks for your review! I\u0027ve made some changes according your comments, please resolve them if I did everything correct 😊","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"81607086d1c64b249f159f439c60688c54662fe3","unresolved":false,"context_lines":[{"line_number":576,"context_line":"            \u0027If this value is set to True then the Mistral will use \u0027"},{"line_number":577,"context_line":"            \u0027md5 hashing for version keys to ensure this keys will be \u0027"},{"line_number":578,"context_line":"            \u0027the same size.\u0027"},{"line_number":579,"context_line":"        )"},{"line_number":580,"context_line":"    )"},{"line_number":581,"context_line":"]"},{"line_number":582,"context_line":""}],"source_content_type":"text/x-python","patch_set":19,"id":"83927698_8371b3b4","line":579,"in_reply_to":"51f51997_fc2488b8","updated":"2024-03-14 09:31:06.000000000","message":"Done","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"}],"mistral/workflow/context_versioning.py":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"01fe359937c192dbac8055465bc2ed3710695788","unresolved":true,"context_lines":[{"line_number":32,"context_line":"        if task_ex.in_context is not None else {}"},{"line_number":33,"context_line":"    )"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    if not cfg.CONF.context_versioning.enabled:"},{"line_number":36,"context_line":"        return in_context"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    task_published \u003d task_ex.published or []"}],"source_content_type":"text/x-python","patch_set":19,"id":"5aea51b3_5e8c010f","line":35,"updated":"2023-05-31 11:56:40.000000000","message":"Why not returning even earlier? We dont need to deepcopy if the versioning is disabled","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"81607086d1c64b249f159f439c60688c54662fe3","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        if task_ex.in_context is not None else {}"},{"line_number":33,"context_line":"    )"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    if not cfg.CONF.context_versioning.enabled:"},{"line_number":36,"context_line":"        return in_context"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    task_published \u003d task_ex.published or []"}],"source_content_type":"text/x-python","patch_set":19,"id":"3759b8a4_c7229a42","line":35,"in_reply_to":"5aea51b3_5e8c010f","updated":"2024-03-14 09:31:06.000000000","message":"Done","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"01fe359937c192dbac8055465bc2ed3710695788","unresolved":true,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    for updated in updated_keys:"},{"line_number":44,"context_line":"        if updated not in in_context[VERSIONS_KEY]:"},{"line_number":45,"context_line":"            in_context[VERSIONS_KEY][updated] \u003d 1"},{"line_number":46,"context_line":"        else:"},{"line_number":47,"context_line":"            in_context[VERSIONS_KEY][updated] +\u003d 1"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-python","patch_set":19,"id":"281f95c1_014d478b","line":45,"updated":"2023-05-31 11:56:40.000000000","message":"You may use python setdefault to avoid this if statement:\nE.G.\nin_context \u003d {}\nin_context[\u0027a\u0027] \u003d {}\nin_context[\u0027a\u0027].setdefault(\u0027updated\u0027, 0)\nin_context[\u0027a\u0027][\u0027updated\u0027] +\u003d 1","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"81607086d1c64b249f159f439c60688c54662fe3","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    for updated in updated_keys:"},{"line_number":44,"context_line":"        if updated not in in_context[VERSIONS_KEY]:"},{"line_number":45,"context_line":"            in_context[VERSIONS_KEY][updated] \u003d 1"},{"line_number":46,"context_line":"        else:"},{"line_number":47,"context_line":"            in_context[VERSIONS_KEY][updated] +\u003d 1"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-python","patch_set":19,"id":"d1deb624_a32de28c","line":45,"in_reply_to":"281f95c1_014d478b","updated":"2024-03-14 09:31:06.000000000","message":"Done","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"01fe359937c192dbac8055465bc2ed3710695788","unresolved":true,"context_lines":[{"line_number":74,"context_line":"            )"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"def _create_versions_data_if_not_presented(in_context):"},{"line_number":78,"context_line":"    if VERSIONS_KEY not in in_context:"},{"line_number":79,"context_line":"        in_context[VERSIONS_KEY] \u003d {}"},{"line_number":80,"context_line":""}],"source_content_type":"text/x-python","patch_set":19,"id":"73e543dd_fec14508","line":77,"updated":"2023-05-31 11:56:40.000000000","message":"Same here, maybe use setdefault in def get_in_context_with_versions instead of having a specific function for this","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"81607086d1c64b249f159f439c60688c54662fe3","unresolved":false,"context_lines":[{"line_number":74,"context_line":"            )"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"def _create_versions_data_if_not_presented(in_context):"},{"line_number":78,"context_line":"    if VERSIONS_KEY not in in_context:"},{"line_number":79,"context_line":"        in_context[VERSIONS_KEY] \u003d {}"},{"line_number":80,"context_line":""}],"source_content_type":"text/x-python","patch_set":19,"id":"8bb4c9d0_54c50086","line":77,"in_reply_to":"73e543dd_fec14508","updated":"2024-03-14 09:31:06.000000000","message":"Done","commit_id":"ab6f1d9d907900636394fd8077a94090e2e009b5"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"65d9c076bb4fd27a8ecae8fc9257d47cb46f8a2a","unresolved":true,"context_lines":[{"line_number":58,"context_line":"        new_prefix \u003d key if not prefix else prefix + \".\" + key"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        if not isinstance(published[key], dict):"},{"line_number":61,"context_line":"            if cfg.CONF.context_versioning.hash_version_keys:"},{"line_number":62,"context_line":"                new_prefix \u003d hashlib.md5("},{"line_number":63,"context_line":"                    new_prefix.encode(\"utf-8\")"},{"line_number":64,"context_line":"                ).hexdigest()"}],"source_content_type":"text/x-python","patch_set":22,"id":"2ce24b23_9c0ceb04","line":61,"updated":"2023-08-22 11:49:42.000000000","message":"hi,\nI still don\u0027t get the purpose of hashing the keys?","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"18067cd6cc8b3668325a6e8ca4de47b2feeda605","unresolved":true,"context_lines":[{"line_number":58,"context_line":"        new_prefix \u003d key if not prefix else prefix + \".\" + key"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        if not isinstance(published[key], dict):"},{"line_number":61,"context_line":"            if cfg.CONF.context_versioning.hash_version_keys:"},{"line_number":62,"context_line":"                new_prefix \u003d hashlib.md5("},{"line_number":63,"context_line":"                    new_prefix.encode(\"utf-8\")"},{"line_number":64,"context_line":"                ).hexdigest()"}],"source_content_type":"text/x-python","patch_set":22,"id":"ffe69df7_354d8fb5","line":61,"in_reply_to":"2ce24b23_9c0ceb04","updated":"2023-10-04 13:44:43.000000000","message":"We should use hashing to avoid hude memory consumptions. If there is deep nested context every key could be very long, hashing provides an ability to limit size of this data.\nBut on the other side, for dev or debug purposes hashing should be disabled (I\u0027ve already had this experience)","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"81607086d1c64b249f159f439c60688c54662fe3","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        new_prefix \u003d key if not prefix else prefix + \".\" + key"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        if not isinstance(published[key], dict):"},{"line_number":61,"context_line":"            if cfg.CONF.context_versioning.hash_version_keys:"},{"line_number":62,"context_line":"                new_prefix \u003d hashlib.md5("},{"line_number":63,"context_line":"                    new_prefix.encode(\"utf-8\")"},{"line_number":64,"context_line":"                ).hexdigest()"}],"source_content_type":"text/x-python","patch_set":22,"id":"2c5039df_62caf3b0","line":61,"in_reply_to":"ffe69df7_354d8fb5","updated":"2024-03-14 09:31:06.000000000","message":"Done","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"65d9c076bb4fd27a8ecae8fc9257d47cb46f8a2a","unresolved":true,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"    _remove_internal_data_from_context(ctx_left)"},{"line_number":80,"context_line":"    _remove_internal_data_from_context(ctx_right)"},{"line_number":81,"context_line":"    del ctx_left[VERSIONS_KEY]"},{"line_number":82,"context_line":"    del ctx_right[VERSIONS_KEY]"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    result \u003d _merge_ctx(ctx_left, versions_left, ctx_right, versions_right)"}],"source_content_type":"text/x-python","patch_set":22,"id":"51478545_1ab02a39","line":81,"updated":"2023-08-22 11:49:42.000000000","message":"maybe you could remove all not needed stuff in CTX inside the remove_internal function?","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"81607086d1c64b249f159f439c60688c54662fe3","unresolved":false,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"    _remove_internal_data_from_context(ctx_left)"},{"line_number":80,"context_line":"    _remove_internal_data_from_context(ctx_right)"},{"line_number":81,"context_line":"    del ctx_left[VERSIONS_KEY]"},{"line_number":82,"context_line":"    del ctx_right[VERSIONS_KEY]"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    result \u003d _merge_ctx(ctx_left, versions_left, ctx_right, versions_right)"}],"source_content_type":"text/x-python","patch_set":22,"id":"b7721459_02955c96","line":81,"in_reply_to":"51478545_1ab02a39","updated":"2024-03-14 09:31:06.000000000","message":"Done","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"}],"mistral/workflow/data_flow.py":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"65d9c076bb4fd27a8ecae8fc9257d47cb46f8a2a","unresolved":true,"context_lines":[{"line_number":272,"context_line":"    # It\u0027s better to avoid using the method copy.deepcopy() because on"},{"line_number":273,"context_line":"    # dictionaries with many entries it significantly increases memory"},{"line_number":274,"context_line":"    # footprint and reduces performance."},{"line_number":275,"context_line":"    in_context \u003d ("},{"line_number":276,"context_line":"        dict(task_ex.in_context)"},{"line_number":277,"context_line":"        if getattr(task_ex, \u0027in_context\u0027, None) is not None else {}"},{"line_number":278,"context_line":"    )"}],"source_content_type":"text/x-python","patch_set":22,"id":"ae8614d6_a31a49b4","line":275,"updated":"2023-08-22 11:49:42.000000000","message":"so this can be removed, this is dead code with the new line below","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"},{"author":{"_account_id":29124,"name":"Oleg Ovcharuk","email":"vgvoleg@gmail.com","username":"vgvoleg"},"change_message_id":"81607086d1c64b249f159f439c60688c54662fe3","unresolved":false,"context_lines":[{"line_number":272,"context_line":"    # It\u0027s better to avoid using the method copy.deepcopy() because on"},{"line_number":273,"context_line":"    # dictionaries with many entries it significantly increases memory"},{"line_number":274,"context_line":"    # footprint and reduces performance."},{"line_number":275,"context_line":"    in_context \u003d ("},{"line_number":276,"context_line":"        dict(task_ex.in_context)"},{"line_number":277,"context_line":"        if getattr(task_ex, \u0027in_context\u0027, None) is not None else {}"},{"line_number":278,"context_line":"    )"}],"source_content_type":"text/x-python","patch_set":22,"id":"2ba60e49_5ac5e778","line":275,"in_reply_to":"ae8614d6_a31a49b4","updated":"2024-03-14 09:31:06.000000000","message":"Done","commit_id":"e7eb940708242ced86cd68d0330b23b077d77a65"}]}