ajayyy / DeArrow

Crowdsourcing better titles and thumbnails on YouTube
https://dearrow.ajay.app
GNU General Public License v3.0
1.41k stars 38 forks source link

'Title Case' title format does not fully capitalise acronyms #120

Closed Reeceeboii closed 1 year ago

Reeceeboii commented 1 year ago

For example, applying Title Case on a title that contains the string "ai" will result in "Ai" rather than "AI". There should be a list of special cases for common acronyms (or another similar implementation) to avoid incorrectly capitalised acronyms.

Example below.

Original: Screenshot 2023-08-01 134917

'Title Case' applied: Screenshot 2023-08-01 134936

ajayyy commented 1 year ago

Right now, we allow any already capitalized words 3 letters or less, and have a whitelist for the larger ones. This reduces the number that we need to keep track of.

For some reason they didn't capitalize it in the original here though

I think maintaining a list for smaller acronyms like this would be a lot of work and probably not worth it

ajayyy commented 1 year ago

The edge case of them capitalizing it wrong probably isn't that common

Reeceeboii commented 1 year ago

Right now, we allow any already capitalized words 3 letters or less, and have a whitelist for the larger ones. This reduces the number that we need to keep track of.

For some reason they didn't capitalize it in the original here though

I think maintaining a list for smaller acronyms like this would be a lot of work and probably not worth it

Okay that's fine, ty for reply. Happy for this to be closed unless you want it open for whatever reason :)