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

[EN] [ES] 10-3 figure edit and captions added #743

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

Conversation

mvcaro
Copy link
Contributor

@mvcaro mvcaro commented Jan 4, 2021

With reference to #493

  1. There were two 'Fig. 5', each under Emulator and Controller sections.

  2. Therefore, the one under Controller section has been changed to 'Fig. 6' and the text refering to this figure ('Refer to Figure 5' under Controller section) has been edited accordingly

  3. As the figure under Detail Model Structure had a caption within the image (pytorch-Deep-Learning-master/docs/images/week10/10-3/fig14.png), the figure was edited to remove it as it would produce confusion

  4. Figure mentioned in (3) has had its caption added - it is now Fig 7

  5. Caption for the figure under Examples of Movement section has been added, which is Fig 8

Deleted figure caption within the image so that Figure captioning would not be confusing
Changes made:
There were two different figures captioned as 'Fig. 5', the first under the Emulator section, and the next under the controller section., therefore 1 and 2 below were done.
1. 'Fig. 5' under Emulator section was changed to 'Fig. 6'
2. Under Controller section, updated to 'Refer to Figure 6' 
3. 'Fig. 7' updated to 'Fig. 8'
4. Caption for Fig. 7 was missing and has been added
1. 'Fig.5' updated to 'Fig.6' (as there were two 'Fig.5', each found under Emulator and Controller sections)
2. 'Refer to Figure 5' updated to 'Refer to Figure 6' (under controller section,to follow new figure captioning mentioned in (1))
3. Caption added for Fig 7
4. Caption added for Fig 8
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.

1 participant