Skip to content

Fix ManhattanPlot: dplyr summarise() multi-row error and duplicate chunk labels - #258

Merged
ShixiangWang merged 2 commits into
mainfrom
copilot/fix-deploy-action-error
Mar 20, 2026
Merged

Fix ManhattanPlot: dplyr summarise() multi-row error and duplicate chunk labels#258
ShixiangWang merged 2 commits into
mainfrom
copilot/fix-deploy-action-error

Conversation

Copilot AI commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

CI build fails rendering Omics/ManhattanPlot.qmd because newer dplyr (≥1.1.0) rejects summarise() returning multiple rows on an ungrouped data frame, and all figure chunks have labels defined in both the chunk header and #| label: pipe comments.

Changes

  • summarise()mutate() — after group_by(CHR) %>% summarise(len=max(BP)), the result is an ungrouped 22-row tibble. The subsequent call was computing a per-row offset, not aggregating — mutate() is the correct verb:
# Before (errors in dplyr ≥1.1.0)
len <- len %>% summarise(pos = cumsum(len) - len)

# After
len <- len %>% mutate(pos = cumsum(len) - len)
  • Remove duplicate chunk header names — all 8 figure chunks had both a header name (e.g., ```{r fig1.1Basicmanhattan}) and a #| label: pipe comment, triggering Duplicated chunk option(s) 'label' warnings. Chunk header names removed; #| label: pipe comments retained per project convention.

Both ManhattanPlot.qmd (EN) and ManhattanPlot.zh.qmd (ZH) are updated.


💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

…uplicate chunk labels

Co-authored-by: ShixiangWang <25057508+ShixiangWang@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix deploy action error in ManhattanPlot Fix ManhattanPlot: dplyr summarise() multi-row error and duplicate chunk labels Mar 20, 2026
Copilot AI requested a review from ShixiangWang March 20, 2026 09:32
@github-actions

Copy link
Copy Markdown
Contributor

✅ QMD Format Check

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

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

@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/ManhattanPlot.qmd Omics/ManhattanPlot.zh.qmd

@ShixiangWang
ShixiangWang marked this pull request as ready for review March 20, 2026 10:46
@ShixiangWang
ShixiangWang merged commit 710b147 into main Mar 20, 2026
3 of 4 checks passed
@ShixiangWang
ShixiangWang deleted the copilot/fix-deploy-action-error branch March 20, 2026 10:46
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