-
Notifications
You must be signed in to change notification settings - Fork 427
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
Model for annotation metadata #8159
Conversation
270cc77
to
e64ca27
Compare
f8f568d
to
55c030a
Compare
e64ca27
to
9bb1fd9
Compare
h/models/annotation_metadata.py
Outdated
unique=True, | ||
) | ||
|
||
meta = sa.Column( |
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.
metadata can't be a model attribute. I've gone for meta in the model and metadata in the table.
Should both (model and table) but just be meta
?
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.
How about just data
?
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.
Yeah, I've been using this in some follow up PRs and the meta/metadata idea was awful. data
it is
9bb1fd9
to
2c3bfaf
Compare
h/models/annotation_metadata.py
Outdated
unique=True, | ||
) | ||
|
||
meta = sa.Column( |
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.
How about just data
?
55c030a
to
314ec92
Compare
2c3bfaf
to
10c2e26
Compare
10c2e26
to
3265493
Compare
Migration changes over: