Skip to content

21_reinforcement_learning#27

Open
sayehjarollahi wants to merge 13 commits intosut-ai:masterfrom
sayehjarollahi:master
Open

21_reinforcement_learning#27
sayehjarollahi wants to merge 13 commits intosut-ai:masterfrom
sayehjarollahi:master

Conversation

@sayehjarollahi
Copy link

No description provided.

@sayehjarollahi
Copy link
Author

@sinatav

Copy link

Choose a reason for hiding this comment

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

don't use unnecessary "that"s in a formal text... for example: "which solves problems that involve" can be "which solves problems involving", if you don't like the sound of "involving learning," consider reconstructing =))))))

Copy link

Choose a reason for hiding this comment

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

Avoid typos, please... also, ameliorate comprehension by changing the structure: "According to Learning policy, there are two types of RL: Passive Reinforcement Learning and Active Reinforcement Learning."

Copy link
Author

Choose a reason for hiding this comment

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

Do you mean that we should only change the structure of that specific sentence?

Copy link

Choose a reason for hiding this comment

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

So basically revise your grammar :)))
"policy of the agent": "the agent's policy"
"exploition": typo
is done: are done

Copy link

Choose a reason for hiding this comment

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

punctuation...


<div id='TemporalDifference(TD)Learning'/>

# Temporal Difference (TD) Learning
Copy link

Choose a reason for hiding this comment

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

Provide the reader with some preparation & introduction before jumping to a topic. Work a bit more on the structure of your notes

Copy link
Author

Choose a reason for hiding this comment

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

Should we add introduction before each subtopic? We have already divided the main topic into subtopics and there is an introduction at the beginning of the lecture note . There is also a definition at the beginning of some subtopics.

Copy link

Choose a reason for hiding this comment

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

change this to summary & conclusion

Copy link

Choose a reason for hiding this comment

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

Don't just use bullets to point out things that were mentioned before... do sth like "In this article we FILANed & DASTANed. We learned FILAN because... So..." (disclaimer: I don't mean you should lose the bullets) this is a very critical part of your work

* Aircraft control and robot motion control

<div id='resources'/>

Copy link

Choose a reason for hiding this comment

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

you can have an "Other useful links" section and provide some interesting material related to these topics (just a suggestion)

Copy link

Choose a reason for hiding this comment

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

okay I know I should stop with the English-related comments, but the last line is bugging me :)))) how about "a new method is required, called..."

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.

2 participants

Comments