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

Deprecate Support for OMOPv4+ #261

Open
aguynamedryan opened this issue Oct 18, 2019 · 0 comments
Open

Deprecate Support for OMOPv4+ #261

aguynamedryan opened this issue Oct 18, 2019 · 0 comments

Comments

@aguynamedryan
Copy link
Contributor

We no longer need to support this data model. Stripping out all the code would actually be disruptive and time-consuming, but we'll no longer be actively testing this data model going forward.
@aguynamedryan aguynamedryan mentioned this issue Oct 18, 2019
Closed
aguynamedryan added a commit that referenced this issue Oct 18, 2019
@aguynamedryan
Travis: don't test omopv4_plus, temp tables #261, #264
24031c4
@aguynamedryan aguynamedryan mentioned this issue Oct 18, 2019
Closed
aguynamedryan added a commit to outcomesinsights/generalized_data_model that referenced this issue Nov 19, 2019
@aguynamedryan
README: remove omopv4_vocabulary_id column, refs outcomesinsights/con…
f4f2b09 
…ceptql#261
aguynamedryan added a commit that referenced this issue Dec 9, 2019
@aguynamedryan
Work with new Lexicon tables
f47ec07 
This commit was supposed to address a small change in Lexicon,
but resulted in tearing out a lot of defunct or deprecated code,
namely:
- Removed (almost) all references to Impala, refs #262
- Removed a lot of OMOP-related tests, refs #261
- More work on defaulting to GDM, refs #265
aguynamedryan added a commit that referenced this issue Jan 17, 2020
@aguynamedryan
Drops all test results for omopv4_plus
e9434a9 
OMOPv4+ is deprecated (#261) and we're no longer actively running
the OMOPv4+ tests, so I'm dropping the expected results, mostly
because they were a pain to deal with in the previous commit.
aguynamedryan added a commit that referenced this issue Jan 17, 2020
@aguynamedryan
Remove more OMOP-specific code, refs #261
85bb72a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@aguynamedryan