We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
There's a few things that I can see.
There can be a view here:
https://github.com/JuliaDiffEq/DiffEqParamEstim.jl/blob/master/src/two_stage_method.jl#L95
We should probably transpose this earlier:
https://github.com/JuliaDiffEq/DiffEqParamEstim.jl/blob/master/src/two_stage_method.jl#L98
The cost function could also be deconstructed like the build_loss_objective in order to be more efficient.
https://github.com/JuliaDiffEq/DiffEqParamEstim.jl/blob/master/src/two_stage_method.jl#L96
Those copies could go away and this could be pre-cached
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
There's a few things that I can see.
There can be a view here:
https://github.com/JuliaDiffEq/DiffEqParamEstim.jl/blob/master/src/two_stage_method.jl#L95
We should probably transpose this earlier:
https://github.com/JuliaDiffEq/DiffEqParamEstim.jl/blob/master/src/two_stage_method.jl#L98
The cost function could also be deconstructed like the build_loss_objective in order to be more efficient.
https://github.com/JuliaDiffEq/DiffEqParamEstim.jl/blob/master/src/two_stage_method.jl#L96
Those copies could go away and this could be pre-cached
The text was updated successfully, but these errors were encountered: