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

11_temporal_probability_models #17

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

Conversation

mofayezi
Copy link
Member

@mofayezi mofayezi commented Dec 17, 2021

Hi:)
Github does not support math equations in markdown files.
Please open the .md file in an original markdown viewer. (Or simply check this file)
Thanks!

@mofayezi
Copy link
Member Author

@sinatav

- [Conclusion](#conclusion)
- [Resources](#resources)

# Introduction
Copy link

Choose a reason for hiding this comment

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

the intro is a bit too brief, elaborate.


Intuitively, the probability of rain increases from day 1 to day 2 because rain persists.

## Prediction
Copy link

Choose a reason for hiding this comment

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

try to make different sections of your lecture note more relevant to one another; don't jump from a topic to another without proper preparation

- The process of discovering the sequence of hidden states, given the sequence of observations, is known as decoding or inference. The **Viterbi** algorithm is commonly used for decoding.
- The parameters of an HMM are the A transition probability matrix and the B observation likelihood matrix. Both can be trained with the **forward-backward** algorithm.

# Resources
Copy link

Choose a reason for hiding this comment

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

it would be good to tell which reference is for which section

|:--:|
| *Error of DBN particle filtering.* |

# Conclusion
Copy link

Choose a reason for hiding this comment

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

have a more detailed conclusion, the body of your notebook is very long, and I think some important points must be missing here

 return S
```

## Useful links
Copy link

Choose a reason for hiding this comment

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

I think it's better to add a "useful links" section at the end of your notebook and mention all the links there, but that's your choice (just a suggestion)

|:--:|
| *Figure 2* |

# Dynamic Bayes Nets
Copy link

Choose a reason for hiding this comment

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

review your english since this is pretty much formal... I will mention a couple of examples in this section

| *Figure 2* |

# Dynamic Bayes Nets
A Bayesian network is a snapshot of the system at a given time and is used to model systems that are in some kind of equilibrium state. Unfortunately, most systems in the world change over time and sometimes we are interested in how these systems evolve over time more than we are interested in their equilibrium states. Whenever the focus of our reasoning is change of a system over time, we need a tool that is capable of modeling dynamic systems.
Copy link

@sinatav sinatav Dec 24, 2021

Choose a reason for hiding this comment

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

"A Bayesian network is a snapshot of the system at a given time and it's used to model systems in some kind of equilibrium state. Unfortunately, most systems in the world change over time, and sometimes we are more interested in how these systems evolve than in their equilibrium states. Whenever the focus of our reasoning is the change of a system over time, we need a tool capable of modeling dynamic systems."

You use "that" very often it's kinda disturbing :(

# Dynamic Bayes Nets
A Bayesian network is a snapshot of the system at a given time and is used to model systems that are in some kind of equilibrium state. Unfortunately, most systems in the world change over time and sometimes we are interested in how these systems evolve over time more than we are interested in their equilibrium states. Whenever the focus of our reasoning is change of a system over time, we need a tool that is capable of modeling dynamic systems.

A dynamic Bayesian network (DBN) is a Bayesian network extended with additional mechanisms that are capable of modeling influences over time. The temporal extension of Bayesian networks does not mean that the network structure or parameters changes dynamically, but that a dynamic system is modeled. In other words, the underlying process, modeled by a DBN, is stationary. A DBN is a model of a stochastic process.
Copy link

Choose a reason for hiding this comment

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

"that are" :((((
also "network structure or parameters changes" is incorrect...

Copy link

@sinatav sinatav left a comment

Choose a reason for hiding this comment

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

first review

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.

4 participants