-
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
Re-export a few generics and types from the TypedTables package #662
Comments
@palday, what would you think of changing the structure of the fitlog to be a
|
I am also considering changing the format of the |
An alternative for the parametric bootstrap is just to introduce a new property, say |
Ergonomic design is only "obvious" in retrospect. We do a decent job of keeping release notes and haven't had a breaking release in a while, so I'm not terribly opposed to having a clean-up cycle + major release in the near future. If the breaking changes don't impact a particular user, then they can feel free to have That said, I haven't yet thought about the proposal. I would suggest implementing it and seeing how much we like it for a little bit. As soon as we have something breaking on |
Another couple of things to throw in the hopper for a breaking version is to change the name and default value of the argument to I am also thinking of changing the structure to be a row-oriented table (Vector{NamedTuple}) with the first column being the objective and the remaining columns being the values of the theta parameters. |
see also #618 |
Exactly. I am getting to the point where I keep coming up with the same ideas, over and over, and considering them to be novel approaches. I suppose in a way that this is better than coming up with contradictory ideas and considering them to be worthwhile, novel approaches. |
I am finding it quite convenient to work with the
Table
type from TypedTables.jl and have added a dependency on that package in theprofile
branch. I plan to re-export a few generics and types likeTable
andgroup
and, maybe,groupinds
fromMixedModels
itself. This is just a heads-up and request for comment in case someone thinks this is a terrible idea.The text was updated successfully, but these errors were encountered: