Fixed punctuation. Introduced new issue links.

twitchtv / acm-mmsys-2020-grand-challenge

Assets for Twitch's ACM MMSys 2020 Grand Challenge

Apache License 2.0

42 stars 21 forks source link

Fixed punctuation. Introduced new issue links. #9

Closed BlaineCan closed 4 years ago

BlaineCan commented 4 years ago

I made changes to the readme files in twitchtv/acm-mmsys-2020-grand-challenge and twitchtv/acm-mmsys-2020-grand-challenge/dashll-server. I corrected some general punctuation errors (periods, commas, colons) and added links that point to opening a new issue. This just eliminates a few clicks and provides a tiny shortcut. Preview below:
issue-link

johnBartos commented 4 years ago

Thanks @Whatchamazog! I personally do not like using periods in lists because it can be confusing around code/templates. For example, in:

`frame=  202 fps= 28 q=30.0 q=25.0 q=26.0 size=N/A time=00:00:06.70 bitrate=N/A dup=6 drop=0 speed=0.943x`.

Developers may accidentally copy the period. For consistency I chose not to use them in any list. However, I like the rest of your edits and would be happy to merge those. Thanks.