Skip to content

Workshop final version - #7

Merged
MendezSebastianP merged 7 commits into
mainfrom
final_version
Aug 31, 2026
Merged

MendezSebastianP merged 7 commits into
mainfrom
final_version

Conversation

@JeanJNMV

Copy link
Copy Markdown
Collaborator

No description provided.

@MendezSebastianP

Copy link
Copy Markdown
Collaborator

Hello @JeanJNMV @maylistran01 ! working in a notebook is a bit challenging because a 1 line modification could be taken as a 1000 lines modification in github, so right now the modifications in fit.ipynb are impossible to verify and review, could you tell me here what you modified so i check the file ? thanks !

…on dataset but with the original direction of the score and the original score, it is meant to be used in the tutorial so we put it as an example of how to load date
…user has to load and modify, the result is the same as parkinson
@maylistran01

Copy link
Copy Markdown
Collaborator

Hi @MendezSebastianP,

Yes of course.
I made 2 changes in the Personalize section:

  • The graph for Reparametrized Time subsection is now plotted using asset : tp.show_asset("reparam_morph.html", height=560) and I removed the long code to plot the previous graphs.
  • I updated the questions.yaml file, question 7, adding a letter at the beginning of each answer:
7:
  prompt: "Put the whole prediction and evaluation workflow back in the correct sequence:"
  options:
    - A - "Fit on the train set → estimate the held-out visit → personalize the test patients → compare with the ground truth (MAE, R²)"
    - B - "Personalize the test patients → fit on the train set → estimate the held-out visit → compare with the ground truth (MAE, R²)"
    - C - "Fit on the train set → personalize the test patients → estimate the held-out visit → compare with the ground truth (MAE, R²)"
    - D - "Fit on the train set → personalize the test patients → compare with the ground truth (MAE, R²) → estimate the held-out visit"
  answer: 2

Thank you !

…on about printing a plot that was too long and impossible to do for the users, moving the number of the rest of the questions
@MendezSebastianP
MendezSebastianP merged commit 28414a3 into main Aug 31, 2026
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