StateManager: Pass intermediates using normalized values, rename coerce to normalize

[paulmelnikow]

No description provided.

[jbeard4]

I used functools.wrap, and I think I'm using it the right way. Requires this small change to artifax: curvewise-forks/artifax#3

I am now getting this error:

    @classmethod
    def coerce(cls, value: t.Any) -> MyModel:
        if not isinstance(value, MyModel):
>           raise ValueError(f"Can't coerce {type(value).__name__} to {cls.__name__}")
E           ValueError: Can't coerce str to MyModelType

werkit/compute/graph/testing_examples.py:75: ValueError

@paulmelnikow I wanted to discuss it with you, because I fell like this is close, but I don't fully understand the error.

[jbeard4]

Somehow we are ending up in here: https://github.com/metabolize/werkit/blob/main/werkit/compute/graph/testing_examples.py#L75

[jbeard4]

@paulmelnikow This is ready for re-review.

[jbeard4]

@paulmelnikow I'm having trouble understanding why main is failing. Is it failing because of coverage?

[paulmelnikow]

Is it because it needs the artifax change?

[jbeard4]

@paulmelnikow This is ready for re-review.

[paulmelnikow]

Let's update the main branch in our fork from the upstream repo, and then PR the change in artifax onto that and re-tag. Also let's give this a commit message that describes what's changing. Apart from that this looks great.