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

Update timings of earlier lessons #299

Merged
merged 3 commits into from
Dec 2, 2020

Conversation

marwahaha
Copy link
Contributor

@rbavery and I taught a workshop this week. This is approximately the time Ryan spent while teaching the earlier lessons.

@jsta
Copy link
Member

jsta commented Sep 20, 2019

Thanks for this! Have you seen this thread with feedback on workshop timings? datacarpentry/geospatial-workshop#31

@marwahaha
Copy link
Contributor Author

For sure. I didn't read the whole thread. I don't understand, would you like me to leave comments there and you will review it later?

@jsta
Copy link
Member

jsta commented Sep 23, 2019

I linked that thread more so to create a trail of breadcrumbs for future time adjustment proposals. Anyway, having linked threads will get us part way there so I think there's no need to comment there also.

I think the adjustments you propose to lessons 1 and 2 are OK but I'm worried about lesson 3. You propose halving its timing but maybe other workshops have needed time closer to 40 min? Hard to tell because most of the past workshops have reported total time instead of a breakdown of teaching versus exercises.

This pull request has gone stale. Referencing pull request changes, @JTSA comments and notes by @DrK-Lo found at datacarpentry/geospatial-workshop#31. Teaching time in episode 3 Reproject Raster Data will not be changed at this time.
@drakeasberry drakeasberry merged commit 9abc498 into datacarpentry:master Dec 2, 2020
zkamvar pushed a commit that referenced this pull request Feb 7, 2023
Update timings of earlier lessons
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