Skip to content

Federated Learning WG update#98

Merged
holgerroth merged 3 commits intoProject-MONAI:masterfrom
holgerroth:fl_wg_update
Mar 11, 2026
Merged

Federated Learning WG update#98
holgerroth merged 3 commits intoProject-MONAI:masterfrom
holgerroth:fl_wg_update

Conversation

@holgerroth
Copy link
Copy Markdown
Contributor

@holgerroth holgerroth commented Mar 11, 2026

Introducing new working group chair Dr. Rafael Dias (KCL).
Adding Emeritus WG Chairs Section

Summary by CodeRabbit

  • Documentation

    • Updated team member information on Advisory Board and working group pages
    • Added Emeritus Working Group Chairs section with member profiles
    • Refreshed group lead profiles with current affiliations and contact information
  • Chores

    • Updated project gitignore configuration

@holgerroth holgerroth requested a review from zephyrie March 11, 2026 15:48
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 11, 2026

Walkthrough

The pull request updates team leadership and organizational structure across multiple HTML files. It reorganizes a team member to an emeritus role, adds a new leadership member to several group positions, and updates .gitignore with additional ignore patterns.

Changes

Cohort / File(s) Summary
Configuration
.gitignore
Added .claude/ and .DS_Store entries to the ignore list.
Team Member Updates
about.html, wg_federated_learning.html, working-groups.html
Updated Advisory Board and group lead positions: moved one team member to Emeritus Working Group Chairs, replaced with new leadership member across multiple group pages. Updated associated images, names, affiliations, and profile links consistently.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A hop and a shuffle, the team takes its shape,
New faces and roles in a fresh landscape!
Some leaders ascend to emeritus rest,
While new rabbits join to do their very best—
The burrow's organized, link by link,
From .gitignore to what all will think!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Federated Learning WG update' is partially related to the changeset. While it accurately describes updates to the Federated Learning working group, the changes also include updates to other working groups, personnel changes in multiple locations, and a new Emeritus section across multiple HTML files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@holgerroth holgerroth requested a review from garciadias March 11, 2026 15:52
Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
about.html (1)

373-378: Make the emeritus status explicit on the card.

The section heading gives the context, but the card itself still reads like an active role. Labeling it as Emeritus Federated Learning Chair avoids ambiguity in screenshots, snippets, and search results.

Proposed tweak
-                <p class="text-sm text-brand-primary">Federated Learning Chair</p>
+                <p class="text-sm text-brand-primary">Emeritus Federated Learning Chair</p>
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@about.html` around lines 373 - 378, The card for "Dr. Sai Praneeth
Karimireddy" currently shows "Federated Learning Chair" and should explicitly
indicate emeritus status; update the displayed role text (the element containing
"Federated Learning Chair" — the <p> with class "text-sm text-brand-primary") to
read "Emeritus Federated Learning Chair" (or adjust the <h5> title to include
"Emeritus") so the card itself clearly shows emeritus status in all contexts.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@about.html`:
- Around line 373-378: The card for "Dr. Sai Praneeth Karimireddy" currently
shows "Federated Learning Chair" and should explicitly indicate emeritus status;
update the displayed role text (the element containing "Federated Learning
Chair" — the <p> with class "text-sm text-brand-primary") to read "Emeritus
Federated Learning Chair" (or adjust the <h5> title to include "Emeritus") so
the card itself clearly shows emeritus status in all contexts.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f77faf21-2dfe-4e13-9ccb-68a7fa85786b

📥 Commits

Reviewing files that changed from the base of the PR and between 7e6a46c and 8a52021.

⛔ Files ignored due to path filters (1)
  • assets/img/RafaelDias.jpg is excluded by !**/*.jpg
📒 Files selected for processing (4)
  • .gitignore
  • about.html
  • wg_federated_learning.html
  • working-groups.html

@holgerroth holgerroth merged commit da8ce69 into Project-MONAI:master Mar 11, 2026
1 check passed
@holgerroth holgerroth deleted the fl_wg_update branch March 11, 2026 15:53
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