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

Remove call for beta testers. #499

Merged
merged 3 commits into from
Jul 23, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,9 @@ This indicates that the maintainers will welcome a pull request fixing this issu
### Teaching this lesson?
We would be very grateful if you can provide us with feedback on this lesson. You can read more about hosting a lesson pilot for an incubator lesson [here](https://docs.carpentries.org/topic_folders/lesson_development/lesson_pilots.html).

You can notify us that you plan to teach this lesson by creating an issue in this repository (and labeling it with beta) or posting a message in the carpentries Slack Machine Learning channel. Please note the questions below to get an indication of the sort of feedback we expect.
You can notify us that you plan to teach this lesson by creating an issue in this repository or posting a message in the carpentries Slack Machine Learning channel. Please note the questions below to get an indication of the sort of feedback we expect.

After the workshop, lease create an issue (or comment on the issue you created before teaching) with general feedback on teaching the lesson, and label it with beta. As a template, you can use the following questions:
After the workshop, please create an issue (or comment on the issue you created before teaching) with general feedback on teaching the lesson. As a template, you can use the following questions:
* How much time did you need for the material? (preferably per episode)
* How much time did you need for the exercises?
* Where there any technical issues that arose during setup?
Expand Down
17 changes: 7 additions & 10 deletions index.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,16 +20,13 @@

::: instructor
Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like this opening statement should also be removed? (or needs a closing if you like my comment below)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I understand that this line now opens the "Breaks and Schedule" div.


## Looking for Beta Testers!
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would suggest to instead replace this with:

We can help you out with teaching this lesson!

Do you want to teach this lesson? Feel free to reach out to us with any questions that you have. Just open a new issue. We also value any feedback on the lesson!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That would seem a bit redundant with the "Teaching this lesson?" section in the README. Maybe a link there would be better to keep the information in sync?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@svenvanderburg How about this?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Good point! I like your solution :)

**We are currently looking for volunteers to test this lesson!**
If you would like to teach this lesson in a pilot workshop,
please let the lesson developers know by
[opening a new issue on the lesson repository](https://github.com/carpentries-incubator/deep-learning-intro/issues/new)
or posting to the [`#machine_learning` Slack channel](https://swcarpentry.slack.com/archives/CKLUYLY2F)
on [The Carpentries Slack](https://swc-slack-invite.herokuapp.com/).
We would love to help you prepare to teach the lesson and
receive feedback on how it could be further improved,
based on your experience in the workshop.
## We can help you out with teaching this lesson

Do you want to teach this lesson?
Find more help in the [README](https://github.com/carpentries-incubator/deep-learning-intro?tab=readme-ov-file#teaching-this-lesson)
Feel free to reach out to us with any questions that you have.
Just open a new issue.
We also value any feedback on the lesson!

:::

Expand All @@ -41,6 +38,6 @@
We suggest to have a break at least every 90 minutes and to switch the instructor regularly, also within episodes.
We have added reminders to the longer episodes with suggestions for when to have a switch and/or a break.

There is an [example schedule](schedule.html) with breaks that can be adapted to your workshop.

Check warning on line 41 in index.md

View workflow job for this annotation

GitHub Actions / Build markdown source files if valid

[missing file]: [example schedule](schedule.html)

:::
Loading