Skip to content

Fix unit tests in all versions + update requirements - #59

Merged
alexisdurieux merged 11 commits into
masterfrom
bug/dss14-sc-293969-model-error-analysis-tends-to-not-install
Feb 10, 2026
Merged

alexisdurieux merged 11 commits into
masterfrom
bug/dss14-sc-293969-model-error-analysis-tends-to-not-install

Conversation

@alexisdurieux

Copy link
Copy Markdown
Contributor

No description provided.

@alexisdurieux alexisdurieux self-assigned this Jan 29, 2026
@nicolasdalsass

Copy link
Copy Markdown
Contributor

@alexisdurieux Can you have a look at running the tests on github actions please ? We have an example here : https://github.com/dataiku/dss-plugin-nlp-offline-translation/tree/master/.github/workflows

@alexisdurieux alexisdurieux added the dependencies Pull requests that update a dependency file label Jan 29, 2026
@alexisdurieux
alexisdurieux requested a review from gsaint February 2, 2026 10:44
@alexisdurieux

Copy link
Copy Markdown
Contributor Author

@gsaint I put you as a reviewer while I am still figuring out how to fix integration tests. The issue is related to the migration to dku54 a while back I presume

Comment thread code-env/python/spec/requirements.txt
Comment thread python-lib/dku_error_analysis_decision_tree/tree.py
@alexisdurieux
alexisdurieux requested a review from gsaint February 3, 2026 09:58
@alexisdurieux

Copy link
Copy Markdown
Contributor Author

@gsaint I fixed the IT tests finally :)

@gsaint gsaint left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on new and legacy versions of python from 3.6 to 3.14.
Thanks for your patience.
Looks good to me ✅ .

@alexisdurieux
alexisdurieux merged commit a7ef1f1 into master Feb 10, 2026
10 checks passed
@alexisdurieux
alexisdurieux deleted the bug/dss14-sc-293969-model-error-analysis-tends-to-not-install branch February 10, 2026 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants