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

Modify documentation. #34

Merged
merged 14 commits into from
Nov 1, 2017
Merged

Modify documentation. #34

merged 14 commits into from
Nov 1, 2017

Conversation

shizejin
Copy link
Member

@shizejin shizejin commented Apr 5, 2017

close #32.

Thanks very much for the suggestions of @oyamad and @sglyon!

Preview on my forked repo here.


## [Lectures](@id lectures)

Some lectures of this package is available on [QuantEcon](https://lectures.quantecon.org):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better to say "notebooks" instead of "lectures".

@oyamad
Copy link
Member

oyamad commented Apr 5, 2017

Thanks! Looks great.

@@ -0,0 +1,64 @@
# Games.jl

*Games.jl* is a [`Julia`](http://www.julialang.org) package for computation of Game Theory.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Games.jl is a Julia package for Game Theory related computations.


## Installation

*Games.jl* is an unregistered package that currently under development.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Games.jl is an unregistered package that is currently under development.

player1 = Player([3 1; 0 2])
```

A 2-player `NormalFormGame` can be created either by passing `Player` instances.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A 2-player NormalFormGame can be created either by passing Player instances**,**

g = NormalFormGame((player1, player2))
```

or by passing a payoff bimatrix directly.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or by passing a payoff matrix directly.
(to keep consistent with line 23)

g = NormalFormGame(payoff_bimatrix)
```

After construction of `NormalFormGame`, we can find its Nash Equilibria by using methods of `Games`, for example, `pure_nash` finds all pure action Nash Equilibria by enumeration.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After constructing a NormalFormGame, we can find its Nash Equilibria by using methods in Games. For example, pure_nash is a method that finds all pure action Nash Equilibria by enumeration.

docs/README.md Outdated
```

The first line sets the order of pages. You can add a file name if you want it to generate a single page, or the name of section you want to create. In the following you need to declare which files are included in each section. Note that you shouldn't put a comma or dot at the end of each line.
The first line sets the order of pages. You can add a file name if you want it to
generate a single page, or the name of section you want to create. In the following
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or the name of the section you want to create.

docs/README.md Outdated
The first line sets the order of pages. You can add a file name if you want it to generate a single page, or the name of section you want to create. In the following you need to declare which files are included in each section. Note that you shouldn't put a comma or dot at the end of each line.
The first line sets the order of pages. You can add a file name if you want it to
generate a single page, or the name of section you want to create. In the following
you need to declare which files are included in each section. Note that you shouldn't
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that you should not

docs/README.md Outdated
you need to declare which files are included in each section. Note that you shouldn't
put a comma or dot at the end of each line.

It is not necessary to declare structure for every file included in the Module.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is not necessary to declare Structure for every file included in the Module.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I am referring to structure literally here, but not the file Structure.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, then I think "It is not necessary to declare the structure of every file included in the Module." would be good?

docs/README.md Outdated
#### Homepage

The Homepage is written in `/docs/src/index.md`. Please keep the last part in Homepage,
called `Library Outline`, being empty. It will be filled by `auto_doc_gen.jl` automatically.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

called Library Outline, empty.
(being is not needed)

docs/README.md Outdated
It is not necessary to declare structure for every file included in the Module. Files not mentioned in `Structure` will simply generate a single page automatically.
This file includes information for "Documenter.jl" to generate documentation.
Details [here](https://juliadocs.github.io/Documenter.jl/stable/man/guide.html#Output-formats-1)
for how to use it.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Details on how to use it can be found here.

@codecov-io
Copy link

codecov-io commented Sep 4, 2017

Codecov Report

Merging #34 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #34   +/-   ##
=======================================
  Coverage   93.78%   93.78%           
=======================================
  Files           6        5    -1     
  Lines         338      338           
=======================================
  Hits          317      317           
  Misses         21       21
Impacted Files Coverage Δ
src/random.jl 100% <ø> (ø) ⬆️
src/support_enumeration.jl 98.66% <ø> (ø) ⬆️
src/pure_nash.jl 100% <ø> (ø) ⬆️
src/normal_form_game.jl 92.24% <ø> (ø) ⬆️
src/repeated_game.jl 90.43% <100%> (+0.71%) ⬆️

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 e5fe948...51c21eb. Read the comment docs.

@coveralls
Copy link

coveralls commented Sep 25, 2017

Coverage Status

Changes Unknown when pulling 43dc2de on shizejin:modify_docs into ** on QuantEcon:master**.

@coveralls
Copy link

coveralls commented Sep 25, 2017

Coverage Status

Changes Unknown when pulling 5b40851 on shizejin:modify_docs into ** on QuantEcon:master**.

@oyamad oyamad added the WIP label Sep 26, 2017
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 679fa03 on shizejin:modify_docs into ** on QuantEcon:master**.

1 similar comment
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 679fa03 on shizejin:modify_docs into ** on QuantEcon:master**.

@coveralls
Copy link

coveralls commented Oct 8, 2017

Coverage Status

Changes Unknown when pulling 98412f6 on shizejin:modify_docs into ** on QuantEcon:master**.

@shizejin
Copy link
Member Author

This is ready to be merged.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.787% when pulling 51c21eb on shizejin:modify_docs into e5fe948 on QuantEcon:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.787% when pulling 51c21eb on shizejin:modify_docs into e5fe948 on QuantEcon:master.

@oyamad
Copy link
Member

oyamad commented Nov 1, 2017

Let me merge this as is. And then let's discuss for further improvements, if any, in a separate issue.

@oyamad oyamad merged commit f9a2e25 into QuantEcon:master Nov 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Todo list for documentation
5 participants