Skip to content
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

When is "American" profane? #3

Closed
anseljh opened this issue Jun 1, 2016 · 4 comments
Closed

When is "American" profane? #3

anseljh opened this issue Jun 1, 2016 · 4 comments

Comments

@anseljh
Copy link

anseljh commented Jun 1, 2016

I ran some text through Alex, and it told me "American" is profane in some cases. Thanks, Trump. 😝

More seriously, I can't think of a context in which "American" would be profane, and its presence in this package seems likely to generate almost entirely false positives. Should it be removed? (There are probably others in this category too.)

It looks like the origin of "American" here is Luis von Ahn's research group's Offensive/Profane Word List.

@wooorm
Copy link
Member

wooorm commented Jun 1, 2016

Hahaha. 😆

Yeah, it’s from that list. I recently created cuss for that - also incorporated into alex to tone down/up some messages, recently.

If you’re using alex, I suggest using it’s ignore mechanisms (only work w/ markdown).
If you’re using profanities to count some score yourself, it might still be a good metric.
Otherwise, maybe switch to using cuss directly?

Does that help? I’ll re-open if not.

@wooorm wooorm closed this as completed Jun 1, 2016
@anseljh
Copy link
Author

anseljh commented Jun 1, 2016

Thanks, that does help. This came up while I was using atom-linter-alex:

2016-06-01-american

Maybe it should use cuss instead, with a threshold of 1 or 2?

@wooorm
Copy link
Member

wooorm commented Jun 1, 2016

Ah, I see. Thing is, It’s already using cuss for changing the messages. I thought “it’s profane in some cases” would be adequate?

Maybe atom-linter-alex should switch to using “Warning” for level-0 cussing instead?

@wooorm
Copy link
Member

wooorm commented Jun 8, 2016

@anseljh Better message types are included in linter-alex 3.1.0: get-alex/atom-linter-alex@2da0000!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants