Skip to content

Student changes#75

Merged
foxik merged 2 commits intoufal:masterfrom
gamecraftCZ:master
Feb 25, 2026
Merged

Student changes#75
foxik merged 2 commits intoufal:masterfrom
gamecraftCZ:master

Conversation

@gamecraftCZ
Copy link
Contributor

Changes by a student from Piazza: https://piazza.com/class/mlsfwoprfj02vr/post/40

  • There are not included changes to the Competitions web tab as I was not able to find where to edit it.

Copilot AI review requested due to automatic review settings February 24, 2026 19:11
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR contains grammar and wording improvements to documentation files submitted by a student from Piazza. The changes make the documentation more grammatically correct and consistent across multiple task description files.

Changes:

  • Fixed grammar in three task files to use "familiarize yourself with" instead of "familiarize with"
  • Added the article "the" before "ReCodEx" for proper grammar
  • Corrected typo "your cloned" to "you cloned" in FAQ
  • Improved wording from "in any case" to "irrespective of this flag" for clarity

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
tasks/pca_first.md Grammar correction: "familiarize with" → "familiarize yourself with"
tasks/numpy_entropy.md Grammar correction with article addition: "familiarize with Python, NumPy and ReCodEx" → "familiarize yourself with Python, NumPy and the ReCodEx"
tasks/mnist_layers_activations.md Grammar correction: "familiarize with" → "familiarize yourself with"
labs/faq.md Two improvements: wording change to "irrespective of this flag" and typo fix "your cloned" → "you cloned"

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@foxik foxik merged commit c0dfbd1 into ufal:master Feb 25, 2026
3 of 4 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.

3 participants