Skip to content

Update dissmapr - #168

Draft
sannegovaert wants to merge 1 commit into
mainfrom
update-dissmapr
Draft

Update dissmapr#168
sannegovaert wants to merge 1 commit into
mainfrom
update-dissmapr

Conversation

@sannegovaert

@sannegovaert sannegovaert commented Jul 22, 2026

Copy link
Copy Markdown
Member

To fix #166

I reinstalled dissmapr with:
remotes::install_github("b-cubed-eu/dissmapr")

And reran the documenation script.

Vignette 8 is fine. There are still errors remaining in vignette 6 and 7.
@peterdesmet Can you do a rerun to check whether this is dependent on the package (as I also had problems with invasimapr).

Vignette 6:

Image Image

Vignette 7:

Image

@peterdesmet

Copy link
Copy Markdown
Collaborator

If I run the scripts, I still get errors (searching for #> Error for each vignette):

3-enviro

24 errors, first one:

#> Error:
#> ! unable to find an inherited method for function 'nlyr' for signature 'x = "NULL"'

@sannegovaert seems like you didn't get this one, so guess I can ignore that. These errors also don't appear in the pkgdown: https://b-cubed-eu.github.io/dissmapr/articles/3-enviro.html

6-predict

5 errors, first one:

#> Error in `numeric()`:
#> ! invalid 'length' argument
# names(env_scenarios) = names(horizons)

These errors don't appear in the pkgdown: https://b-cubed-eu.github.io/dissmapr/articles/6-predict.html

Vignette 7

1 error, first one:

#> Error in `plot.xy()`:
#> ! invalid type passed to graphics function

These errors don't appear in the pkgdown: https://b-cubed-eu.github.io/dissmapr/articles/7-bioregion.html

@sannegovaert

Copy link
Copy Markdown
Member Author

@macSands see above

@macSands

Copy link
Copy Markdown
Collaborator

I've addressed the remaining dissmapr vignette issues. Will still need to rebuild the documentation using dissmapr 0.2.2.

Vignette 6 now renders without the numeric(): invalid 'length' argument error.
Vignette 7 now renders without the plot.xy() error.
I also checked the regenerated dissmapr documentation for #> Error output.
Vignette 8 continues to render correctly.

The documentation must still be regenerated using src/external/rmd_to_md.Rmd and the updated output pushed to this PR.

@macSands macSands closed this Aug 28, 2026
@sannegovaert sannegovaert reopened this Aug 31, 2026
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.

Add additional dissmapr tutorials

3 participants