-
Notifications
You must be signed in to change notification settings - Fork 4
Create training collateral #200
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
Create training collateral #200
Conversation
sc-4580/bny-create-training-collateral
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see newer comment.
### SESSION 1 | ||
|
||
- Initialize the developer framework | ||
- Start the model development process |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Start the model development process | |
- Initialize the model development process |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MichaelIngvarRoenning we can update this, but the current text essentially repeats what's in the introductory notebook for devs. If we change here, we should change there as well.
::: {.column width="50%"} | ||
### SESSION 1 | ||
|
||
- Initialize the developer framework |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of Developer Framework can we use ValidMind library (don't change if we don't agree)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree but we should make this change across all the content for consistency. This text currently lives in all our customer-facing notebooks.
|
1 similar comment
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few points:
-
It took me quite some time to understand that each screen was interactive that I could click and scroll in the UI and on overall slides that had more information below the starting screen of the slide. Make it very clear that the user needs to interactively move around the slide.
-
For both the developer and validator track we are using a ValidMind instance with a lot of models registered in the inventory. There are a lot of weird names, duplicates etc. and I suspect it's hard for a user to locate what the description is telling them to locate. We need a clean up of the model inventory or we need a "clean" instance.
-
I think maybe we need to include an admin slide in Session 1 because workflows need to be set up and configured for a model before we can continue the documentation process. I feel like that is a more natural flow of things. Feel free to disagree of course.
-
One specific comment for Slide 12/28 on validator track: there needs to be more guidance around the term review. When I go and click on the validation report for the model stated in description I see the standard report but how should they know what to actually do in the task, is it review model documentation (which is huge) or is it review the validation report template and just click through?
…hub.com:validmind/documentation into nrichers/sc-4580/bny-create-training-collateral
Good point, the slides were not explicit enough about interactivity. We now include a callout on the "In this module" page that tells people training is interactive and provides navigation hints.
That's the plan! We now have a training org that includes a smaller set of models, see the screenshot. Eventually, we will also have:
Together, these things will provide a more realistic training environment that mimicks how you would use ValidMind in production.
For tomorrow, I will leave things as is but perhaps it would make sense to add a slide with more detail around the prerequisites, e.g. explore the current workflow and then tell developers to look in the administrator training for the details on how to set it up. (For context, AFAIK devs can't configure the workflow themselves and workflow setup is a pretty big chunk of the training for administrators.)
That's a good point — we're short on guidance not just in the training but in the docs generally. I'll follow up with @juanmleng to see if I can pick his brain on what's entailed in the validator review. |
Global comments
Training for model validators
![]() Training for model developers
![]() Training for administrators
|
AFAIK, no, but I've been toying with the idea of adding
Good catch! Looks like some late-night copy pasta. Should be fixed in 519e0e6
The current setup is a workaround and I agree it has some issues with how the content is rendered. I hope to overlay the training right over JupyterHub when it's all said and done. |
This plugin looks promising: https://github.com/McShelby/reveal-helpbutton
🎉 Can confirm! |
I'm going to park this in a new story, but a couple of comments:
That said, I'm all for testing and tweaking, and improving the training MVP iteratively. If that's something you want to tinker with, just claim the story! |
Internal Notes for Reviewers
This PR demonstrates a standard for our first training modules, with an eye towards solving an important piece of the training puzzle: how to enable hands-on training for our users.
Features:
I want you to test this training module and tell me what you think. It's not perfect — the intent here is to improve on the static Google Slides we've been using and to pave the way for future refinements.
How can I try this?
Since we trialled an earlier version of this with some users, the training modules are already live:
If you test locally:
gh pr checkout 200
Bugs, issues, known problems
These need to be fixed before merging:
OAuth cookie error→ TRACKED VIA https://app.shortcut.com/validmind/story/4946/fix-oauth-error-in-jupyterhub-for-training-modulesSplit users & permissions pages (recent UI change):→ TRACKED VIA https://app.shortcut.com/validmind/story/4437/documentation-design-settings-permissionsCheck if add roles has changed?Check if manage permissions has changedContent blocks (recent UI change):→ TRACKED VIA https://app.shortcut.com/validmind/story/4971Update content block docs to match latest UI changesAdd edit content blocks section and include as tab in developer training moduleAdd a slide with more detail around the workflow prerequisites, e.g. explore the current workflow and then tell developers to look in the administrator training for the details on how to set it up.→ TRACKED VIA https://app.shortcut.com/validmind/story/4972"One specific comment for Slide 12/28 on validator track: there needs to be more guidance around the term review."→ TRACKED VIA https://app.shortcut.com/validmind/story/4973External Release Notes
We're introducing the first training modules that will be part of our training program for model developers, validators, and administrators. Our training modules are interactive, combining instructional content with our live product, and are easy to use. Try it: Welcome to ValidMind Academy