Skip to content

content: update PI Mila affiliation and role order - #32

Merged
chengjiahao1234 merged 1 commit into
mainfrom
feature/team-pi-mila-affiliation
Nov 13, 2026
Merged

chengjiahao1234 merged 1 commit into
mainfrom
feature/team-pi-mila-affiliation

Conversation

@chengjiahao1234

Copy link
Copy Markdown
Collaborator

Type of Change

  • Content Update (team)

Summary

Update Mahdi S. Hosseini's Mila affiliation to Associate Academic Member and place the PI's roles in the requested order.

The Team profile now reads:

Mahdi S. Hosseini
Lab Director, Assistant Professor at Concordia University
Associate Academic Member of Mila-Quebec AI Institute
Adjunct Professor at McGill University

Changes Made

  • _data/principal_investigator.yml: replace the former Affiliate Member title and order the role lines as Concordia, Mila, then McGill.
  • _pages/home.md: update the About us paragraph to the same Mila title and affiliation order for consistency.
  • Preserve the PI's name styling, portrait, email, profile URLs, and all other team data. No template or CSS changes.

Testing Checklist

  • Changes display correctly on desktop (1920x1080)
  • Changes display correctly on tablet (768x1024)
  • Changes display correctly on mobile (375x667)
  • Tested in light theme
  • Tested in dark theme
  • Local images load correctly; all six PI profile/contact URLs remain unchanged
  • No console errors or horizontal overflow
  • Ran bundle exec jekyll serve locally without errors
  • Jekyll build and git diff --check passed
  • YAML and rendered-page assertions confirm the exact role text and order

Screenshots (if applicable)

Desktop, tablet, and mobile screenshots were reviewed locally in both themes; not attached.

Related Issues

None.

Additional Notes

  • Uses the wording supplied by the PI. This updates the lab website only, not Mila's external directory.
  • Existing non-blocking Ruby native-extension warnings remain; the build and local server succeed.
  • Ready for admin review; no merge or production deployment has been performed.

By submitting this PR, I confirm:

  • I have tested these changes locally
  • I have followed the code standards in DEVELOPMENT.md
  • I have updated documentation if needed (not required for this content-only update)
  • I am ready for this to be reviewed and merged

@chengjiahao1234
chengjiahao1234 merged commit b54ee5d into main Nov 13, 2026
1 check passed
@chengjiahao1234
chengjiahao1234 deleted the feature/team-pi-mila-affiliation branch November 13, 2026 18:43
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