test(webapp): cover decentralized browser training across browsers by ale18V · Pull Request #1239 · epfml/disco

ale18V · 2026-10-01T14:44:14Z

What & why

Adds end-to-end webapp coverage for decentralized browser-to-browser training, as requested in #1196 and the review of #1230. Four real browser participants (two Chrome and two Firefox) train the default MNIST task through the webapp against a live Disco server over WebRTC. The run exercises Chrome–Chrome, Firefox–Firefox, and Chrome–Firefox model exchange.

Technical plan

Keep the test with the other webapp training suites. Start the standard built Disco server and a Vite webapp, then run four Cypress browser processes on a shared Xvfb display in CI. Assert that all participants complete 20 epochs and 10 model-sharing rounds.

Deviations from plan

None. This replaces the Node participant prototype in #1232 with browser-to-browser collaboration.

Todo

None.

ale18V · 2026-10-10T15:45:16Z

I've also refactored the support folder to have a dedicated support/training folder

JulienVig

Thanks for the great work! I mostly have nitpicking comments but I'm requesting changes because the CI is currently flaky. I just merged #1259 which should fix the error. You can request the review again once you rebased and see that the CI passes consistently

JulienVig

All good! Just curious about why you ended up switching a firefox for edge rather than a chrome

JulienVig · 2026-10-16T16:41:18Z

 import { createInterface } from "node:readline";

-const browsers = ["chrome", "edge", "firefox", "firefox"];
+const browsers = ["chrome", "chrome", "edge", "firefox"];


You mentioned it would be a better idea to swap out a chrome for edge rather than a firefox. Did someting make you change your mind?
Is this linked to the commit mentioning a "geckodriver race condition"?

JulienVig · 2026-10-16T16:52:36Z

-This runs both Titanic and LUS COVID with a mocked server. The command starts
-and stops Vite itself; port 1351 must be available.
+The local command runs both Titanic and LUS COVID with a mocked server.
+The decentralized command runs MNIST across Chrome and Firefox participants with a real server; install Chrome and Firefox (plus `xvfb` and `xauth` on headless Linux).


Suggested change

The decentralized command runs MNIST across Chrome and Firefox participants with a real server; install Chrome and Firefox (plus `xvfb` and `xauth` on headless Linux).

The decentralized command runs MNIST across Chrome, Edge and Firefox participants with a real server; install Chrome, Edge and Firefox (plus `xvfb` and `xauth` on headless Linux).

JulienVig · 2026-10-16T16:54:32Z

+      stdio: "pipe",
+      env: {
+        ...process.env,
+        DISCO_TRAINING_E2E: "1",


DISCO_TRAINING_E2E is already set in process.env isn't it? Do you need to set it again here?

JulienVig · 2026-10-16T16:56:39Z

+    cy.contains("button", "next").click();
+
+    cy.contains("button", "group").click();
+    cy.task<string[]>("readdir", "../datasets/CIFAR10").then((files) =>


Suggested change

cy.task<string[]>("readdir", "../datasets/CIFAR10").then((files) =>

// Train on dummy images from CIFAR10, this test doesn't care about actual performance

cy.task<string[]>("readdir", "../datasets/CIFAR10").then((files) =>

JulienVig · 2026-10-16T17:00:21Z

+  test-e2e-decentralized:
+    name: decentralized webapp (mnist)
+    runs-on: ubuntu-latest
+    timeout-minutes: 10


Suggested change

timeout-minutes: 10

timeout-minutes: 12

The current timeouts add up to ~8min + 4 browser startut, I think 10min is a bit tight and we want to leave enough time to upload the screenshots if it fails

ale18V mentioned this pull request Oct 2, 2026

test(webapp): cover decentralized browser training #1232

Closed

ale18V force-pushed the test/browser-browser-cypress-ci branch from 55697bc to e489b7f Compare October 2, 2026 09:51

ale18V force-pushed the test/browser-browser-cypress-ci branch from e489b7f to 0e054da Compare October 2, 2026 12:33

ale18V mentioned this pull request Oct 2, 2026

Test federated webapp training #1230

Merged

ale18V force-pushed the test/browser-browser-cypress-ci branch from 0e054da to e76d17c Compare October 3, 2026 11:17

ale18V force-pushed the test/browser-browser-cypress-ci branch from e76d17c to a7b402f Compare October 3, 2026 11:20

This was referenced Oct 3, 2026

Document WebGPU feasibility and browser training benchmarks #1240

Draft

Decentralized GPT weight exchange exceeds WebRTC message size between Chrome and Firefox #1246

Open

ale18V force-pushed the test/browser-browser-cypress-ci branch 2 times, most recently from ee165f3 to f335779 Compare October 5, 2026 21:57

ale18V changed the base branch from test/decentralized-webapp-ci to main October 5, 2026 21:57

ale18V force-pushed the test/browser-browser-cypress-ci branch from f335779 to f6cc2e0 Compare October 6, 2026 07:12

ale18V changed the title ~~test(webapp): probe Chrome–Firefox decentralized training~~ test(webapp): cover decentralized browser training across browsers Oct 6, 2026

ale18V force-pushed the test/browser-browser-cypress-ci branch 2 times, most recently from 0112c7a to 6da8cc1 Compare October 10, 2026 13:59

ale18V marked this pull request as ready for review October 10, 2026 14:01

ale18V requested a review from JulienVig October 10, 2026 14:13

ale18V added this pull request to stack #1257 October 11, 2026 10:39

ale18V force-pushed the test/browser-browser-cypress-ci branch from be145af to bdc2a99 Compare October 12, 2026 12:25

JulienVig requested changes Oct 12, 2026

View reviewed changes

ale18V added 5 commits October 15, 2026 11:27

test(webapp): cover decentralized browser training across browsers

2604193

test(webapp): start standard server for browser collaboration

b93767a

refactor(webapp): group training test support by mode

a49571d

style(webapp): format moved federated peer

e74568a

test(webapp): include Edge in decentralized browser coverage

1b4816a

ale18V added 3 commits October 15, 2026 11:27

test(webapp): fix decentralized E2E browser matrix

4549764

test(webapp): scope decentralized training timeouts

161794e

test(webapp): address review feedback for decentralized browser tests

a84faef

ale18V force-pushed the test/browser-browser-cypress-ci branch from 50a5eff to a84faef Compare October 15, 2026 09:32

ale18V requested a review from JulienVig October 15, 2026 09:33

ale18V added 2 commits October 15, 2026 11:52

test(webapp): add timeout for final decentralized model aggregation

0f24e2d

test(webapp): use single Firefox to prevent geckodriver race condition

c7aec64

JulienVig approved these changes Oct 16, 2026

View reviewed changes

	The decentralized command runs MNIST across Chrome and Firefox participants with a real server; install Chrome and Firefox (plus `xvfb` and `xauth` on headless Linux).
	The decentralized command runs MNIST across Chrome, Edge and Firefox participants with a real server; install Chrome, Edge and Firefox (plus `xvfb` and `xauth` on headless Linux).

	cy.task<string[]>("readdir", "../datasets/CIFAR10").then((files) =>
	// Train on dummy images from CIFAR10, this test doesn't care about actual performance
	cy.task<string[]>("readdir", "../datasets/CIFAR10").then((files) =>

Conversation

ale18V commented Oct 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What & why

Technical plan

Deviations from plan

Todo

Uh oh!

ale18V commented Oct 10, 2026

Uh oh!

JulienVig left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

JulienVig left a comment

Choose a reason for hiding this comment

Uh oh!

JulienVig Oct 16, 2026

Choose a reason for hiding this comment

Uh oh!

JulienVig Oct 16, 2026

Choose a reason for hiding this comment

Uh oh!

JulienVig Oct 16, 2026

Choose a reason for hiding this comment

Uh oh!

JulienVig Oct 16, 2026

Choose a reason for hiding this comment

Uh oh!

JulienVig Oct 16, 2026

Choose a reason for hiding this comment

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

ale18V commented Oct 1, 2026 •

edited

Loading