BiologicalRecordsCentre / BRCindicators

An R package for creating indicators from trends data
4 stars 11 forks source link

Update to accommodate bma method for trend_assessment #78

Closed 03rcooke closed 3 years ago

03rcooke commented 3 years ago

I've updated species_assessment and trend_assessment so they can be applied to either the lambda method or bma method (via the method argument).

I had to add --no-manual to the Check Package options, else it produces a build error that is unfixable on my OS as far as I can tell: "Error in texi2dvi(file = file, pdf = TRUE, clean = clean, quiet = quiet, : pdflatex is not available"

AugustT commented 3 years ago

Was the no manual thing a change you made only on your pc? Or did you change the way Travis runs?

03rcooke commented 3 years ago

After installing MikTex and fiddling around with Sys.setenv paths to find the executable I managed to pass the build checks locally without the --no-manual argument, so have removed this argument from the .Rproj and restored the build configuration settings to their previous setup.

AugustT commented 3 years ago

@03rcooke can you bump version number?

AugustT commented 3 years ago

happy for you to merge once checks pass

03rcooke commented 3 years ago

@AugustT I'm not sure I have the right to merge the changes, I just don't see a Merge pull request box.