Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vis.fgam broken #116

Open
mwmclean opened this issue Jul 4, 2024 · 0 comments
Open

vis.fgam broken #116

mwmclean opened this issue Jul 4, 2024 · 0 comments

Comments

@mwmclean
Copy link
Contributor

mwmclean commented Jul 4, 2024

@julia-wrobel any recollection of 35206e8? Seems to have removed all the tests for vis.fgam. vis.fgam currenty seems to fail for any model.

Something to do with those deprecation warnings? Guessing that deprecation work won't happen? @huangracer doesn't seem too active on GitHub anymore and those were added by him in 2015.

predict.fgam also looks for a component of the fitted fgam object named fgam, which seems to no longer exist.

Potentially vis.fgam and predict.fgam could be removed if they're too difficult to patch. Dangerous to keep functions around with no tests better to just remove them or have tests that don't run on CRAN.

Got a question about plotting af terms with Qtransform=TRUE. Came across #47. Seems to work with plot.prf but not vis.pfr or vis.fgam.

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

No branches or pull requests

1 participant