forked from open-education-hub/operating-systems
-
Notifications
You must be signed in to change notification settings - Fork 61
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
asg/parallel-graph: Update assignment #30
Merged
razvand
merged 6 commits into
cs-pub-ro:main
from
razvand:razvand/asg/update-parallel-graph
Nov 20, 2023
Merged
asg/parallel-graph: Update assignment #30
razvand
merged 6 commits into
cs-pub-ro:main
from
razvand:razvand/asg/update-parallel-graph
Nov 20, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
razvand
force-pushed
the
razvand/asg/update-parallel-graph
branch
from
November 20, 2023 06:45
edbb947
to
50bdada
Compare
Alex-deVis
added
needs-rendering
The PR makes changes to the website that need to be rendered
area/assignment
Update to homework assignments
topic/compute
Related to "Compute" chapter
labels
Nov 20, 2023
Published at https://cs-pub-ro.github.io/operating-systems/30/ |
Alex-deVis
reviewed
Nov 20, 2023
razvand
force-pushed
the
razvand/asg/update-parallel-graph
branch
from
November 20, 2023 08:38
50bdada
to
b3f9063
Compare
Improve wording. Add detailed information on checking and grading, similar to the other assignments. Signed-off-by: Razvan Deaconescu <[email protected]>
Simplify Makefile in `tests/` to the required bits for testing the assignment. Signed-off-by: Razvan Deaconescu <[email protected]>
Add `utils/` directory with utilities files as symlinks. They are used for logging and ending the program. Signed-off-by: Razvan Deaconescu <[email protected]>
The include path is used to point to the utils directory included in source code files. Signed-off-by: Razvan Deaconescu <[email protected]>
Synchronize skeleton implementation with the official soluion. The official solution is a redesign of the thread pool, together with the use of a generic list implementation (inspired from the Linux kernel). Signed-off-by: Razvan Deaconescu <[email protected]>
Update README.md to the new skeleton contents, based on the update implementation of the thread pool. Signed-off-by: Razvan Deaconescu <[email protected]>
razvand
force-pushed
the
razvand/asg/update-parallel-graph
branch
from
November 20, 2023 09:05
b3f9063
to
2b7fa34
Compare
Alex-deVis
approved these changes
Nov 20, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
gabrielmocanu
added
needs-rendering
The PR makes changes to the website that need to be rendered
and removed
needs-rendering
The PR makes changes to the website that need to be rendered
labels
Nov 20, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area/assignment
Update to homework assignments
needs-rendering
The PR makes changes to the website that need to be rendered
topic/compute
Related to "Compute" chapter
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Make overhaul update of the assignment: skeleton implementation, README, Makefiles.