Skip to content
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

Improved OptimizationProblem, OptimizationFunction, and Quadrature docs #174

Merged
merged 1 commit into from
May 28, 2022

Conversation

ChrisRackauckas
Copy link
Member

Fixes #159 and fixes #158

@ChrisRackauckas
Copy link
Member Author

@Vaibhavdixit02 @ValentinKaisermayer notice the full definition of the interface. We don't quite hit it right now (especially with the IIP stuff), but this is where we need to go for full interface conformity. You might want to check out https://scimlbase.sciml.ai/dev/ for the full interface definition as well.

@codecov
Copy link

codecov bot commented May 28, 2022

Codecov Report

Merging #174 (1cf6af8) into master (052412a) will decrease coverage by 10.52%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #174       +/-   ##
==========================================
- Coverage   17.81%   7.28%   -10.53%     
==========================================
  Files          40      40               
  Lines        2986    2963       -23     
==========================================
- Hits          532     216      -316     
- Misses       2454    2747      +293     
Impacted Files Coverage Δ
src/problems/basic_problems.jl 0.00% <ø> (-17.86%) ⬇️
src/scimlfunctions.jl 6.60% <ø> (-1.95%) ⬇️
src/alg_traits.jl 0.00% <0.00%> (-60.00%) ⬇️
src/solutions/ode_solutions.jl 0.00% <0.00%> (-56.53%) ⬇️
src/solutions/basic_solutions.jl 0.00% <0.00%> (-55.56%) ⬇️
src/utils.jl 12.16% <0.00%> (-52.48%) ⬇️
src/SciMLBase.jl 0.00% <0.00%> (-50.00%) ⬇️
src/ensemble/ensemble_problems.jl 0.00% <0.00%> (-50.00%) ⬇️
src/ensemble/basic_ensemble_solve.jl 0.00% <0.00%> (-40.21%) ⬇️
src/remake.jl 0.00% <0.00%> (-40.00%) ⬇️
... and 11 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@ChrisRackauckas ChrisRackauckas merged commit a2cb72b into master May 28, 2022
@ChrisRackauckas ChrisRackauckas deleted the docstrings branch May 28, 2022 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Finish the QuadratureProblem docstring Finish the OptimizationProblem docstring
1 participant