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

Payouts for Cancel in ReadMe's payout table can be stated more simply #19

Open
ghost opened this issue Nov 16, 2018 · 2 comments
Open
Assignees

Comments

@ghost
Copy link

ghost commented Nov 16, 2018

In the ReadMe's payouts matrix, the Cancel column can be stated more simply as:

Cases Success Cancel
Owner P 0
Bidder 0 P
CFO T - P T - P
Sum T T

rather than

Cases Sucess Cancel
Owner P 0
Bidder 0 T - S * P
CFO T - P S * P
Sum T T

since these are mathematically equivalent if P = T / (1 + S), and the ReadMe states that P always equals T / (1 + S).

This serves the further purpose of emphasizing the similarity between the fee structure of the Success case and the Cancel case. Currently, a first glance makes it seem like the Success case and the Cancel case do not have corresponding fee structures, it took a little math to realize that.

@ghost ghost changed the title Payouts for cancelOffer do not match ReadMe's payout table Payouts for Cancel in ReadMe's payout table include more variables than are necessary Nov 16, 2018
@ghost ghost changed the title Payouts for Cancel in ReadMe's payout table include more variables than are necessary . Nov 16, 2018
@ghost ghost closed this as completed Nov 16, 2018
@ghost ghost changed the title . Payouts for Cancel in ReadMe's payout table can be stated more simply Nov 16, 2018
@ghost ghost reopened this Nov 16, 2018
@hwrdtm
Copy link
Contributor

hwrdtm commented Nov 16, 2018

Thanks for your feedback @TomLeeFounder ! We'll take that into consideration and try to make our descriptions clearer.

@hwrdtm hwrdtm self-assigned this Nov 16, 2018
@jordankitty
Copy link
Contributor

Thanks for your participation, @TomLeeFounder! Our team has reviewed your submission, and we are pleased to reward you for your report.

Severity: Note
Points: 10

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