Create saem generic for fitting saemix models

The reasons for this decision were
- Creating an saemix generic in the saemix package caused problems with
  roxygen, because functions like saemix.plot.xy were documented in
  their help files as S3 methods, although explicitly exported with
  @export
- Creating an saemix generic in this package is possible, but would
  make it necessary to load samix with exclude = "saemix" in order to
  avoid overwriting the generic when loading saemix.
- The return object of such an saemix generic in this package cannot
  be an S3 class with class attribute c("saemix.mmkin", "SaemixObject")
  similar to nlme.mmkin, as saemix returns an S4 class.
- Extending the S4 class SaemixObject using simple inheritance to
  a class SaemixMmkinObject with additional slots did not work
  as expected.  When the initialize method was left untouched, it
  prevented creation of an SaemixMmkinObject even if it was based
  on an initialised SaemixObject, as the initialize method seems
  to always be called by new(). This could potentially be circumvented
  by a coerce method. If an alternative initialize method was
  used, an SaemixMmkinObject could be created. However, the methods
  written for SaemixObjects only worked in some instances, either
  because they checked for the class, and not for class inheritance
  (like compare.saemix), or because the initialize method was called
  for some reason. Therefore, the idea of creating a derived S4 class
  was abandoned.
- A side effect of this decision is that the introduction of the saem
  generic opens the possibility to use the same generic also for other
  backends like nlmixr with the SAEM algorithm.

1 files changed, 7 insertions, 14 deletions

diff --git a/test.log b/test.log
index 318cc304..22501b25 100644
--- a/test.log
+++ b/test.log
@@ -5,11 +5,11 @@ Testing mkin
 ✔ |   2       | Export dataset for reading into CAKE
 ✔ |  14       | Results for FOCUS D established in expertise for UBA (Ranke 2014) [0.9 s]
 ✔ |   4       | Calculation of FOCUS chi2 error levels [0.4 s]
-✔ |   7       | Fitting the SFORB model [3.4 s]
-✔ |   5       | Analytical solutions for coupled models [3.1 s]
+✔ |   7       | Fitting the SFORB model [3.3 s]
+✔ |   5       | Analytical solutions for coupled models [3.0 s]
 ✔ |   5       | Calculation of Akaike weights
 ✔ |  10       | Confidence intervals and p-values [1.0 s]
-✔ |  14       | Error model fitting [4.3 s]
+✔ |  14       | Error model fitting [4.1 s]
 ✔ |   4       | Test fitting the decline of metabolites from their maximum [0.3 s]
 ✔ |   1       | Fitting the logistic model [0.2 s]
 ✔ |   1       | Test dataset class mkinds used in gmkin
@@ -33,23 +33,16 @@ Reason: getRversion() < "4.1.0"  is TRUE
 ────────────────────────────────────────────────────────────────────────────────
 ✔ |   4       | Residuals extracted from mkinfit models
 ✔ |   2       | Complex test case from Schaefer et al. (2007) Piacenza paper [1.5 s]
-✖ |   3 1     | Summary [0.1 s]
-────────────────────────────────────────────────────────────────────────────────
-test_summary.R:17: failure: Summaries are reproducible
-Results have changed from known value recorded in 'summary_DFOP_FOCUS_C.txt'.
-1/82 mismatches
-x[66]: "parent     2.661       4  5"
-y[66]: "parent     2.495       3  6"
-────────────────────────────────────────────────────────────────────────────────
+✔ |   4       | Summary [0.1 s]
 ✔ |   1       | Summaries of old mkinfit objects
 ✔ |   4       | Results for synthetic data established in expertise for UBA (Ranke 2014) [2.2 s]
 ✔ |   9       | Hypothesis tests [7.0 s]
 ✔ |   4       | Calculation of maximum time weighted average concentrations (TWAs) [2.5 s]
 
 ══ Results ═════════════════════════════════════════════════════════════════════
-Duration: 38.1 s
+Duration: 37.7 s
 
-OK:       145
-Failed:   1
+OK:       146
+Failed:   0
 Warnings: 0
 Skipped:  3