Skip to content

Clarify system dependencies step: explain OCR/CAS tooling#47

Merged
d-morrison merged 2 commits into
mainfrom
copilot/explain-system-dependencies
May 4, 2026
Merged

Clarify system dependencies step: explain OCR/CAS tooling#47
d-morrison merged 2 commits into
mainfrom
copilot/explain-system-dependencies

Conversation

Copilot AI commented May 4, 2026

Copy link
Copy Markdown
Contributor

The "Install system dependencies" step comment only mentioned R package libraries, but the step also installs OCR (tesseract-ocr) and CAS (maxima) tooling with no explanation for future maintainers.

Changes

  • .github/workflows/copilot-setup-steps.yml: Replaced the single-line step comment with a categorized bullet list describing each group of packages installed:
    • C/C++ libraries for R packages
    • PDF processing tools (poppler-utils)
    • OCR tools (tesseract-ocr)
    • Computer algebra system (maxima)
    • Python pip
# Install system dependencies needed for this project, including:
#
# - C/C++ libraries required by R packages (libcurl, libssl, libxml2, etc.)
# - PDF processing tools (poppler-utils) for working with PDF documents
# - OCR tools (tesseract-ocr) for optical character recognition tasks
# - A computer algebra system (maxima) for symbolic math computations
# - Python pip for additional tooling

Copilot AI linked an issue May 4, 2026 that may be closed by this pull request
Agent-Logs-Url: https://github.com/d-morrison/qwt/sessions/cc271470-4bcb-44f7-b99d-1f349eaceeb1

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Copilot AI changed the title [WIP] Update documentation to explain system dependencies Clarify system dependencies step: explain OCR/CAS tooling May 4, 2026
Copilot AI requested a review from d-morrison May 4, 2026 05:24
@d-morrison d-morrison marked this pull request as ready for review May 4, 2026 05:26
Copilot AI review requested due to automatic review settings May 4, 2026 05:26
@d-morrison d-morrison merged commit 483a607 into main May 4, 2026
10 checks passed
@d-morrison d-morrison deleted the copilot/explain-system-dependencies branch May 4, 2026 05:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Copilot setup workflow documentation to better explain why certain system packages are installed, improving maintainability for future contributors working on the agent environment.

Changes:

  • Expanded the “Install system dependencies” comment into a categorized list covering:
    • R build dependencies
    • PDF tooling (poppler)
    • OCR tooling (tesseract)
    • CAS tooling (maxima)
    • Python pip

@github-actions

github-actions Bot commented May 4, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-05-04 05:28 UTC

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.

explain system dependencies

3 participants