trippw / sts-crawler Goto Github PK

The bot seems to always have a 100% confidence rate for "Beta", even though it's a common non-card word for thread titles, e.g. Playing since beta, 480 hours logged, I have now beat the heart on A20 with all four characters, when is the 5th coming?. Not sure how confidence is calculated right now, and I assume it'll be too much work right now to train it (e.g. measure confidence against how likely a card-name is used in a non-card-sense by bulk downloading/analyzing r/slaythespire data), but putting "Beta" on the ignore list for now seems reasonable. Especially since it seems very rare in practice that anyone will start a thread talking about the "Beta" card, given its actual mechanics.

(Collect might be another common false positive, or at least in the case when it's uncapitalized)

These cards always work together, so it would be helpful for them to be mentioned in the bot response if at least one is found.

Example 1

Title

Win with an Omega Deck!

Response

I am 100% sure you mentioned [[Omega]].

You may be interested in [[Alpha]] and [[Beta]] as well.

Example 2

Title

Title: Alpha/Omega is a tons of fun.

Response

I am 100% sure you mentioned [[Alpha]] and [[Omega]].

You may be interested in [[Beta]] as well.

Courier does not match any relic, despite it being a relic. Well... That's not 100% true. The relic is actually called "the courier" but most people refer to it as just "courier".

Suggestion: Add an "articles removal" function when pulling in object names (cards, relics, etc) to make it possible to ignore "the, a, etc" when referencing cards/relics that contain an article in their proper name.