Skip to content

Post june #109

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

Merged
merged 12 commits into from
Jun 2, 2025
Merged

Post june #109

merged 12 commits into from
Jun 2, 2025

Conversation

drmowinckels
Copy link
Owner

No description provided.

Copy link
Contributor

🎉 The preview is built! Check it out 👀

Copy link
Contributor

🎉 The preview is built! Check it out 👀

Copy link
Contributor

🎉 The preview is built! Check it out 👀

@drmowinckels drmowinckels requested a review from Copilot May 29, 2025 17:45
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates several files to refresh submodule pointers, add RDS file saving for reproducibility in blog posts, and modify build process messaging.

  • Updated submodule commit for theme "hugo-igloo".
  • Added RDS output code blocks in RMarkdown/Quarto blog posts to persist intermediate dataset states.
  • Updated GitHub Actions build workflow to print messages for production and preview builds.

Reviewed Changes

Copilot reviewed 35 out of 35 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
themes/hugo-igloo Updated submodule commit hash.
content/blog/2025/06-01_longcovid_clusters/test.R Added R code for data transformation and output, using dplyr/tidyr.
content/blog/2025/05-01_visible/index.qmd Added R code blocks to save RDS files and minor text adjustments.
content/blog/2025/05-01_visible/index.md Similar R code updates as in the QMD version with file output.
.github/workflows/build-site.yaml Inserted echo messages to indicate the build type in the CI workflow.

out <- .df %>%
dplyr::select(!!!.vars, value = !!value_var) %>%
dplyr::mutate(..r = dplyr::row_number()) %>%
tidyr::gather(x, node, -..r, -value) %>%
Copy link
Preview

Copilot AI May 29, 2025

Choose a reason for hiding this comment

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

Consider using tidyr::pivot_longer as gather() is deprecated in favor of pivot_longer, which may offer improved clarity and support in future versions.

Copilot uses AI. Check for mistakes.

@@ -651,7 +664,7 @@ remove_0_cols <- function(data) {

### Clustering

Just for better modelling, we scale the data, then fit a Ward.D2 model with eucledian distance.
Just for better modelling, we scale the data, then fit a Ward.D2 model with euclidian distance.
Copy link
Preview

Copilot AI May 29, 2025

Choose a reason for hiding this comment

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

The term 'euclidian' appears to be misspelled; consider correcting it to 'Euclidean' for clarity and consistency.

Suggested change
Just for better modelling, we scale the data, then fit a Ward.D2 model with euclidian distance.
Just for better modelling, we scale the data, then fit a Ward.D2 model with Euclidean distance.

Copilot uses AI. Check for mistakes.

Copy link
Contributor

🎉 The preview is built! Check it out 👀

@drmowinckels drmowinckels requested a review from Copilot May 29, 2025 17:53
Copy link
Contributor

🎉 The preview is built! Check it out 👀

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR, titled "Post june", updates the Hugo submodule reference and introduces new R code blocks in the blog posts to save intermediate R objects, along with minor workflow enhancements for build logging.

  • Updated subproject commit in themes/hugo-igloo
  • Added new R code blocks in both index.qmd and index.md to save RDS files and modified an in-code comment
  • Enhanced build-site workflow with echo messages for production and preview builds

Reviewed Changes

Copilot reviewed 34 out of 34 changed files in this pull request and generated 1 comment.

File Description
themes/hugo-igloo Updated the submodule commit reference
content/blog/2025/05-01_visible/index.qmd Added R code blocks to save R objects and updated a comment in the modelling section
content/blog/2025/05-01_visible/index.md Synchronized similar R code block additions and metadata adjustments
.github/workflows/build-site.yaml Introduced echo messages to distinguish between production and preview builds

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

🎉 The preview is built! Check it out 👀

Copy link
Contributor

🎉 The preview is built! Check it out 👀

Copy link
Contributor

🎉 The preview is built! Check it out 👀

Copy link
Contributor

🎉 The preview is built! Check it out 👀

@drmowinckels drmowinckels added the pending Post pending, scheduled label May 31, 2025
@github-actions github-actions bot merged commit 8d03548 into main Jun 2, 2025
3 checks passed
@github-actions github-actions bot deleted the post_june branch June 2, 2025 12:00
Copy link
Contributor

github-actions bot commented Jun 2, 2025

🎉 The preview is built! Check it out 👀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending Post pending, scheduled
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant