Skip to content

Update cleaning - #226

Merged
avallecam merged 18 commits into
mainfrom
update-cleaning
Jun 15, 2026
Merged

Update cleaning#226
avallecam merged 18 commits into
mainfrom
update-cleaning

Conversation

@Degoot-AM

@Degoot-AM Degoot-AM commented Feb 23, 2026

Copy link
Copy Markdown
Contributor

This PR addresses early tutorials issues related to data cleaning episode. These includes issues #205, #206, #216

@github-actions

github-actions Bot commented Feb 23, 2026

Copy link
Copy Markdown

❌ DANGER ❌

This pull request has modified workflows that created output. Close this now.

github-actions Bot pushed a commit that referenced this pull request Feb 23, 2026
github-actions Bot pushed a commit that referenced this pull request Feb 23, 2026
github-actions Bot pushed a commit that referenced this pull request Feb 23, 2026
github-actions Bot pushed a commit that referenced this pull request Feb 23, 2026
github-actions Bot pushed a commit that referenced this pull request Feb 24, 2026
@Degoot-AM
Degoot-AM requested a review from avallecam February 24, 2026 11:07
github-actions Bot pushed a commit that referenced this pull request Jun 14, 2026
github-actions Bot pushed a commit that referenced this pull request Jun 14, 2026

@avallecam avallecam 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.

Thanks! Sharing some registries of format decisions below. I will push some edits on top from my review briefly. Most of them are building on top of this work.

Comment thread episodes/clean-data.Rmd Outdated
4\.
1\) to use the base R pipe operator (`|>`) instead of `%>%`.

We also encourage using the `{pak}` package when installing R packages as shown below.

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.

best placed in setup. added in separate PR as 2106892

Comment thread episodes/clean-data.Rmd Outdated
Comment on lines +50 to +52
We encourage users with recent versions of R (version > 4.
4\.
1\) to use the base R pipe operator (`|>`) instead of `%>%`.

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.

we agreed to keep the magrittr pipe #96

github-actions Bot pushed a commit that referenced this pull request Jun 15, 2026
github-actions Bot pushed a commit that referenced this pull request Jun 15, 2026
Comment thread episodes/validate.Rmd Outdated
This episode requires you to:

- Download the [cleaned_data.csv](https://epiverse-trace.github.io/tutorials-early/data/cleaned_data.csv) file
- Download the [cleaned\_data.csv](https://epiverse-trace.github.io/tutorials-early/data/cleaned_data.csv) file

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.

@Degoot-AM lets avoid using this simbol \. I found it in multiple lines. It does not align with writing style and breaks Ctrl+F for searching patterns

Comment thread episodes/validate.Rmd Outdated
Comment on lines +198 to +200
You notice at some point that the data stream you have been relying on has a set of new entries (i.
e.
, rows or observations), and the data type of one variable has changed.

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.

@Degoot-AM please, also avoid breaking the i.e.,, because it breaks readability. Already fixed,

github-actions Bot pushed a commit that referenced this pull request Jun 15, 2026
@avallecam
avallecam merged commit 3d4e24b into main Jun 15, 2026
9 checks passed
@avallecam
avallecam deleted the update-cleaning branch June 15, 2026 19:09
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.

2 participants