easystats / modelbased

:chart_with_upwards_trend: Estimate effects, contrasts and means based on statistical models
https://easystats.github.io/modelbased/
GNU General Public License v3.0
234 stars 19 forks source link

Add model_marginaleffects to pkgdown reference list #174

Closed moritzkoerber closed 2 years ago

moritzkoerber commented 2 years ago

Currently, the pkgdown build workflow fails because model_marginaleffects is not listed in the reference index: https://github.com/easystats/modelbased/runs/5433065598?check_suite_focus=true#step:10:66

See also the pkgdown documentation on this.

This PR adds model_marginaleffects to the refences under "Miscellaenous".

codecov-commenter commented 2 years ago

Codecov Report

Merging #174 (508b061) into master (954a1f7) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #174   +/-   ##
=======================================
  Coverage   36.91%   36.91%           
=======================================
  Files          21       21           
  Lines         967      967           
=======================================
  Hits          357      357           
  Misses        610      610           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 954a1f7...508b061. Read the comment docs.

IndrajeetPatil commented 2 years ago

Thanks!

Websites builds are still failing, but due to errors in vignettes.

@DominiqueMakowski The vignette build fails here:

Quitting from lines 160-173 (describe_nonlinear.Rmd) 
Error in .get_datagrid_clean_target(x, ...) : 
moritzkoerber commented 2 years ago

Thanks!

Websites builds are still failing, but due to errors in vignettes.

@DominiqueMakowski The vignette build fails here:

Quitting from lines 160-173 (describe_nonlinear.Rmd) 
Error in .get_datagrid_clean_target(x, ...) : 

Hm, could be that target is already given as an argument in an underlying function further down the line, e.g. in insight::get_datagrid.data.frame

IndrajeetPatil commented 2 years ago

Possible. Looks like this might be due to some internal restructuring of functions that has happened recently.

DominiqueMakowski commented 2 years ago

Thansk for the notif, will fix it