ahp icon indicating copy to clipboard operation
ahp copied to clipboard

Added consistency checks to geomean and normalized mean calculation methods

Open mestinso opened this issue 8 years ago • 2 comments

mestinso avatar Mar 04 '17 18:03 mestinso

Thx, makes sense, but there are two test cases that now fail:

2. Failure: Analyze Identity (@test-Analyze.R#12) ------------------------------
  names(df) not equal to c(...).
  Lengths differ: 9 vs 8
  
  
  testthat results ================================================================
  OK: 466 SKIPPED: 0 FAILED: 2
  1. Failure: Analyze Identity (@test-Analyze.R#11) 
  2. Failure: Analyze Identity (@test-Analyze.R#12) 
  
  Error: testthat unit tests failed
  Execution halted
* DONE

gluc avatar Mar 05 '17 08:03 gluc

There is still a warning in the build:

checking for code/documentation mismatches ... WARNING
Codoc mismatches from documentation object 'PrioritiesFromPairwiseMatrixEigenvalues':
PrioritiesFromPairwiseMatrixMeanNormalization
  Code: function(mat, allowedConsistency = 1)
  Docs: function(mat)
  Argument names in code not in docs:
    allowedConsistency
PrioritiesFromPairwiseMatrixGeometricMean
  Code: function(mat, allowedConsistency = 1)
  Docs: function(mat)
  Argument names in code not in docs:
    allowedConsistency

gluc avatar Mar 13 '17 19:03 gluc