-
Notifications
You must be signed in to change notification settings - Fork 48
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
raneftables returning NamedTuple of DictTables #634
Conversation
Suggestion from JuliaFormatter Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
It may be better to return a Update: not really a big win julia> collect(raneftables(slm01))
1-element Vector{TypedTables.DictTable{String, NamedTuple{(:subj, Symbol("(Intercept)"), :days), Tuple{String, Float64, Float64}}, NamedTuple{(:subj, Symbol("(Intercept)"), :days), Tuple{Dictionaries.Indices{String}, Dictionaries.Dictionary{String, Float64}, Dictionaries.Dictionary{String, Float64}}}, Dictionaries.Indices{String}}}:
{"S308" = (subj = "S308", var"(Intercept)" = 2.8158188821572576, days = 9.075511758123813), "S309" = (subj = "S309", var"(Intercept)" = -40.04844169634357, days = -8.644079444480187), "S310" = (subj = "S310", var"(Intercept)" = -38.43306372090656, days = -5.513397995849837), "S330" = (subj = "S330", var"(Intercept)" = 22.832112155574148, days = -4.658717421575887), "S331" = (subj = "S331", var"(Intercept)" = 21.549840457094, days = -2.944492928895566), "S332" = (subj = "S332", var"(Intercept)" = 8.815541388546052, days = -0.23520071393588618), "S333" = (subj = "S333", var"(Intercept)" = 16.441907858079695, days = -0.15880872279991523), "S334" = (subj = "S334", var"(Intercept)" = -6.996670702793803, days = 1.0327272102714635), "S335" = (subj = "S335", var"(Intercept)" = -1.0375872924983427, days = -10.599415818014403), "S337" = (subj = "S337", var"(Intercept)" = 34.66629369085548, days = 8.6323845720063), …} |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #634 +/- ##
=======================================
Coverage 95.90% 95.90%
=======================================
Files 29 29
Lines 2732 2733 +1
=======================================
+ Hits 2620 2621 +1
Misses 112 112
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
I think changing it a |
@dmbates if you want to take this out of draft, add NEWS (now at |
Co-authored-by: Phillip Alday <[email protected]>
NEWS.md
Outdated
MixedModels v4.10.0 Release Notes | ||
============================== | ||
* `raneftables` returns a `NamedTuple` where the names are the grouping factor names and the values are some `Tables.jl`-compatible type. Currently this type is a `DictTable` from `TypedTables.jl`. [#634] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MixedModels v4.10.0 Release Notes | |
============================== | |
* `raneftables` returns a `NamedTuple` where the names are the grouping factor names and the values are some `Tables.jl`-compatible type. Currently this type is a `DictTable` from `TypedTables.jl`. [#634] | |
MixedModels v4.11.0 Release Notes | |
============================== | |
* `raneftables` returns a `NamedTuple` where the names are the grouping factor names and the values are some `Tables.jl`-compatible type. Currently this type is a `DictTable` from `TypedTables.jl`. [#634] | |
MixedModels v4.10.0 Release Notes | |
============================== |
Project.toml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you bump the version to 4.11? we had a 4.10 earlier this morning 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do. I forgot to warn you that I was getting this PR ready to go.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dmbates No stress. It's not like we'll run out of version numbers. 😄
merge and release when ready! |
* undo #634 return Table not DictTable Co-authored-by: Phillip Alday <[email protected]>
MixedModel
properties throughTypedTables.jl
raneftables
to be aNamedTuple
ofDictTables
NamedTuple
ofDictTables
is messy and confusingDictTable
can be awkward b/c the column name "(Intercept)", purposely chosen to be difficult to represent as aSymbol
, must be written asvar"(Intercept)"
or an equivalentFor example,