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

Tutorials first #204

Merged
merged 1 commit into from
Jul 18, 2020
Merged

Tutorials first #204

merged 1 commit into from
Jul 18, 2020

Conversation

ChrisRackauckas
Copy link
Member

I always think a tutorials first approach to documentation is much more inviting, because it becomes immediately clear to the user (a) what is solved and (b) how to use it. All of the extra details of what other things you can choose to do can come after the tutorial, but most people want to just look at the first tutorial, copy/paste, and run.

With that in mind, I think the first tutorial could use a bit more. I think a demonstration of surrogate optimization should appear in that first page, otherwise most people won't know it exists.

I always think a tutorials first approach to documentation is much more inviting, because it becomes immediately clear to the user (a) what is solved and (b) how to use it. All of the extra details of what other things you can choose to do can come after the tutorial, but most people want to just look at the first tutorial, copy/paste, and run.

With that in mind, I think the first tutorial could use a bit more. I think a demonstration of surrogate optimization should appear in that first page, otherwise most people won't know it exists.
@codecov
Copy link

codecov bot commented Jul 18, 2020

Codecov Report

Merging #204 into master will increase coverage by 0.24%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #204      +/-   ##
==========================================
+ Coverage   84.60%   84.85%   +0.24%     
==========================================
  Files          19       19              
  Lines        1604     1604              
==========================================
+ Hits         1357     1361       +4     
+ Misses        247      243       -4     
Impacted Files Coverage Δ
src/Optimization.jl 90.63% <0.00%> (+0.70%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5e817a7...72be10d. Read the comment docs.

@ludoro
Copy link
Contributor

ludoro commented Jul 18, 2020

Good point! I will freshen up that part

@ludoro ludoro merged commit 7af24d6 into master Jul 18, 2020
@ludoro ludoro deleted the ChrisRackauckas-patch-1-1 branch July 27, 2020 14:41
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.

None yet

2 participants