Skip to content

Reply differently to questions than to statements. - #1

Open
m-ou-se wants to merge 1 commit into
dazeus:masterfrom
m-ou-se:master
Open

m-ou-se wants to merge 1 commit into
dazeus:masterfrom
m-ou-se:master

Conversation

@m-ou-se

@m-ou-se m-ou-se commented Mar 7, 2020

Copy link
Copy Markdown

Questions will now get a X% yes. reply.

}oracle This code is tested.
That is 0% true.
}oracle Does the code work?
50% yes.

Questions will now  get a 'X% yes.' reply.
Comment thread dazeus-oracle.pl
if (!defined($arg) || $arg eq "") {
reply("You'll have to give the oracle something to work with, " . $sender . ".", $network, $sender, $channel);
} elsif ($arg =~ m/\?$/) {
reply(oracle($arg) . "% yes.", $network, $sender, $channel);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not too keen on the string in question. I'd prefer a full sentence, like the other reply.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I grepped for }oracle in my irc logs, and it's actually pretty hard to come up with some sentence that fits all questions. (But That is X% true almost never fits if it was a question.)

Any ideas?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess I was never too bothered by the oracle's reply to questions. Perhaps we could just use 'likely' instead of 'true' for questions?

Suggested change
reply(oracle($arg) . "% yes.", $network, $sender, $channel);
reply("That is " . oracle($arg) . "% likely.", $network, $sender, $channel);

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

Successfully merging this pull request may close these issues.

2 participants