fix: adding conversion to string for notif id comparison (#638)

3fa35f9e · Neelay Shah · GitHub · e3857ba0 · 3fa35f9e
Unverified Commit 3fa35f9e authored Apr 11, 2025 by Neelay Shah Committed by GitHub Apr 11, 2025
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

container/deps/vllm/vllm_v0.7.2-dynamo-kv-disagg-patch.patch container/deps/vllm/vllm_v0.7.2-dynamo-kv-disagg-patch.patch +1 -1

No files found.
--- a/container/deps/vllm/vllm_v0.7.2-dynamo-kv-disagg-patch.patch
+++ b/container/deps/vllm/vllm_v0.7.2-dynamo-kv-disagg-patch.patch
@@ -4214,7 +4214,7 @@ index 819b81fb..2891854b 100644
 +            for notifs in all_new_notifs:
 +                for req_ids in notifs.values():
 +                    for req_id in req_ids:
-+                        request_notif_counter[req_id] += 1
+                        request_notif_counter[req_id.decode("utf-8")] += 1
 +
 +            if request_notif_counter:
 +                logger.debug("Request notif counter: %s", request_notif_counter)