search

MechanicalRabbit / OHDSICohortExpressions.jl

reimplementation of OHDSI's Circe JSON->SQL compiler
Other
7 stars 1 forks source link

[FEATURE REQUEST] Support for DuckDB? #6

Open TheCedarPrince opened 8 months ago

TheCedarPrince commented 8 months ago

Following on from a discussion from here: https://github.com/MechanicalRabbit/FunSQL.jl/issues/48#issuecomment-1904268427

@xitology suggested I open a request here for supporting DuckDB in OHDSICohortExpressions.jl. I had tried running a Model against a DuckDB database with the model syntax changed to :postgresql and :sqlite but the computation failed.

What could I do next here @xitology and @clarkevans to help bring this feature to full? Is there anything I could do?

Thanks!

~ tcp :deciduous_tree:

clarkevans commented 2 months ago

@TheCedarPrince What might move this ticket forward is if the 100 person MIMIC sample was converted to CSVS, packaged as a Julia data package with DuckDB loader. We might then start to use it as the primary regression test.

TheCedarPrince commented 2 weeks ago

Thanks to https://github.com/MechanicalRabbit/FunSQL.jl/pull/69, this looks like it can be possible. I can provision this through https://github.com/JuliaHealth/HealthSampleData.jl to provide that sample and start a PR for this.