Skip to content

Fix MebocostFlowPlot CI failure and clean up both new Omics tutorials - #256

Merged
ShixiangWang merged 3 commits into
mainfrom
copilot/fix-cran-mirror-setup
Mar 20, 2026
Merged

Fix MebocostFlowPlot CI failure and clean up both new Omics tutorials#256
ShixiangWang merged 3 commits into
mainfrom
copilot/fix-cran-mirror-setup

Conversation

Copilot AI commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

CI fails rendering MebocostFlowPlot.qmd because it calls install.packages("devtools") without a CRAN mirror set — triggered by an unnecessary MEBOCOST GitHub install block (the package is never used in evaluated code). Secondary issues: duplicated chunk labels, deprecated ggraph params, missing DESCRIPTION entries.

Package setup fixes

  • Remove unused MEBOCOST install block — only referenced in eval: false blocks; simulated data is used throughout
  • Replace for loop installs with individual calls — the update-dependencies.R scanner regex only matches quoted package names in install.packages("pkg"), not variables
  • Switch devtools::install_githubremotes::install_github for CellChat — remotes is already in DESCRIPTION, avoids pulling in heavy devtools
# Before (scanner can't detect these; devtools install fails without CRAN mirror)
for (pkg in c("ggplot2", "dplyr", "ggraph")) {
  if (!requireNamespace(pkg, quietly = TRUE)) install.packages(pkg)
}

# After (matches project convention, scanner-compatible)
if (!requireNamespace("ggplot2", quietly = TRUE)) install.packages("ggplot2")
if (!requireNamespace("dplyr", quietly = TRUE)) install.packages("dplyr")
if (!requireNamespace("ggraph", quietly = TRUE)) install.packages("ggraph")

Chunk label deduplication

  • Remove label from chunk header, keep only #| label: pipe comment — Quarto warns when both are present

ggraph deprecation fixes

  • curvaturestrength in geom_edge_arc() (deprecated since ggraph 2.0.0)
  • Remove invalid label.size param from geom_node_label()

DESCRIPTION

  • Add ggraph, tidygraph, patchwork, scales, RColorBrewer

Demo images

  • Regenerated both MebocostFlowPlot_demo.png and CellChatCirclePlot_demo.png from actual R code output

All changes applied to both EN and ZH versions of both tutorials. Verified both tutorials run to completion with zero errors/warnings on R 4.5.3.


📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

… regenerate demo images

- Remove unused MEBOCOST GitHub install block (not used in evaluated code)
- Replace for-loop install pattern with individual install.packages() calls
  matching project convention and enabling update-dependencies.R scanner
- Use remotes::install_github instead of devtools::install_github for CellChat
- Fix duplicated chunk labels by removing label from chunk header
  (keep only #| label: pipe comment)
- Fix deprecated curvature -> strength in geom_edge_arc()
- Remove invalid label.size param from geom_node_label()
- Add missing packages to DESCRIPTION: ggraph, tidygraph, patchwork, scales,
  RColorBrewer
- Regenerate demo images from actual R code output

Co-authored-by: ShixiangWang <25057508+ShixiangWang@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CRAN mirror setup issue in MebocostFlowPlot Fix MebocostFlowPlot CI failure and clean up both new Omics tutorials Mar 20, 2026
Copilot AI requested a review from ShixiangWang March 20, 2026 06:56
@github-actions

Copy link
Copy Markdown
Contributor

✅ QMD Format Check

✅ All 2 checked QMD file(s) pass format validation!

Automated check by the PR Review workflow.
See contribution guidance for the expected tutorial format.

Delete MEBOCOST Flow Plot tutorial and related assets: remove Omics/MebocostFlowPlot.qmd and its Chinese translation, delete the demo image and skill markdown, and strip the MEBOCOST entries from Skills.qmd, Skills.zh.qmd, _quarto.yml, index.qmd, and the skills JSON indexes (skills/bizard_skills.json, skills/index.json). Update site navigation and skill registry to reflect removal.
@github-actions

Copy link
Copy Markdown
Contributor

❌ QMD Format Check

❌ Format Errors (must fix)

Skills.qmd

  • Missing required YAML field: author

index.qmd

  • Missing required YAML field: author

⚠️ Format Warnings (recommended)

Skills.qmd

  • Missing recommended section: ## Example
  • Missing recommended section: ## Setup
  • Missing recommended section: ## Data Preparation
  • Missing recommended section: ## Visualization
  • No demo image found (recommended: ![](../images/...))

index.qmd

  • Missing recommended section: ## Example
  • Missing recommended section: ## Setup
  • Missing recommended section: ## Data Preparation
  • Missing recommended section: ## Visualization
  • No demo image found (recommended: ![](../images/...))

Automated check by the PR Review workflow.
See contribution guidance for the expected tutorial format.

@ShixiangWang
ShixiangWang marked this pull request as ready for review March 20, 2026 07:28
@ShixiangWang
ShixiangWang merged commit 0012a0a into main Mar 20, 2026
2 of 4 checks passed
@ShixiangWang
ShixiangWang deleted the copilot/fix-cran-mirror-setup branch March 20, 2026 07:28
@github-actions

Copy link
Copy Markdown
Contributor

📊 Quarto Render Preview

The new or modified Quarto files have been rendered and are available as artifacts.

🔗 View and Download Rendered Files

Click the link above, scroll to the "Artifacts" section at the bottom of the page, and download quarto-rendered-files to preview the rendered HTML files.

Modified files:

Omics/CellChatCirclePlot.qmd Omics/CellChatCirclePlot.zh.qmd Omics/MebocostFlowPlot.qmd Omics/MebocostFlowPlot.zh.qmd

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