8000 concepts_postgres: updates (on top of `mimiciv_v2.2_updates`) by schu · Pull Request #1468 · MIT-LCP/mimic-code · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

concepts_postgres: updates (on top of mimiciv_v2.2_updates) #1468

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

Conversation

schu
Copy link
Contributor
@schu schu commented Jan 12, 2023

A few changes on top of https://github.com/MIT-LCP/mimic-code/tree/mimiciv_v2.2_updates to update the Postgres concepts code. Please see the individual commit messages for details.

schu added 3 commits January 12, 2023 10:33
The `search_path` part of `postgres-make-concepts.sql` was not added in
`convert_bigquery_to_postgres.sh` yet. Add it on order to be able to
re-run the conversion.

Follow-up for 547f623
Re-run `convert_bigquery_to_postgres.sh` to update the Postgres
concepts. In particular, remove an old include..

```
\i measurement/differential_detailed.sql
```

..that leads to the following error:

```
psql:postgres-make-concepts.sql:35: error: measurement/differential_detailed.sql: No such file or directory
```
@schu schu mentioned this pull request Jan 12, 2023
1 task
@alistairewj
Copy link
Member

Ah thanks for pulling in those updates, though it's interesting your run sorts first_day_bg.sql after first_day_bg_art.sql, as mine doesn't. first_day_bg.sql needs to come before first_day_bg_art.sql so I will manually make the fix, but it might be worth taking a look to see why it's happening in the future.

@alistairewj alistairewj merged commit 5e05342 into MIT-LCP:mimiciv_v2.2_updates Jan 15, 2023
@alistairewj
Copy link
Member

Nevermind about the above, I forgot I already made them both rely on the bg view to avoid that above inconsistency :)

@schu schu deleted the schu/update-mimic-iv-concepts-postgres branch January 18, 2023 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0