add SIDER #438
add SIDER #438
Conversation
data/tabular/SIDER/meta.yaml
Outdated
| names: | ||
| - noun: metabolism and nutrition disorders | ||
| - id: product_issues | ||
| description: product issues |
There was a problem hiding this comment.
what does this mean? what is an example of a product issue?
There was a problem hiding this comment.
@AdrianM0 do you know what product issues is supposed to mean? Does this noun fit the column name?
data/tabular/SIDER/meta.yaml
Outdated
| type: boolean | ||
| names: | ||
| - noun: gastrointestinal disorders | ||
| - id: social_circumstances |
There was a problem hiding this comment.
is this a disorder?
There was a problem hiding this comment.
or is this some kind of disorder?
There was a problem hiding this comment.
@kjappelbaum I can also remove these two?
There was a problem hiding this comment.
also an option if you cannot figure out what those are about
data/tabular/SIDER/meta.yaml
Outdated
| - id: investigations | ||
| description: eye disorders | ||
| type: boolean | ||
| names: | ||
| - noun: eye disorders | ||
| - id: musculoskeletal_and_connective_tissue_disorders | ||
| description: investigations | ||
| type: boolean | ||
| names: | ||
| - noun: investigations |
There was a problem hiding this comment.
isn't there a mismatch between your column names and the names?
They seem one off.
Generally, it is probably better if we can manage to get multiple names to increase the diversity of the training data
There was a problem hiding this comment.
fixed this now, added one noun to each
data/tabular/SIDER/meta.yaml
Outdated
| - id: investigations | ||
| description: investigations | ||
| type: boolean | ||
| names: | ||
| - noun: investigations | ||
| - noun: medical investigations |
There was a problem hiding this comment.
do we want to sample this one/do you know what this one means?
data/tabular/SIDER/meta.yaml
Outdated
| - id: injury_poisoning_and_procedural_complications | ||
| description: injury, poisoning and procedural complications | ||
| type: boolean | ||
| names: | ||
| - noun: injury, poisoning and procedural complications | ||
| - noun: injuries, poisonings, and complications from procedures |
There was a problem hiding this comment.
"complications from procedures" -> this does not sound like a drug side effect to me ...
There was a problem hiding this comment.
refers to post-surgery complications likely
|
Thanks for all the updates, and sorry for being so picky! |
|
@kjappelbaum this can be closed I think |
|
With closed, do you mean merged? |
data/tabular/SIDER/meta.yaml
Outdated
| - id: injury_poisoning_and_procedural_complications | ||
| description: injury, poisoning and procedural complications | ||
| type: boolean | ||
| names: | ||
| - noun: injury, poisoning and procedural complications | ||
| - noun: injuries and poisonings |
There was a problem hiding this comment.
i'm still not sure about this. You put in SMILES and then predict something about "procedural complications" which sounds more like surgery
There was a problem hiding this comment.
@AdrianM0, what are your thoughts on this one?
|
can you, as discussed, remove the columns we are uncertain about? (https://discord.com/channels/@me/1138533229381550181/1168595210876485673) |
One more dataset from moleculeNet. 🥳