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

Accumulate could do with another hint #1857

Open
samsmithnz opened this issue Jan 8, 2022 · 3 comments
Open

Accumulate could do with another hint #1857

samsmithnz opened this issue Jan 8, 2022 · 3 comments

Comments

@samsmithnz
Copy link
Contributor

The problem is straight forward, but the lazy part through me. This, took it from easy to Medium/Hard for me. I think, given it's supposed to be easy, a better link/reference in the description to lazy loading/initialization would be fair.

@ErikSchierboom
Copy link
Member

Sure. Would you be interested in submitting a PR?

@samsmithnz
Copy link
Contributor Author

samsmithnz commented Jan 10, 2022

Definitely happy to help.

  • The complication is that lazy loading is poorly documented. I feel like the hints should have links to both lazy initialization and yield.
  • Additionally, how do you feel about upgrading the difficulty from Easy to Medium?

@ErikSchierboom
Copy link
Member

The complication is that lazy loading is poorly documented. I feel like the hints should have links to both lazy initialization and yield.

The latter link (yield) is great. I'm not sure about the former though, as I don't think students need to know about lazy initialization for this exercises.

Additionally, how do you feel about upgrading the difficulty from Easy to Medium?

Makes sense.

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

No branches or pull requests

2 participants