Skip to content

Fix (Notebooks): Corrected typos/small errors in the text cells with explanations#1399

Merged
nickfraser merged 7 commits intoXilinx:devfrom
JP-Amboage:nb-typos
Oct 23, 2025
Merged

Fix (Notebooks): Corrected typos/small errors in the text cells with explanations#1399
nickfraser merged 7 commits intoXilinx:devfrom
JP-Amboage:nb-typos

Conversation

@JP-Amboage
Copy link
Collaborator

Reason for this PR

The Jupyter notebooks had some typos or small errors in the explanations in the text cells.

Changes Made in this PR

I corrected the typos/small mistakes that I found. Only text cells where edited. No changes were done to code cells or output cells.

Testing Summary

  • Only text cells changed so no tests are needed.

Juan P Garcia Amboage and others added 6 commits October 21, 2025 16:59
Signed-off-by: Juan P Garcia Amboage <jgarciaa@XIRJGARCIAA01.amd.com>
Signed-off-by: Juan P Garcia Amboage <jgarciaa@XIRJGARCIAA01.amd.com>
Signed-off-by: Juan P Garcia Amboage <jgarciaa@XIRJGARCIAA01.amd.com>
Signed-off-by: Juan P Garcia Amboage <jgarciaa@XIRJGARCIAA01.amd.com>
@pablomlago pablomlago self-requested a review October 22, 2025 11:07
@nickfraser
Copy link
Collaborator

Oh damn, these failing tests are likely from: #1398 - didn't see that coming!

@JP-Amboage JP-Amboage changed the title Fixed typos/small errors in the Notebooks text explanations Fix (Notebooks): Corrected typos/small errors in the text cells with explanations. Oct 22, 2025
@JP-Amboage JP-Amboage changed the title Fix (Notebooks): Corrected typos/small errors in the text cells with explanations. Fix (Notebooks): Corrected typos/small errors in the text cells with explanations Oct 22, 2025
Copy link
Collaborator

@nickfraser nickfraser left a comment

Choose a reason for hiding this comment

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

LGTM!

@nickfraser nickfraser requested review from nickfraser and removed request for nickfraser October 22, 2025 12:34
@nickfraser nickfraser merged commit e04b825 into Xilinx:dev Oct 23, 2025
966 of 987 checks passed
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