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

Typo in documentation #511

Closed
slwu89 opened this issue Jul 19, 2023 · 1 comment · Fixed by #512
Closed

Typo in documentation #511

slwu89 opened this issue Jul 19, 2023 · 1 comment · Fixed by #512

Comments

@slwu89
Copy link

slwu89 commented Jul 19, 2023

In this section (http://juliapomdp.github.io/POMDPs.jl/latest/concepts/#POMDPs-and-MDPs), the docs state that "Z is the agent's observation space, and O defines the probability of receiving each observation at a transition".

However in this section (http://juliapomdp.github.io/POMDPs.jl/latest/def_pomdp/#State,-action-and-observation-spaces) it says "The state, action and observation spaces (S, A, and O)" and later on "The transition and observation keyword arguments are used to define the transition distribution, T, and observation distribution, Z, respectively."

So it seems like the docs need to consistently select one symbol to represent the observation space and another for the function that takes in an action, state, and observation and returns a probability.

lassepe added a commit that referenced this issue Jul 19, 2023
@lassepe
Copy link
Member

lassepe commented Jul 19, 2023

Thank you for letting us know. That was indeed a typo (O is the agent's observation space, Z is the observation probability). Now fixed.

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 a pull request may close this issue.

2 participants