Code

Split up epichains classes#107

Code

Merged

jamesmbaazam merged 40 commits intomainepiverse-trace/epichains:mainfrom split-epichains-classesepiverse-trace/epichains:split-epichains-classesCopy head branch name to clipboardDec 4, 2023

+1,025-622

Commits

Commits on Dec 4, 2023

• Use helper functions to create objects

  jamesmbaazam

  committed
  fae6c1f

• Remove epichains_aggregate_df class

  jamesmbaazam

  committed
  1a1b9e3

• Use new validation function

  jamesmbaazam

  committed
  1564d35

• Clean up documentation of aggregate method

  jamesmbaazam

  committed
  f4b4ffc

• Condense head and tail methods for new class

  jamesmbaazam

  committed
  69e9cb2

• Add constructor and helper for epichains_tree class

  jamesmbaazam

  committed
  c3a4a4a

• Add constructor for epichains_summary class

  jamesmbaazam

  committed
  fbc4134

• Add print and format methods for epichains_tree

  jamesmbaazam

  committed
  8d32e18

• Add print and format methods for epichains_summary class

  jamesmbaazam

  committed
  e00182e

• Add summary method for epichains_tree and epichains_summary

  jamesmbaazam

  committed
  b10974f

• Add validation checkers for the two classes

  jamesmbaazam

  committed
  50c6edc

• Generate new NAMESPACE

  jamesmbaazam

  committed
  8eb6fbc

• Add comments

  jamesmbaazam

  committed
  bfcb19a

• Bind right class to aggregate method

  jamesmbaazam

  committed
  76fd61f

• Replace old epichains class with the right classes

  jamesmbaazam

  committed
  52d8149

• Remove default types

  jamesmbaazam

  committed
  d8e8e94

• Loosen assertion for stat_max

  jamesmbaazam

  committed
  ca581e7

• Update snapshot tests

  jamesmbaazam

  committed
  c47c58e

• Bind epichains_tree class to aggregate method

  jamesmbaazam

  committed
  cdca312

• Styling to fix lintr issues

  jamesmbaazam

  committed
  6d172db

• Linting: Fix indentation

  jamesmbaazam

  committed
  67f9294

• Remove intvn_mean_reduction argument

  jamesmbaazam

  committed
  a62c319

• Improve documentation

  jamesmbaazam

  committed
  eb81100

• Remove intvn_mean_reduction

  jamesmbaazam

  committed
  07b830a

• Remove trailing comma

  jamesmbaazam

  committed
  7a13621

• Apply suggestions from code review

  Show description for 69a8611
  

  jamesmbaazam

  and

  sbfnk

  committed
  69a8611

• Rename chains_run to nchains

  jamesmbaazam

  committed
  bd85c5d

• Remove hardcoded superclass

  jamesmbaazam

  committed
  f775ed5

• Rename grouping_var to by

  jamesmbaazam

  committed
  b883168

• Rename nchains to ntrees

  jamesmbaazam

  committed
  21d8bed

• Revise function docs

  jamesmbaazam

  committed
  b94fadb

• Use new column names

  jamesmbaazam

  committed
  e4cf58b

• Reword function documentation

  jamesmbaazam

  committed
  27af1fd

• Rename variables for clarity

  jamesmbaazam

  committed
  fea39e4

• Remove rownames (got lost in merge conflicts)

  jamesmbaazam

  committed
  9deb88c

• Fix a doc

  jamesmbaazam

  committed
  59a74e8

• Remove unwanted variable

  jamesmbaazam

  committed
  6e04463

• Replace chain with trees in comments to remove confusion

  jamesmbaazam

  committed
  93ab5e4

• Update snaps

  jamesmbaazam

  committed
  febd504

• Return offspring as part of object

  jamesmbaazam

  committed
  f46b001