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.

Sign up for GitHub

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

Fix buy->sell typo #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix buy->sell typo #2

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9c4db6b
Fix buy->sell typo
langner Apr 27, 2018
7c3ff80
Fix typo
langner Apr 27, 2018
13ba6c6
Remove stray backticks
langner Apr 27, 2018
ab91e3e
Fix another typo
langner Apr 27, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions docs/quickstart/quickstart.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -285,7 +285,7 @@ The *"exit concept"* will be an easy one:

Additionally and to simplify:

- Do only allow a Buy order if not yet in the market
- Do only allow a Sell order if not yet in the market

.. note:: The *next* method gets no "bar index" passed and therefore it seems
obscure how to understand when 5 bars may have elapsed, but this has
Expand Down Expand Up @@ -461,7 +461,7 @@ Portfolio above the initial 100000 monetary units the strategy started with.
Customizing the Strategy: Parameters
====================================

It would a bit unpractical to hardcode some of the values in the strategy and
It would be a bit unpractical to hardcode some of the values in the strategy and
have no chance to change them easily. *Parameters* come in handy to help.

Definition of parameters is easy and looks like::
Expand All @@ -487,7 +487,7 @@ the strategy to the Cerebro engine::
here for anyone looking at old samples of the sources. The sources
have been update to use::

cerebro.addsizer(bt.sizers.FixedSize, stake=10)``
cerebro.addsizer(bt.sizers.FixedSize, stake=10)

Please read the section about *sizers*

Expand Down Expand Up @@ -812,7 +812,7 @@ Results:

* For periods below 18 the strategy (commissionless) loses money.
* For periods between 18 and 26 (both included) the strategy makes money.
* Above 26 money is lost agagin.
* Above 26 money is lost again.

And the winning period for this strategy and the given data set is:

Expand Down