Update controller-start-up.md:- Corrects a few grammatical errors #1692
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
PRUBHTEJ
added
the
documentation
Needless, Sir. |
| @@ -2,10 +2,10 @@ | |||
|
|
|||
| The .wbt file contains the name of the controller that needs to be started for each robot. | |||
| The controller name is a platform and language independent field; for example when a controller name is specified as "xyz\_controller" in the .wbt file, this does not say anything about the controller's programming language or platform. | |||
| This is done deliberately to ensure the platform and programming language independence of .wbt files. | |||
| This is done deliberately to ensure the independence between platform and programming language of .wbt files. | |||
There was a problem hiding this comment.
Are you sure that this change is correct?
Because in the new sentence to me it seems that platform and programming languages should be independent from each other.
But are WBT files that are independent from the platform and they are independent from the programming language used.
There was a problem hiding this comment.
That sounds wrong to me as well...
There was a problem hiding this comment.
@PRUBHTEJ: can you propose a rephrasing for this?
There was a problem hiding this comment.
@omichel , @stefaniapedrazzi I'll try to rephrase it as soon as possible .
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
A few grammatical errors are committed in the documentation, which are corrected by this PR.
Tasks
Add the list of tasks of this PR.
Documentation
https://cyberbotics.com/doc/guide/controller-start-up