crypto

Is there an existing issue for this?

• I have searched the existing issues

Is your feature request related to a problem? Please describe.

Having docstrings for all methods and classes will help new contributors understand the codebase and also provide a method of autogenerated API docs. Right now only some methods and classes have docstrings.

Describe the solution you'd

A useful feature where the user specifies the quantity to take in an orderbook and callback returns the average price paid for that lot size. For example if the offers in the order book for BTC-USD was the following:

Price | size
11,000 | 1
10,500 | 1
10,000 | 1

If the user specified quantity 3, the callback would return 10,500.

There appears to be a broken method in hash extend that is not valid due to it missing a def ():
In addition to this, there appears to be no body for the method.

https://github.com/nccgroup/featherduster/blob/52ed10dc441c45649ce4f4bedbf59bb72c63cda8/feathermodules/hash/hash_extend.py#L11

It is unclear how to use something like get_token_balance() with respect to the Uniswap v2 factory contract. I am trying to view the amount of a token in the Uniswap v2 liquidity pool, and it seems all I can check is the ERC20 token balance globally.

Edit: I see now I can use get_ex_token_balance(), but I am receiving the Uniswap v1 pool amounts. When I switch to version 2 the Factory methods