[PR #1073] [MERGED] Don't include lambdas in __eq__ for BinOp #1870

Closed
opened 2026-02-20 17:45:44 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraph/pull/1073
Author: @nfcampos
Created: 7/19/2024
Status: Merged
Merged: 7/22/2024
Merged by: @nfcampos

Base: mainHead: nc/19jul/binop-lambda-eq


📝 Commits (1)

  • 4dc1195 Don't include lambdas in eq for BinOp

📊 Changes

1 file changed (+5 additions, -3 deletions)

View changed files

📝 libs/langgraph/langgraph/channels/binop.py (+5 -3)

📄 Description

When using forward refs, inline lambdas in Annotated are re-evaluated for every subclass, thus making the comparison fail


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/langgraph/pull/1073 **Author:** [@nfcampos](https://github.com/nfcampos) **Created:** 7/19/2024 **Status:** ✅ Merged **Merged:** 7/22/2024 **Merged by:** [@nfcampos](https://github.com/nfcampos) **Base:** `main` ← **Head:** `nc/19jul/binop-lambda-eq` --- ### 📝 Commits (1) - [`4dc1195`](https://github.com/langchain-ai/langgraph/commit/4dc1195bcdd8642472978ef4d896781dd3e5b7f6) Don't include lambdas in __eq__ for BinOp ### 📊 Changes **1 file changed** (+5 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `libs/langgraph/langgraph/channels/binop.py` (+5 -3) </details> ### 📄 Description When using forward refs, inline lambdas in Annotated are re-evaluated for every subclass, thus making the comparison fail --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-20 17:45:44 -05:00
yindo closed this issue 2026-02-20 17:45:44 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langgraph#1870