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.

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

Support sqlite as OMOP backend #73

Open
eroell opened this issue Jan 9, 2025 · 0 comments
Open

Support sqlite as OMOP backend #73

eroell opened this issue Jan 9, 2025 · 0 comments
Labels
enhancement New feature or request

Comments

@eroell
Copy link
Collaborator

eroell commented Jan 9, 2025

Description of feature

Suggested approach:

  1. Allow sqlite connection to be passed

    def _check_valid_backend_handle(backend_handle) -> None:
    if not isinstance(backend_handle, duckdb.duckdb.DuckDBPyConnection):
    raise TypeError("Expected backend_handle to be of type DuckDBPyConnection.")

  2. Include sqlite as additional parametrization in test suites
    test_omop.py

  3. Fix the tests that break

  4. Cover some miscellaneous issues that pop up along this path

@eroell eroell added the enhancement New feature or request label Jan 9, 2025
@eroell eroell assigned eroell and unassigned eroell Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant