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

Duckdb instead of sqlite? #238

Open
baslat opened this issue Jul 10, 2024 · 1 comment
Open

Duckdb instead of sqlite? #238

baslat opened this issue Jul 10, 2024 · 1 comment

Comments

@baslat
Copy link

baslat commented Jul 10, 2024

Is your feature request related to a problem? Please describe.
Very useful tool! I'm wondering about forking it to create duckdb instead of sqlite as duckdb has some analytical advantages over sqlite.

Describe the solution you'd like
I'm interested in knowing your thoughts on how much work this would be, where to start, whether it would be something "official" you would consider, etc? I'm not specifically asking you to do it (although if you want to that would be cool), more to point me in the right direction so that I might have a tinker.

Describe alternatives you've considered
I tried converting the output sqlite db files to duckdb using sqlite2duckdb but get invalid time errors (I think it's due to some durations being stored as all 0s in sqllite and that trying to be converted to a timestamp by sqlite2duckdb, but haven't confirmed).

@tcgoetz
Copy link
Owner

tcgoetz commented Aug 30, 2024

I don't think you need to fork. Feel free to offer a pull request with the needed functionality. GarminDb releis heavily on SqlAlchemy. If SqlAchemy support DuckDB, then adding support for DuckDB to GarminDb should be relatively easy.

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

No branches or pull requests

2 participants