Skip to content

chore: 🍱 add csv files for activities and data points - #269

Open
K-Beicher wants to merge 9 commits into
mainfrom
chore/add-timeline-csvs
Open

K-Beicher wants to merge 9 commits into
mainfrom
chore/add-timeline-csvs

Conversation

@K-Beicher

Copy link
Copy Markdown
Collaborator

Description

Added three csv files which show the activities for feasibility and main, as well as an overview of the points where data is created for main.

Closes #211

Needs a quick review.

Checklist

  • Ran just run-all

@K-Beicher K-Beicher self-assigned this Sep 22, 2026
@K-Beicher K-Beicher linked an issue Sep 22, 2026 that may be closed by this pull request
@K-Beicher
K-Beicher requested a review from a team as a code owner September 22, 2026 07:31
@K-Beicher K-Beicher moved this from Todo to In Review in Data development Sep 22, 2026
@lwjohnst86 lwjohnst86 changed the title chore: add csv files for activities and data points chore: 🍱 add csv files for activities and data points Sep 22, 2026
Comment thread supplemental/activities_feasibility.csv Outdated
Comment thread supplemental/activities_main.csv Outdated
Comment thread supplemental/data_points.csv Outdated
@github-project-automation github-project-automation Bot moved this from In Review to In Progress in Data development Sep 22, 2026
@lwjohnst86 lwjohnst86 changed the title chore: 🍱 add csv files for activities and data points chore: 🍱 add csv files for activities and data points Sep 22, 2026
@K-Beicher K-Beicher moved this from In Progress to In Review in Data development Sep 23, 2026

@lwjohnst86 lwjohnst86 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just a few more changes 😁

Comment thread supplemental/activities_main.csv Outdated
VO2 peak;Baseline;visit2
Randomisation;Baseline;visit2
MMTT;Baseline;visit2_MMTT
CGM on (intervention arm);Week_1;Dietary_education

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe add another column here for "dietary_education", so that visit is kept as a number?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is it necessary that there are _ instead of spaces in dietary_education? Do you plan on it being used in code?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Old habit, makes it more safe if I want to edit it in excel, I'll take it out.

Are you thinking a column called dietary_education, and then an X (or yes) as value, or...?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yea, something like that, "yes" and "no" (or empty). Or a comment about it if it needs it.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I've moved it around so that it also covers the exercise bit.

Comment thread supplemental/activities_feasibility.csv Outdated

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

Include the timeline of data collection

2 participants