pkel / cpr

consensus protocol research
8 stars 2 forks source link

Test tailstorm[ll] reward schemes. #30

Open pkel opened 1 year ago

pkel commented 1 year ago

It happens from time to time that I have to change the implementation of reward schemes. Last time in #28. Currently I do not have any tests for these functions. I resort to manually looking at the output of make visualize. Currently everything looks fine, but the manual inspection is prone to error.

If reward scheme bugs sneak it, they probably will stay undetected for some time. Maybe invalidate expensive RL results.

I should add some tests for that. Maybe render short blockchains with rewards into a string. Then do expect tests.