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

fa24/mentor11 #217

Merged
merged 3 commits into from
Nov 3, 2024
Merged

fa24/mentor11 #217

merged 3 commits into from
Nov 3, 2024

Conversation

jimfangx
Copy link
Collaborator

@jimfangx jimfangx commented Oct 28, 2024

@jimfangx jimfangx requested a review from a team as a code owner October 28, 2024 05:49
@jimfangx jimfangx changed the title mentor11 fa24/mentor11 Oct 28, 2024
Copy link
Collaborator

@Mehul-Jaiswal Mehul-Jaiswal left a comment

Choose a reason for hiding this comment

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

Approximately how long did each problem take you? (The relative length is what we care about here, I don't care how fast you actually are.)

Q1: 2 mins
Q2: 2 mins
Q3: 4-5 mins

Minor errors: Are there any formatting issues or typos? Is there sufficient space for solutions and scratch work? Are answer blanks long enough? Does anything look weird?

Problem is repeated twice : Let’s practice using ifs and conds to evaluate Scheme problems! on 1.3.2 and Q1

Problematic problems: Were there any problem statements/explanations that could use clarification or simplification? Any wrong doctests, skeletons, or solutions? Was anything even a bit confusing?

Looks good, pretty much accurate.

Worksheet design: Did the problems work well together? Is the number of problems acceptable? Was there a variety of difficulty levels available? Were the problems listed in order of difficulty?

Short worksheet, the questions are on the easier end.

What were the best and worst problems on the worksheet? (Why?)

Best: Q1 and Q2 very good intro to scheme
Worst: Q3: not worst but a good start for scheme

Anything else?

lgtm

@jimfangx jimfangx merged commit b447cd9 into main Nov 3, 2024
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