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

teeny fix to read_model #36

Closed
wants to merge 1 commit into from
Closed

Conversation

mcgoodman
Copy link
Contributor

This is a very silly pull request - model$paths instead of model$path in read_model causes problems when adding missing variance parameters. All tests pass on my local machine, of course.

@James-Thorson-NOAA
Copy link
Owner

OK, I'm changing this on the dev branch which I plan to merge soon. Closing here.

James-Thorson-NOAA pushed a commit that referenced this pull request Feb 13, 2025
James-Thorson-NOAA added a commit that referenced this pull request Feb 13, 2025
* avoid crash when quiet=FALSE and without data

* adding total_effect(.)

* Update test-platform.R

* add `total_effect(.)` to vignette

* add log-linked Poisson family

* add spatial_diffusion vignette

* make CI happy

* another try to make CI happy

* another try

* small change to new vignette

* allow negative lags for testing conditional independence

* adding test_dsep(.)

* update semantic version

* fix vignettes

* fix CI and add test_dsep example

* use simpler stepwise_selection example

* Update test_dsep.R

* Update test_dsep.R

* update test_dsep

* updates to test_dsep

* fix bug in test_dsep

* Update DESCRIPTION

* Update DESCRIPTION

* Update DESCRIPTION

* Fixes for issues #35 and #36

* add class when run_model = FALSE

* Update spatial_diffusion.Rmd

* eliminate \(.) notation

* Update explore_dsep.R

---------

Co-authored-by: Jim Thorson <James.T.Thorson@gmail.com>
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.

2 participants