ms609 / TreeDist

Calculate distances between phylogenetic trees in R
https://ms609.github.io/TreeDist/
28 stars 6 forks source link

Fix Dependency Resolution Issues, Enhance Compatibility Checks, and Add Dockerfile #118

Closed SermetPekin closed 3 months ago

SermetPekin commented 3 months ago

Summary

This PR addresses an error related to package resolution which was caused by dependencies of dependencies. Specifically, some packages required by our package were dependent on other packages which caused conflicts or issues during installation.

Changes Made

  1. Created a Dockerfile:

    • Base Image: Switched to rocker/r-ver:4.3.0 for consistency and compatibility.
    • System Dependencies: Added necessary system dependencies to support package installations.
    • R Packages Installation:
      • Installed essential R packages (devtools, remotes, pak, covr, testthat).
      • Installed the latest version of Rcpp from GitHub to ensure compatibility with the latest features.
      • Explicitly installed Matrix to resolve dependencies required by fGarch.
      • Installed all other dependencies (fGarch, univariateML, kdensity, TreeDistData, TreeDist, phangorn).
    • Testing: Included a step to run tests using devtools::test() to ensure the package and its dependencies are functioning correctly.
  2. Updated GitHub Actions Workflow:

    • Matrix Configuration: Ensured that tests are run on multiple operating systems and R versions, including R 3.4.0.
    • Dependencies Installation: Added steps to install devtools and remotes packages before using them to install other dependencies.

How These Changes Address the Issue

ms609 commented 3 months ago

Closing as redundant to #119