X Tutup
Skip to content

Fix author name typo: Abous-Rjeili → Abou-Rjeili#3038

Merged
pplantinga merged 1 commit intospeechbrain:developfrom
georgesabr:fix/author-name-typo
Mar 1, 2026
Merged

Fix author name typo: Abous-Rjeili → Abou-Rjeili#3038
pplantinga merged 1 commit intospeechbrain:developfrom
georgesabr:fix/author-name-typo

Conversation

@georgesabr
Copy link
Collaborator

@georgesabr georgesabr commented Mar 1, 2026

What does this PR do?

Fixes a typo in the author name "Georges Abous-Rjeili" → "Georges Abou-Rjeili" across 6 files (comments/headers only, no code changes).

I'm the author, and sorry it was bothering me way more than It should haha. My original contribution was in
https://github.com/mravanelli-mila/speechbrain_tts

Affected files:

  • recipes/LibriTTS/TTS/mstacotron2/hparams/train.yaml
  • recipes/LibriTTS/TTS/mstacotron2/train.py
  • recipes/LJSpeech/TTS/tacotron2/train.py
  • recipes/LJSpeech/TTS/tacotron2/hparams/train.yaml
  • speechbrain/lobes/models/MSTacotron2.py
  • speechbrain/lobes/models/Tacotron2.py

No breaking changes.

Before submitting
  • Did you read the contributor guideline?
  • Did you make sure your PR does only one thing, instead of bundling different changes together?
  • Did you make sure to update the documentation with your changes? (if necessary) — N/A, typo-only fix
  • Did you write any new necessary tests? (not for typos and docs) — N/A, typo fix
  • Did you verify new and existing tests pass locally with your changes? — Only comment/header changes, no functional impact
  • Did you list all the breaking changes introduced by this pull request? — None
  • Does your code adhere to project-specific code style and conventions?

Copy link
Collaborator

@pplantinga pplantinga left a comment

Choose a reason for hiding this comment

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

Names are important! LGTM

@pplantinga pplantinga merged commit 3eb32f6 into speechbrain:develop Mar 1, 2026
5 checks passed
@georgesabr georgesabr deleted the fix/author-name-typo branch March 1, 2026 10:32
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.

2 participants

X Tutup