Convenience wrapper for running subroutines about graviton

[tzaffi]

Hi @zejoeh. I'm glad you are finding graviton useful in your PyTEAL development work.

Integration with PyTeal has been steadily improving. In particular, the soon to be merged feature/abi branch has the class PyTealDryRunExecutor which sounds like it has some overlap with your @SubroutineRunner decorator. I recommend you have a look at the existing code and see if it could be integrated or leveraged in some way. It may also be the case that the functionality that you are proposing is an obvious improvement. We welcome pull requests. We can also continue the discussion via this issue.

[zejoeh]

Hi @tzaffi, yes this indeed looks already like a step in the same direction I was going. I wrote this decorator based on the code in the current master branch, so didn't see you already had a similar thing in the feature branch. I will hopefully find some time in the next days to have a closer look at the feature/abi branch and see if I can contribute anything there.

Probably the only "improvement" from my version is that it returns an instance of a container dataclass for the list of DryRunInspectors on sequences, which also stores the inputs in a format consistent with the output and lets you skip writing out list comprehensions each time you want to retrieve the same attribute for many dryruns. I found it pretty nice for writing out many short tests where you only care about evaluating some output quickly and just want to iterate over the inputs and outputs in pairs, but you could of course achieve exactly the same with a range loop or enumerating over one of the input lists and accessing the elements directly.

[tzaffi]

Thanks for looking at feature/abi. Keep me posted!

[zejoeh]

Hi @tzaffi, I added a draft pull request on the PyTeal repository. Happy to receive feedback, but still adding improvements as they come up in my own project. I think it can be improved significantly by making use of all the work put into the ABI typing, but I have not had the time yet to dig into it.