-
Notifications
You must be signed in to change notification settings - Fork 3
Add more contextual information to Jupyter notebooks #54
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
Add more contextual information to Jupyter notebooks #54
Conversation
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.
Looks great 👏. I left a small comment: the "how to run a test" notebook has been replaced by a new one.
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.
- Thank you for explaining the embedded vs sidebar ToC to me!
- I like the new more imperative titles, very clear and simple
- The new intro / CTAs are great too!
- For the individual notebooks I just left suggestions in them. :)
Weirdness
notebooks/code_samples/nlp_and_llm/foundation_models_integration_demo.ipynb
- For some reason it wouldn't let me leave comments or suggestions on lines 20-300ish.
- I wanted to add the exclamation mark at the end of "Sign up now"
notebooks/code_samples/nlp_and_llm/foundation_models_summarization_demo.ipynb
- Same here as above, huh.
notebooks/code_samples/nlp_and_llm/hugging_face_integration_demo.ipynb
- ... and here...
notebooks/code_samples/nlp_and_llm/hugging_face_summarization_demo.ipynb
- ... aaaaand here...
notebooks/code_samples/nlp_and_llm/llm_summarization_demo.ipynb
- ... nth verse, same as the 1st...
- Can't change the better way to an m-dash either, but that's just a style quibble of mine.
notebooks/code_samples/quickstart_customer_churn_full_suite.ipynb
- Just the better-way m-dash for this notebook
notebooks/code_samples/regression/quickstart_regression_full_suite.ipynb
- Both the signup! and the better way-- here
notebooks/code_samples/time_series/tutorial_time_series_forecasting.ipynb
- Wanted to adjust the "ValidMind is a platform for [...]" here, couldn't
- Wanted to adjust the "Get started with the ValidMind Developer Framework" blurb, couldn't
- Wanted to adjust the "Sign up now," couldn't
- Wanted to adjust the "better way," couldn't...
notebooks/code_sharing/llm/foundation_models_summarization_high_code.ipynb
- Exact same as above, ehehehe...
- Except I notice there isn't a CTA to review the results in the UI... is this on purpose?
notebooks/how_to/configure_dataset_features.ipynb
- Couldn't adjust the "Sign up now"
- Couldn't adjust the "better way"
notebooks/how_to/configure_test_parameters.ipynb
- Couldn't adjust the "better way"
notebooks/how_to/load_datasets_predictions.ipynb
- Wanted to adjust the "ValidMind is a platform for [...]" here, couldn't
- Wanted to adjust the "Get started with the ValidMind Developer Framework" blurb, couldn't
- Wanted to adjust the "Sign up now," couldn't
- Wanted to adjust the "better way," couldn't...
notebooks/how_to/run_a_test.ipynb
- Looks like this notebook will be replaced?
notebooks/how_to/run_unit_metrics.ipynb
- Couldn't adjust the "better way" here
I'm sure I missed something... either way, just minor formatting/semantic quibbles! Looking much better. :)
notebooks/code_samples/custom_tests/integrate-external_test_providers.ipynb
Outdated
Show resolved
Hide resolved
notebooks/code_samples/custom_tests/integrate-external_test_providers.ipynb
Outdated
Show resolved
Hide resolved
notebooks/code_samples/custom_tests/integrate-external_test_providers.ipynb
Outdated
Show resolved
Hide resolved
notebooks/code_samples/custom_tests/integrate-external_test_providers.ipynb
Outdated
Show resolved
Hide resolved
notebooks/code_samples/custom_tests/integrate-external_test_providers.ipynb
Show resolved
Hide resolved
notebooks/how_to/run_tests_that_require_multiple_datasets.ipynb
Outdated
Show resolved
Hide resolved
notebooks/how_to/run_tests_that_require_multiple_datasets.ipynb
Outdated
Show resolved
Hide resolved
…oviders.ipynb Co-authored-by: Beck <164545837+validbeck@users.noreply.github.com>
@validbeck anything in |
Oh right, yes, I totally forgot that (didn't realise what folder that was in). All good! |
…s-into' of github.com:validmind/developer-framework into nrichers/sc-3795/add-explanations-definition-of-concepts-into
…nrichers/sc-3795/add-explanations-definition-of-concepts-into
Pull requests must include at least one of the required labels: |
Pull requests must include a description in the release notes section. |
@cachafla I might need your help with the
|
Internal Notes for Reviewers
This PR makes a number of improvements to our most visible notebooks to make them easier to consume and more standalone:
:
at the end of a title(Uses the Jupyter TOC VS Code extension, Apache 2.0 license)
Output examples
A version of these edits is already live, as they were required for the reworked developer framework getting started:
New notebook structure
An example that shows the improved intro, new table of concepts, "About ValidMind" section with key concepts, and next steps that point to more learning resources.
How notebooks appear in our developer docs
An example of why consistent short descriptions matter:
Fine print, read carefully
<div class="alert alert-block alert-info">CTA</div>
but haven't applied the changes yetExternal Release Notes
Many of our Jupyter notebooks have received improvements to make them easier to consume and more standalone: