Skip to content

Develop#357

Open
mvankessel-EMC wants to merge 36 commits intorcfrom
develop
Open

Develop#357
mvankessel-EMC wants to merge 36 commits intorcfrom
develop

Conversation

@mvankessel-EMC
Copy link
Collaborator

No description provided.

mvankessel-EMC and others added 30 commits November 6, 2025 13:05
* Updated colapse algorithm

* updated setup

* added optional skips

* added if to check if cdmconnector is installed

* updated setup

* updated test

* dep update fixes

* Updated setup

* fixed eunomia download?

* made tests faster

* updated tests

* revamped tests

* updated checks

* removed dfer

* updated eunomia_data_folder

* updated setup

* updated path

* fixed reproduced error from actions

* comment out CG to see if DatabaseConnector overrides some CDMConnector functions

* added stats for setNames

* explicitly load stats

* updated tests

* updated tests
* Moved dependencies to suggests

* Updated setup

* download through CDMConnector

* download both

* Updated setup

* Replaced stringr with stringi

* Fixed tests and updated setup

* Fixed warnings

* removed png

* removed stray stringr call

* updated action

* updated arg

* ignore vignettes

* moved args

* updated example
@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mvankessel-EMC
Copy link
Collaborator Author

@ablack3 Your fix seems to work for ODBC Spark. But now ODBC Snowflake is failing. Not sure if that is internal to TreatmentPatterns or somthing similar to your suggested fix for Spark.

@ablack3
Copy link
Collaborator

ablack3 commented Feb 14, 2026

@ablack3 Your fix seems to work for ODBC Spark. But now ODBC Snowflake is failing. Not sure if that is internal to TreatmentPatterns or somthing similar to your suggested fix for Spark.

Hmm can you point me to the error? It looks like the current develop branch is passing on snowflake.

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