)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"63ca57d1dd916d24d06309b7bd3f1e9130cd894a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b2f738a6_1ae479b2","updated":"2024-02-05 10:56:34.000000000","message":"nice find!","commit_id":"a94343863fa8e578d6555324074caddf50d7d5ee"}],"test/unit/common/ring/test_builder.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"63ca57d1dd916d24d06309b7bd3f1e9130cd894a","unresolved":true,"context_lines":[{"line_number":3327,"context_line":"        # but if you only give it out half of that"},{"line_number":3328,"context_line":"        rb.set_overload(expected_overload / 2.0)"},{"line_number":3329,"context_line":"        # ... you can expect it\u0027s not going to full disperse"},{"line_number":3330,"context_line":"        # (but see aboave about sum() accuracy)"},{"line_number":3331,"context_line":"        w \u003d (0.6547619047619048 if sys.version_info \u003e\u003d (3, 12) else"},{"line_number":3332,"context_line":"             0.6547619047619049)"},{"line_number":3333,"context_line":"        expected \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"d74edc29_6c3e94f0","line":3330,"range":{"start_line":3330,"start_character":19,"end_line":3330,"end_character":25},"updated":"2024-02-05 10:56:34.000000000","message":"typo: above","commit_id":"a94343863fa8e578d6555324074caddf50d7d5ee"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"3464bad95ca482c6c341b537e299c9178069c3d1","unresolved":false,"context_lines":[{"line_number":3327,"context_line":"        # but if you only give it out half of that"},{"line_number":3328,"context_line":"        rb.set_overload(expected_overload / 2.0)"},{"line_number":3329,"context_line":"        # ... you can expect it\u0027s not going to full disperse"},{"line_number":3330,"context_line":"        # (but see aboave about sum() accuracy)"},{"line_number":3331,"context_line":"        w \u003d (0.6547619047619048 if sys.version_info \u003e\u003d (3, 12) else"},{"line_number":3332,"context_line":"             0.6547619047619049)"},{"line_number":3333,"context_line":"        expected \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"72e359c0_ff949c50","line":3330,"range":{"start_line":3330,"start_character":19,"end_line":3330,"end_character":25},"in_reply_to":"d74edc29_6c3e94f0","updated":"2024-02-05 18:30:12.000000000","message":"Done","commit_id":"a94343863fa8e578d6555324074caddf50d7d5ee"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"63ca57d1dd916d24d06309b7bd3f1e9130cd894a","unresolved":true,"context_lines":[{"line_number":4388,"context_line":"                    (1, 1): 2.0,"},{"line_number":4389,"context_line":"                }"},{"line_number":4390,"context_line":"            }"},{"line_number":4391,"context_line":"            wr \u003d rb._build_weighted_replicas_by_tier()"},{"line_number":4392,"context_line":"            for tier_len, expected in expectations.items():"},{"line_number":4393,"context_line":"                self.assertEqual(expected, {t: ceil(r) for (t, r) in"},{"line_number":4394,"context_line":"                                            wr.items() if len(t) \u003d\u003d tier_len})"},{"line_number":4395,"context_line":""},{"line_number":4396,"context_line":""},{"line_number":4397,"context_line":"class TestRingBuilderDispersion(unittest.TestCase):"}],"source_content_type":"text/x-python","patch_set":4,"id":"2cfcb5ab_0f8bf424","line":4394,"range":{"start_line":4391,"start_character":12,"end_line":4394,"end_character":43},"updated":"2024-02-05 10:56:34.000000000","message":"shouldn\u0027t this be dedented so the assertion is made for all versions? IIUC with 3.12 the result is correct so expectations do not need to change?","commit_id":"a94343863fa8e578d6555324074caddf50d7d5ee"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"3464bad95ca482c6c341b537e299c9178069c3d1","unresolved":false,"context_lines":[{"line_number":4388,"context_line":"                    (1, 1): 2.0,"},{"line_number":4389,"context_line":"                }"},{"line_number":4390,"context_line":"            }"},{"line_number":4391,"context_line":"            wr \u003d rb._build_weighted_replicas_by_tier()"},{"line_number":4392,"context_line":"            for tier_len, expected in expectations.items():"},{"line_number":4393,"context_line":"                self.assertEqual(expected, {t: ceil(r) for (t, r) in"},{"line_number":4394,"context_line":"                                            wr.items() if len(t) \u003d\u003d tier_len})"},{"line_number":4395,"context_line":""},{"line_number":4396,"context_line":""},{"line_number":4397,"context_line":"class TestRingBuilderDispersion(unittest.TestCase):"}],"source_content_type":"text/x-python","patch_set":4,"id":"87d38322_efd926f3","line":4394,"range":{"start_line":4391,"start_character":12,"end_line":4394,"end_character":43},"in_reply_to":"2cfcb5ab_0f8bf424","updated":"2024-02-05 18:30:12.000000000","message":"Done","commit_id":"a94343863fa8e578d6555324074caddf50d7d5ee"}]}
