Meridian-IE / impact-evaluator

Impact Evaluator smart contract
Other
4 stars 0 forks source link

Add `reward()` implementation. Closes #6 #3

Closed juliangruber closed 1 year ago

juliangruber commented 1 year ago

Closes #6

AmeanAsad commented 1 year ago

Hey @juliangruber , can you elaborate on the setRoundReward function usage?

Is it setting the expected amount of funds for that round?

juliangruber commented 1 year ago

It's setting the reward for all future rounds, until changed again. We can probably improve, but for now this is the simplest way I could think of that lets us control the cost per round

AmeanAsad commented 1 year ago

I see, so when the reward function is called, the reward contract that spins out is funded with that amount. This makes sense and works for now. The only caveat is that we have to make sure the IE contract is funded before rounds.

juliangruber commented 1 year ago

Agreed. How else would we solve this?

AmeanAsad commented 1 year ago

@juliangruber I think this is a good solution for now. Im thinking the condition that the contract must be funded acts as an incentive for stakeholders to participate. More funds will likely be correlated with better service and vice versa.