Add federated mixed effects algorithms and tests#570
Closed
AndreasKtenidis wants to merge 51 commits intomasterfrom
Closed
Add federated mixed effects algorithms and tests#570AndreasKtenidis wants to merge 51 commits intomasterfrom
AndreasKtenidis wants to merge 51 commits intomasterfrom
Conversation
84dd87c to
4b354a1
Compare
…flow into feature/mixed-effects # Conflicts: # tests/standalone_tests/federated_algorithms/mixed_effects/test_glmm_binary.py # tests/standalone_tests/federated_algorithms/mixed_effects/test_glmm_ordinal.py # tests/standalone_tests/federated_algorithms/mixed_effects/test_lmm.py
1767074 to
6136ef5
Compare
…pecs Nominal variables should not be treated as integers. This commit updates the JSON specifications for various algorithms (Naive Bayes, Logistic Regression, etc.) to ensure strict typing for nominal inputs.
…pecs Updates K-Means and Linear Regression specifications to accept both integer and real values for numerical input fields, ensuring broader compatibility with input data.
Enhance logistic_regression.py to gracefully handle invalid input data.
…DuckDB loader Replaces generic categorical coercion with explicit type mapping in `DuckDBCSVLoader`. This improves type safety and removes the need for `udf_service.py` workarounds. Also adds comprehensive tests for type loading and removes obsolete coercion tests.
Enhances `DuckDBCSVLoader` to correctly handle various CSV delimiters. Adds `test_csv_delimiters.py` to verify delimiter handling.
- Move _build_column_types inside _create_primary_data_table for better encapsulation - Update unit tests to reflect API change - Modify aggregation server template to conditionally apply nodeSelector based on managed_cluster value
Use dict-based lookups to prevent pandas from treating string keys like "0" as positional indices. Fixes duplicated counts.
…istics algorithms.
Fix the duckdb connection conflicts due to non read-only connections.
Added where clause, group_by and aggregate.
The controller will no longer have knowledge of the actual metadata, only the worker. This was changed to be compatible with future pipelines.
Add helper method.
Hard-coded column names have been centralized in variable file.
Remove preprocessing step compatible algorithms logic.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.