Skip to content

feat: configure aggregator strategy and params in task creation UI - #1291

Draft
ale18V wants to merge 1 commit into
feat/fedproxfrom
feat/webapp-task-creation-aggregator-params
Draft

ale18V wants to merge 1 commit into
feat/fedproxfrom
feat/webapp-task-creation-aggregator-params

Conversation

@ale18V

@ale18V ale18V commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator

What & why

This PR updates the webapp task creation form to support the unified aggregator configuration architecture.
Previously, the task creation UI bound to legacy fields like `trainingInformation.maxShareValue` and `privacy.byzantineFaultTolerance`.
This change binds aggregator parameters to `trainingInformation.aggregationParams`, adds support for FedProx in the UI, and validates the task using `aggregationConfigSchema`.
Obsolete form fields for BFT under privacy are removed, ensuring created tasks conform strictly to the modern task schema.

Technical plan

The technical plan aligns the webapp form with the core aggregator configuration schema.
Form fields for Byzantine, Secure, and FedProx aggregators bind directly to properties of `aggregationParams`.
The local form validation schema adopts `aggregationConfigSchema` directly, simplifying the validation pipeline.
End-to-end Cypress tests verify task submission and deserialization with custom aggregation parameters.

Deviations from plan

None.

Todo

None.

@ale18V
ale18V added this pull request to stack #1292 November 12, 2026 15:58
@ale18V
ale18V force-pushed the feat/webapp-task-creation-aggregator-params branch 2 times, most recently from dfb1d44 to 542bb01 Compare November 12, 2026 16:22
@ale18V
ale18V force-pushed the feat/webapp-task-creation-aggregator-params branch from 542bb01 to 9e85bd0 Compare November 12, 2026 17:14

This branch has not been deployed

No deployments
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.

1 participant