get wrong checkpoint id when put

[ZhCoding]

In put interface of RedisSaver, checkpoint_id is from config，but in that of InMemorySaver, it is from checkpoint.

langgraph-redis/langgraph/checkpoint/redis/__init__.py

Line 248 in 78222c9

checkpoint_id = (

[tylerhutcherson]

We will investigate. Can you share a bit more comparison here in what you found? Thanks @ZhCoding

[ZhCoding]

We will investigate. Can you share a bit more comparison here in what you found? Thanks @ZhCoding
RedisSaver

    def put(
        self,
        config: RunnableConfig,
        checkpoint: Checkpoint,
        metadata: CheckpointMetadata,
        new_versions: ChannelVersions,
    ) -> RunnableConfig:
        """Store a checkpoint to Redis."""
        """other code"""
        checkpoint_id = (
            configurable.pop("checkpoint_id", configurable.pop("thread_ts", ""))
            or thread_ts
        )
        """other code"""

InMemorySaver

    def put(
        self,
        config: RunnableConfig,
        checkpoint: Checkpoint,
        metadata: CheckpointMetadata,
        new_versions: ChannelVersions,
    ) -> RunnableConfig:
        """Save a checkpoint to the in-memory storage.

        This method saves a checkpoint to the in-memory storage. The checkpoint is associated
        with the provided config.

        Args:
            config: The config to associate with the checkpoint.
            checkpoint: The checkpoint to save.
            metadata: Additional metadata to save with the checkpoint.
            new_versions: New versions as of this write

        Returns:
            RunnableConfig: The updated config containing the saved checkpoint's timestamp.
        """
        """other code"""
        self.storage[thread_id][checkpoint_ns].update(
            {
                checkpoint["id"]: (
                    self.serde.dumps_typed(c),
                    self.serde.dumps_typed(get_checkpoint_metadata(config, metadata)),
                    config["configurable"].get("checkpoint_id"),  # parent
                )
            }
        )
        """other code"""

config["configurable"].get("checkpoint_id") is the parent checkpoint_id, and checkpoint["id"] is id of current checkpoint that we want to use as key

[bsbodden]

THis has been addressed in 0.1.0