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

data/lab/arena: Add challenges for data lab (arena) #26

Merged
merged 4 commits into from
Nov 6, 2023

Conversation

razvand
Copy link

@razvand razvand commented Nov 6, 2023

Add challenges for data lab, including checkers.

@razvand razvand added kind/new New content / item area/code Update or new source code support / solution files topic/data Related to "Data" chapter area/content Content (Markdown) update labels Nov 6, 2023
@razvand razvand force-pushed the razvand/data/arena branch from 5cedee1 to feb2b14 Compare November 6, 2023 09:14
A challenge to exercise working with mmap().

Signed-off-by: Razvan Deaconescu <[email protected]>
@razvand razvand force-pushed the razvand/data/arena branch 2 times, most recently from 56fe558 to 2db4c3c Compare November 6, 2023 09:52
@razvand
Copy link
Author

razvand commented Nov 6, 2023

@StefanJum , @teodutu , @Alex-deVis , the rest of the checker errors are either stupid or difficult to handle in an elegant manner. I would leave them as such, and consider, at some point, working on them.

Let's get this PR in and then we can discuss about them.

@StefanJum StefanJum added the needs-rendering The PR makes changes to the website that need to be rendered label Nov 6, 2023
Copy link

github-actions bot commented Nov 6, 2023

Copy link

@StefanJum StefanJum left a comment

Choose a reason for hiding this comment

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

Looks fine, see the few comments.

content/chapters/data/lab/content/arena.md Outdated Show resolved Hide resolved
content/chapters/data/lab/content/arena.md Outdated Show resolved Hide resolved
content/chapters/data/lab/content/arena.md Show resolved Hide resolved
Copy link

@teodutu teodutu left a comment

Choose a reason for hiding this comment

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

On a very superficial look, the challenges, look interesting.

@razvand
Copy link
Author

razvand commented Nov 6, 2023

Will add the suggestions from @StefanJum then I'll approve the PR and merge it.

A challenge to exercise working with mmap() and memory copying.

Signed-off-by: Razvan Deaconescu <[email protected]>
A challenge to exercise mprotect() and an intro in SIGSEGV handling.

Signed-off-by: Razvan Deaconescu <[email protected]>
Fix: "assignement" -> "assignment"

Signed-off-by: Razvan Deaconescu <[email protected]>
@razvand razvand force-pushed the razvand/data/arena branch from 2db4c3c to 994f975 Compare November 6, 2023 10:36
@razvand razvand merged commit d594aa3 into cs-pub-ro:main Nov 6, 2023
0 of 3 checks passed
@razvand razvand deleted the razvand/data/arena branch November 6, 2023 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/code Update or new source code support / solution files area/content Content (Markdown) update kind/new New content / item needs-rendering The PR makes changes to the website that need to be rendered topic/data Related to "Data" chapter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants