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

breaking question down to possibly facilitate solution? #10

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rfbowen
Copy link
Collaborator

@rfbowen rfbowen commented Jul 16, 2024

Is this useful? Something we keep in our back pocket if someone gets stuck?

@rileyshanahan20
Copy link
Contributor

imo, i still think its better to simplify to one question / one function. I think its more confusing with three but happy to tinker and experiment.

@rfbowen
Copy link
Collaborator Author

rfbowen commented Jul 16, 2024

imo, i still think its better to simplify to one question / one function. I think its more confusing with three but happy to tinker and experiment.

i think we need to make this repo private and send candidates a zip of it, so given that i'm open to you making a branch of the project and trying it with a few candidates

my reservation is mostly that we're seeing candidates struggle even when the problem is decomposed.

i believe putting it into a single question will create something with a high ceiling (excellent candidates can differentiate), but may represent a stress-wall for a lot of candidates across the rest of the spectrum.

e.g., they'll need to either recognize the nice solution that can handle the presence and absence of the filter routine in one tidy loop, or they'll wind up creating two logical branches. seems like inviting new opportunities for things to go poorly, which - if we want to dial difficulty up i think will actually be a good move, but i think we may be needing to go in the other direction?

@msohcw
Copy link

msohcw commented Jul 17, 2024

IMO I really like having the signature to pull out. I think I would've pulled this out at the 40min mark when I saw candidates spinning. But I also feel like it's useful signal to see a candidate spinning, so I wouldn't add this immediately.

@rileyshanahan20 rileyshanahan20 removed their request for review July 19, 2024 17:24
@msohcw msohcw removed their request for review August 13, 2024 15:00
@clairecfu clairecfu removed their request for review August 22, 2024 14:33
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