Michael Schuster

Results 57 comments of Michael Schuster

@AlexejPenner all of it ;) I'm guessing the filtering pipelines by latest run status is the only major change that happened since your last review

@htahir1 The only concern I have with merging this is that we will allow multiple stack components of a type at some point, and we will have to either remove...

@jlopezpena Thanks for the information, we'll definitely have to rework this once 3.14 is released. I don't think this will be a quick/easy fix though, so we probably won't get...

The code/CLI commands for deploying should only be shown if `snapshot.body.deployable` is True: - Deploy (SDK): Maybe when showing this in the frontend, we simply use the snapshot name as...

@ablazejuk The PR is still failing the linting. Can you run `bash scripts/format.sh` to resolve this please

@stefannica Can you explain what the supposed circular import is here? If circular imports really were an issue here, it would not actually be running because they cause an exception....

I have more of a general question: Your current implementation solves how an artifact was saved, but how do we expose the way it was loaded? E.g. even if an...

I see, makes sense let's handle that separately. I think the problem is mostly on how to visualize what type of input it is right? Could just be a different...

That's also why the input type can't be stored in the artifact table, one artifact might be different types of inputs for different steps/runs or even the same exact step