fix_several-minor-grammar-corrections #2392
Conversation
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
Minor grammatical fixes
|
If your intention is indeed to correct grammar mistakes, please only submit those. This PR changes a lot and having looked at just the first set of changes, they appear to be stylistic rewrites. |
docs/user_guide/rendering.rst
Outdated
|
|
||
| ### Changes Made: | ||
| - Corrected minor grammatical errors. | ||
| - Standardized capitalization and punctuation. | ||
| - Improved readability by breaking up long paragraphs. | ||
| - Fixed typos and inconsistencies. | ||
| - Ensured consistent formatting throughout. | ||
| <summary>input tokens: 1776, prefill time: 1.36s, output tokens: 1792, decode speed: 74.01 tokens/s</summary> |
There was a problem hiding this comment.
Looks like whatever tool you used injected a summary of its work into one of the files.
That's something that definitely needs to go.
There was a problem hiding this comment.
Sure is some IA script to rewrite/fix typo
Remove AI output
|
Really not a big fan of the using of AI without any indication in the PR, even more so when say AI ad a bit of garbage and completely broke doc generation by replacing the content of one file by another... But please, in the future, if you use IA for such task, please let us know. IA can be ok in my opinion, but human readers need to be notified so they can be extra precocious and check for unintended hallucination/side effects. |
Several (mostly minor) grammatical errors have been corrected.