search

kdpsingh / clinspacy

Clinical Natural Language Processing using spaCy, scispacy, and medspacy
Other
96 stars 19 forks source link

Change name of ID column #6

Closed kdpsingh closed 3 years ago

kdpsingh commented 3 years ago

If the text being processed contains the entity "id", this leads bind_clinspacy() to fail because of duplicate columns prior to the spread.

I will plan to rename the id column to "clinspacy_id" to prevent this error.

kdpsingh commented 3 years ago

Fixed.

I renamed the id column to clinspacy_id in https://github.com/ML4LHS/clinspacy/commit/d6aff34828619005ec0f29042b8505f0a0de5fa6. This prevents the hassle introduced by having an entity named "id" which interferes with the bind_clinspacy() function.