-
Notifications
You must be signed in to change notification settings - Fork 8
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
Define API for Release 1.0 (and introduction to issues) #18
Comments
In this sense, storing the training data on the model object would be very useful to simplify how to call confidence intervals and other statistical methods, as it will require only the parameters of the stat test instead of model.fit train [t,I] data. 2.1 It's also possible to fit with the recovered population (R). I will think on a more abstract but yet intuitive way for the user to consider R as a separate fit variable, in principle. I will open another issue with the discussion of fitting over two different sets of data, as that will require a more complicated cost function but will be novel. 2.2 Post_regression routines are likely to be one of the most complicated parts of the code and I think it will be great to mantain them in a different file, but they can set attributes of the model. For example, after doing in the future model.ci_block_cv() it would be great to store the confidence intervals in the model object.
Ideally the user would be fetching model results and doing this, but for exploration and debugging I think these two function will be incredibly useful. |
The export and fetch functions are already doing that. We can mimic this behavior
I'm in for this!
I agree they should be in a separate file. We can expose them via "mixins"
Ideally the user would be fetching model results and doing this, but for exploration and debugging I think these two function will be incredibly useful. This also sounds reasonable to me. |
This issue serves as an introduction to GH issues and an entry point to discuss the API of the model.
The model class
Since #17 got merged, the model class (and children) has now the following public API
Note
solve
,set_params
andfit
return the model object (so it's possible to compose the functions like `Model.fit().solve().export()). All the rest return values.As mentioned in #17, there are some new post regression functions that can be included in the class model (analogue to
export
orfetch
).Also as proposed by @felipehuerta17 , we could add a
plot
function.So, concrete questions:
plot
function would look like? What kind of arguments?The text was updated successfully, but these errors were encountered: