Skip to content

Analyze the package with RcppDeepState#23

Open
FabrizioSandri wants to merge 1 commit intoBMasinde:masterfrom
RcppDeepState:RcppDeepState
Open

Analyze the package with RcppDeepState#23
FabrizioSandri wants to merge 1 commit intoBMasinde:masterfrom
RcppDeepState:RcppDeepState

Conversation

@FabrizioSandri
Copy link

@FabrizioSandri FabrizioSandri commented Sep 6, 2022

This PR adds a new Github Action which runs RcppDeepState+valgrind on your package. That means the C++ functions of your package will be tested with random inputs, and there will be a comment like this one for each new PR (which reports if valgrind found any issues with random inputs).

RcppDeepState analysis result

This package contains problems, according to RcppDeepState. The report was generated by RcppDeepState-action in this repository's fork and is accessible here.

@FabrizioSandri
Copy link
Author

RcppDeepState Report

function name message file line address trace R code
minpowspeed_cpp Invalid read of size 8 minimum_power_speed.cpp:23 No Address Trace found
Test code
testlist <- list(airDensity = -5.15765034623318e+31, bdc = 9.09398615486649e-164,     bm = c(-2.0374084571111e-05, 6.25353587561935e-22, -3.14525096286877e+40,     4.96613900006511e-244, -7.86865293011007e-191, 3.38179149991318e+135,     -3.72820138245044e-113, -2.80355229428131e+119, 7.12981325082077e+236,     2.68813462762586e-243, 0.198795572787604, 1.27979076028105e+225,     3.98573361673831e-222, -1.49671927442476e+44, -4.48370321053622e+281,     -3.49052751711369e+131, 1.9435262047378e+294, 2.11183475175717e-82,     7.57869019090068e+245, -1.13442395650577e+260, 9.64040438484829e-13,     3.25415848342009e-31, -5.45522394934584e+30, -8.95401730007894e+63,     6.30907243110838e+139, 1.12865093406818e-226, 2.71564190152268e+259,     1.17703483929389e+288, -6.44601918076588e+132, 4.14270275441291e-297    ), g = -1.63143041647574e+135, ipf = 6.66425371380266e+174,     ws = c(1.40106092575938e+25, 1.32752902469842e+184, -6.37016660097932e-307,     1.82963689427576e+261, 2.58787365428118e+300, 2.08382193580117e-26,     2.31547739525916e-65, -4.33861794620859e+248, -4.19043901825808e+252,     -1.91450027263831e+69, 2.86020603784802e-37, 8.81878164230477e-199,     3.09777666919361e+45, 3.00064491106139e+207, 1.49709758025628e+156,     -7.12157904331817e-13, 3.78982671254603e-134, 2.2444841246124e+259,     7.87610429448914e+157, -2.3523657499851e+144, -9.9817959619754e-71,     6.76002968507254e+40, -1.20911880763317e-227, 1.62616613540587e-134,     -6.91491138169895e-116, 2.92797537750378e-198, 2.64685409632618e-118,     -4.52321981844777e-185))
result <- do.call(FlyingR:::minpowspeed_cpp, testlist)

Analyzed functions summary

function name tested inputs inputs with issues
basal_metabolic_pow 3 0
minpowspeed_cpp 3 2
power_curve 3 0
total_Mech_Pow_cpp 3 0

Report details

  • Report generated by: 234ffe4
  • Inputs generator seed: 1662468467

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.

1 participant