pavitthrap

Results 10 comments of pavitthrap

Update: will replace the MARF with a smaller one (up to 10 blocks). Will re-benchmark the functions with the new MARF once the PR for the function `replace-at` is done.

> > This PR tests runtimes against a real DB, so the runtime cost includes the I/O costs of hitting a real MARF. Before, we had determined the read_count and...

> * It appears [Load costs-3 contract  #3266](https://github.com/stacks-network/stacks-blockchain/issues/3266) is needed for 2.1. Could we include it to this PR? > * Do we have a programmatic check that the ClarityCostFunction...

> This LGTM, @pavitthrap, but I am a little concerned that the `Any` case can end up doing 2x admission checks for an invalid tx. Would it make sense for...

> @pavitthrap I'm curious about where this PR stands these days and whether a release ETA might be available? Update here: https://github.com/blockstack/stacks-blockchain/issues/2774#issuecomment-922077240

@wileyj @CharlieC3 Wondering what your thoughts are on the response type in `handle_get_health` (in `rpc.rs`). Would you prefer 1) always getting a HttpResponseType of GetHealth, with a detailed error field...

I prefer using the distinction of which clarity version each function is: `Clarity1` / `Clarity2`. Also, it would be nice to make the functions alphabetical as well. This would require...

Re-implemented in #3729