Delegations should use Percents or Amounts

[rigelrozanski]

Remove the use of "Max" shares - just 100% Shares

Based on feedback from validators

[cwgoes]

Wasn't there a concern about limited rational precision (IIRC correctly from a discussion)?

Or do you mean literally having the transaction take a percentage?

[rigelrozanski]

Yeah - if limited rational precision is an issue we need to fix it anyways - and yes I mean literally taking a percentage - ex. you could redelegate either "5 delegator shares" or "10% of your delegator shares held in a bond"

[cwgoes]

If we fix limited rational precision, can't this just be done at the UI layer? What reason would we have to compute the percentages on-chain?

I agree gaiacli could expose percentages though.

[rigelrozanski]

Oh alright - I like just adding it to gaiacli, only downside is gaiacli has to do a query if percentage is chosen

[ValarDragon]

Do we want gaiacli to round down or just do standard rounding when giving percentages?

[rigelrozanski]

I don't think it actually needs to round as it's using rational math - just multiple the percent and pass the result rational in the message