langchain-ai / langchain

🦜🔗 Build context-aware reasoning applications
https://python.langchain.com
MIT License
93.05k stars 14.96k forks source link

DOC: remove_node typo in runnables/ graph #23861

Open mathias-aparicio opened 3 months ago

mathias-aparicio commented 3 months ago

URL

https://api.python.langchain.com/en/latest/runnables/langchain_core.runnables.graph.Graph.html

Checklist

Issue with current documentation:

The code of the remove_node in https://github.com/langchain-ai/langchain/blob/master/libs/core/langchain_core/runnables/graph.py is as follow

 def remove_node(self, node: Node) -> None:
        """Remove a node from the graphm and all edges connected to it."""
        self.nodes.pop(node.id)
        self.edges = [
            edge
            for edge in self.edges
            if edge.source != node.id and edge.target != node.id
        ]
``` graph is spelled graphm

### Idea or request for content:

```python
 def remove_node(self, node: Node) -> None:
        """Remove a node from the **graph** and all edges connected to it."""
        self.nodes.pop(node.id)
        self.edges = [
            edge
            for edge in self.edges
            if edge.source != node.id and edge.target != node.id
        ]

Should replace graphm with graph

g-sree-jith commented 3 months ago

URL: https://github.com/langchain-ai/langchain/pull/23910#event-13407839115

Typo has been fixed(grahm-->graph)