Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fixed vtm pipeline name #122

Merged
merged 4 commits into from
Jul 18, 2024
Merged

Fixed vtm pipeline name #122

merged 4 commits into from
Jul 18, 2024

Conversation

jenniferjiangkells
Copy link
Member

@jenniferjiangkells jenniferjiangkells commented Jul 18, 2024 •
edited
Loading

Fixes #121

Typo in create_pipeline function means the VTM converter function is never added to the default pipeline so there's no VTM -> VMP conversion step 😓

@jenniferjiangkells jenniferjiangkells added the type: bug Something isn't working label Jul 18, 2024
@jenniferjiangkells jenniferjiangkells self-assigned this Jul 18, 2024
@jenniferjiangkells jenniferjiangkells linked an issue Jul 18, 2024 that may be closed by this pull request
Default Meds Allergy annotator pipeline does not include VTM -> VMP step #121
Closed
@jenniferjiangkells
Copy link
Member Author

jenniferjiangkells commented Jul 18, 2024 •
edited
Loading

FYI - @jamesbrandreth - you might want to re-run any meds/allergy if you need that conversion step

@jenniferjiangkells jenniferjiangkells merged commit debff0e into master Jul 18, 2024
1 check passed
@jenniferjiangkells jenniferjiangkells deleted the bugfix/med-pipeline-bug branch July 18, 2024 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jenniferjiangkells jenniferjiangkells

Labels
type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Default Meds Allergy annotator pipeline does not include VTM -> VMP step
1 participant
@jenniferjiangkells