Skip to content
This repository has been archived by the owner on Apr 24, 2020. It is now read-only.

fixing additional (None) in finite_markov #40

Merged
merged 2 commits into from
Oct 4, 2018
Merged

fixing additional (None) in finite_markov #40

merged 2 commits into from
Oct 4, 2018

Conversation

fkazemian
Copy link
Contributor

@fkazemian fkazemian commented Sep 17, 2018

(Issue #27) in the text of finite_markov there is an additional (None) which is not related to text.
45193925-00048400-b294-11e8-99d1-df837f1fb26c

This is not because of
If we think of :math:\psi_{t+1} and :math:\psi_t as *row vectors* (as is traditional in this literature), these :math:nequations are summarized by the matrix expression

this is because of the formula before that. It was something like

.. _mc_fdd:

.. math::

\psi_{t+1}(y) = \sum_{x \in S} P(x,y) \psi_t(x)

I add some space before .. math:: like

.. _mc_fdd:

    .. math::

    \psi_{t+1}(y) = \sum_{x \in S} P(x,y) \psi_t(x)

and that (None) disappeared.

screenshot from 2018-09-17 16-59-56

screenshot from 2018-09-17 17-08-38


\psi_{t+1}(y) = \sum_{x \in S} P(x,y) \psi_t(x)
.. math::
\psi_{t+1}(y) = \sum_{x \in S} P(x,y) \psi_t(x)
Copy link
Collaborator

Choose a reason for hiding this comment

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

@fkazemian please add a newline between math directive and math

@fkazemian
Copy link
Contributor Author

screenshot from 2018-10-04 11-24-03

Hi @mmcky , I just updated this PR, would you please merge it? Thanks

@fkazemian fkazemian added ready and removed in-work labels Oct 4, 2018
@mmcky mmcky merged commit 10287ae into master Oct 4, 2018
@mmcky mmcky deleted the fix_label branch October 4, 2018 04:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants