Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates to OpenMP/MPI Materials Based on August Pilot Feedback #188

Open
wants to merge 40 commits into
base: main
Choose a base branch
from

Conversation

mehtaparabaci
Copy link
Contributor

Based on feedback from the OpenMP/MPI pilot in August, we’ve updated the course materials to fix errors, clarify concepts, and add examples to make the lessons easier to follow.

OpenMP

  • Fixed typos across all episodes.
  • Key concepts like private variables, collapse clauses, and reduction were clarified with new explanations and examples.
  • Race conditions and barriers were updated with improved standalone code examples (e.g., barriers.c, matrix_update.c) and updates to the corresponding texts
  • Refined the reporting progress exercise with better instructions and a fixed solution.
  • New explanations and examples for synchronisation, atomic operations were added

MPI

  • Fixed typos and improved clarity across episodes.
  • Corrected errors in example code (e.g., fixed type mismatches in "Hello World, Again!").
  • Expanded explanations for key topics such as message passing, buffer usage, and derived data types.
  • Improved the Ping Pong challenge with output flushing and additional context.
  • Added notes on MPI_STATUS_IGNORE and clarified MPI_Broadcast matching send/receive requirements.
  • Signposted connections to the derived data type episode for better flow.
  • Made example solutions more robust and beginner-friendly.

mehtaparabaci and others added 22 commits August 8, 2024 16:33
The changes are covering OpenMP, MPI and Parallelisation sessions.
…ed loops callout with collapse clause, fixed scheduler terminology, and addressed firstprivate caveat. Also fixed typos.
…, expanded and clarified scheduler explanations, added compilable atomic example, new section on reduction clauses with examples, and clarified nested region restrictions. Fixed issues with synchronization routines, race condition explanation, and thread-specific matrix updates. Addressed indentation, typos, and other minor improvements.
…addressed buffer initialisation issues, and improved explanations for MPI concepts.
…essed rank-specific behaviours, added link for derived types episode, and explained matching in collective comms
Post-Pilot Updates for MPI Episodes
Copy link
Contributor

@abhidg abhidg left a comment

Choose a reason for hiding this comment

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

Hi @mehtaparabaci looks good! Few minor comments for improvement and some typos.

@mehtaparabaci
Copy link
Contributor Author

Thanks for the review & feedback @abhidg! I've made the updates-let me know if anything else needs tweaking.

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.

4 participants