-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add workflow_complete call to LabelRowV2 #319
Conversation
851429f
to
b8fa13d
Compare
daac53f
to
07b226f
Compare
pass | ||
|
||
|
||
class LabelWorkflowGraphNodePayload(base_orm.BaseORM): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mhh I think we're trying to get rid of the base orms in general. Is there a good reason to use those over dataclasses?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not really, mostly for consistency and because it's serialisable to json by default.
dataclasses.asdict(obj) should be equally fine, I suppose.
If we want to bring pydantic and match server-side approach I guess is a separate question. Totally happy to discuss, but would vote for not making anything like that in scope of this PR
9ce7e73
to
1b25efd
Compare
1b25efd
to
b421617
Compare
Co-authored-by: david-babuschkin-encord <126681387+david-babuschkin-encord@users.noreply.github.com>
Co-authored-by: david-babuschkin-encord <126681387+david-babuschkin-encord@users.noreply.github.com>
Co-authored-by: david-babuschkin-encord <126681387+david-babuschkin-encord@users.noreply.github.com>
Co-authored-by: david-babuschkin-encord <126681387+david-babuschkin-encord@users.noreply.github.com>
No description provided.