-
Notifications
You must be signed in to change notification settings - Fork 18
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
Include objective function values and gradients at nominal parameter values #259
Comments
I would be happy to validate with PEtab.jl, and I fully agree that many of these things should be included. From experience comparing gradients against AMICI, we have to use really small tolerances (around The only things I wonder about are:
|
Great. Let's talk when you are available.
Right, there will be numerical differences for sure. I would try to deposit reference values obtained with low integration tolerances and leave it up to users to decide what differences they are willing to accept.
Strictly needed, no. However, from own experience, if you have some failing test, it's nice if you can quickly narrow down the problem. For example: Is the problem in the initial conditions of the simulations? Does it occur later in the simulation? Is it some specific condition? Is the likelihood okay, but only the posterior differs? |
Fair point, we can then include nllh per simulation condition as well as prior to help pinpoint issues. And agree, we can do objective value at least for each problem.
Great, I have some things to wrap up before holidays, and then I will be gone until 3:rd of February, but after we can coordinate everything. |
As mentioned in #208, I think the relevance of this repository could be increased further, by including objective function values and gradients at the provided nominal parameter values, supporting its use a PEtab integration test suite.
I would like to include:
The specific format is to be discussed. I'd suggest using some easily extendable json/yaml file in each problem directory.
I could provide the results from pypesto/amici. Would somebody be willing to do the same using another tool for validation?
The text was updated successfully, but these errors were encountered: