Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 7, 2026

  • Fix pkgdown workflow path triggers and deprecated actions
  • Investigate pkgdown build failures
  • Fix getting-started.qmd variable reference errors (case_data → simulated_data)
  • Remove redundant postprocess_jags_output() call in getting-started.qmd
  • Verify vignette renders successfully with quarto render
  • Clean up accidentally committed artifacts
  • Verify CI passes

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix pkgdown website rendering issues Fix pkgdown workflow: correct file paths and update deprecated actions Jan 7, 2026
Copilot AI requested a review from d-morrison January 7, 2026 08:15
@codecov
Copy link

codecov bot commented Jan 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@d-morrison d-morrison marked this pull request as ready for review January 7, 2026 09:10
Copy link
Collaborator

@d-morrison d-morrison left a comment

Choose a reason for hiding this comment

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

pkgdown workflow is still failing; fix issues in getting-started.qmd

Copy link
Collaborator

@d-morrison d-morrison left a comment

Choose a reason for hiding this comment

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

pkgdown is still broken; fix

Copilot AI and others added 2 commits January 8, 2026 02:26
…redundant postprocess_jags_output call

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 8, 2026

pkgdown is still broken; fix

Fixed in e79c669. The getting-started.qmd vignette had two issues:

  1. Variable reference errors: The plot_predicted_curve call referenced case_data but should have referenced simulated_data (the data used to fit the model)
  2. Incorrect postprocess call: The vignette called postprocess_jags_output() on output from run_mod(), but run_mod() already returns processed output. The call should go directly to post_summ()

The vignette now renders successfully with quarto render.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • cdn.jsdelivr.net
    • Triggering command: /opt/R/4.5.2/lib/R/bin/exec/R /opt/R/4.5.2/lib/R/bin/exec/R --slave --no-save --no-restore -f /tmp/RtmpXApEqA/callr-scr-2d5d2893d4e (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

📕 Preview documentation for this PR has been cleaned up.

@d-morrison d-morrison merged commit 432dbac into test-mcp Jan 8, 2026
12 checks passed
@d-morrison d-morrison deleted the copilot/sub-pr-163-again branch January 8, 2026 03:05
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