-
Notifications
You must be signed in to change notification settings - Fork 25
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
Projectile: Code: Fix \brief for doxygen #3218
Comments
I am trying to use the |
I added a comment in #3248 (#3248 (comment)) that seems relevant here. If we distill the purpose into its essential elements, instead of thinking of it as a line of text in an introduction, then we would drop the "efficiently and correctly" part. The purpose would now just be the "functional purpose". (I used functional in the same sense as functional requirements. The purpose is what the software does, not the quality of how it does it.) If the purpose is simply "predict whether a launched projectile hits its target", we could use that in the README and (I think) in the introduction by changing the sentence to "A program to ..." as you suggest. I feel like that pattern could be repeated across all of the examples. |
I'm fine with keeping the non-functional requirements in the introduction. We do want the software to be efficient, even though we don't do much about it other than say what we want. 😄 |
The
\brief
tag in the generatedProjectile.py
file is currently a comment to the developers, as opposed to a comment to users of the code. The current tag says "Contains the entire Projectile program." We should fix this to provide the purpose of the program. The purpose\brief
should be "Efficiently and correctly predict whether a launched projectile hits its target." Ideally the purpose here should come from the same source as for the README file (#3159) and the SRS.The text was updated successfully, but these errors were encountered: