-
Notifications
You must be signed in to change notification settings - Fork 8
No genlasso #595
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
Merged
Merged
No genlasso #595
Changes from 18 commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
cc07ab0
rm genlasso
dajmcdon 975e783
fix trendfilter + growth_rate
dajmcdon a236706
add tests for parameter constructor and splines
dajmcdon 967f16a
pass minimal tests
dajmcdon 2f8b06f
docs: document (GHA)
dajmcdon b41b5fb
style: styler (GHA)
dajmcdon bbf5a07
slight modifications to the vignette
dajmcdon 92f3eb0
add install instructions to vignette
dajmcdon a6e8785
Merge branch 'no-genlasso' of https://github.com/cmu-delphi/epiproces…
dajmcdon f2c9383
redocument
dajmcdon 5a242a1
merge dev, bump version.
dajmcdon 0e10f94
redocument
dajmcdon 5101627
add tests, pass checks
dajmcdon 6d7091c
style: styler (GHA)
dajmcdon 61628fd
handle annoying lintr crud
dajmcdon 3b77ef6
add missing fn for pkgdown
dajmcdon bcdb216
Merge branch 'dev' into no-genlasso
dajmcdon 8758128
docs: document (GHA)
dajmcdon 5280faf
docs(growth_rate): reword + typo
brookslogan 0426ec4
Use glmgen/trendfilter installation command as suggestion
brookslogan 393370f
Make `growth_rate(method = "trendfilter")` without {trendfilter} a ha…
brookslogan 341be94
refactor(growth_rate): preoptimize/normalize some ops
brookslogan d10c440
docs: document (GHA)
brookslogan 8c92285
fix(growth_rate): single_lambda check
brookslogan b0d3236
Rename growth_rate_global_params() -> growth_rate_params()
brookslogan 2baa9e0
docs: document (GHA)
brookslogan 730bc55
Further NEWS rewording
brookslogan 264fbcf
remove scaling by sd (now done internally by the method)
dajmcdon 5a037bc
remove sorting of x0, not really needed
dajmcdon 2ce5daa
rm straggling sdy
dajmcdon 5612df0
satisfy the linter
dajmcdon 85e6530
fix rcmd check errors
dajmcdon d5c3498
Revert "satisfy the linter"
dajmcdon 059b968
Re-lint test line lengths
brookslogan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
Package: epiprocess | ||
Type: Package | ||
Package: epiprocess | ||
Title: Tools for basic signal processing in epidemiology | ||
Version: 0.10.3 | ||
Version: 0.10.4 | ||
Authors@R: c( | ||
person("Jacob", "Bien", role = "ctb"), | ||
person("Logan", "Brooks", , "[email protected]", role = c("aut", "cre")), | ||
|
@@ -23,17 +23,17 @@ Authors@R: c( | |
person("Posit", role = "cph", | ||
comment = "Copyright holder of included rlang fragments"), | ||
person("Johns Hopkins University Center for Systems Science and Engineering", role = "dtc", | ||
comment = "Owner of COVID-19 cases and deaths data from the COVID-19 Data Repository"), | ||
comment = "Owner of COVID-19 cases and deaths data from the COVID-19 Data Repository"), | ||
person("Johns Hopkins University", role = "cph", | ||
comment = "Copyright holder of COVID-19 cases and deaths data from the COVID-19 Data Repository"), | ||
comment = "Copyright holder of COVID-19 cases and deaths data from the COVID-19 Data Repository"), | ||
person("Carnegie Mellon University Delphi Group", role = "dtc", | ||
comment = "Owner of claims-based CLI data from the Delphi Epidata API") | ||
comment = "Owner of claims-based CLI data from the Delphi Epidata API") | ||
) | ||
Description: This package introduces common data structures for working with | ||
epidemiological data reported by location and time and offers associated | ||
utilities to perform basic signal processing tasks. The package is designed | ||
to be used in conjunction with `epipredict` for building and evaluating | ||
epidemiological models. | ||
Description: This package introduces common data structures for working | ||
with epidemiological data reported by location and time and offers | ||
associated utilities to perform basic signal processing tasks. The | ||
package is designed to be used in conjunction with `epipredict` for | ||
building and evaluating epidemiological models. | ||
License: MIT + file LICENSE | ||
URL: https://cmu-delphi.github.io/epiprocess/ | ||
Depends: | ||
|
@@ -44,7 +44,6 @@ Imports: | |
cli, | ||
data.table, | ||
dplyr (>= 1.1.0), | ||
genlasso, | ||
ggplot2, | ||
glue, | ||
lifecycle (>= 1.0.1), | ||
|
@@ -71,6 +70,7 @@ Suggests: | |
readr, | ||
rmarkdown, | ||
testthat (>= 3.1.5), | ||
trendfilter, | ||
withr | ||
VignetteBuilder: | ||
knitr | ||
|
@@ -79,7 +79,7 @@ Remotes: | |
cmu-delphi/epidatasets, | ||
cmu-delphi/epidatr, | ||
cmu-delphi/epipredict, | ||
glmgen/genlasso, | ||
glmgen/trendfilter, | ||
reconverse/outbreaks | ||
Config/Needs/website: cmu-delphi/delphidocs | ||
Config/testthat/edition: 3 | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.