Skip to content

Commit

Permalink
chore: Delete obsolete estimator scripts (#6786)
Browse files Browse the repository at this point in the history
`ci.sh`: No longer used because the buildkite pipeline has ben archived.
The same task, among others, is performed by continuous estimation.

`compare_costs.py` and `safety_multiplier.py`: Both operate on the
replaced JSON runtime config format. We have not used these scripts in
a long time, since estimations did not produce complete JSONs anyway.
But there always was the chance that we might go that way again.
Now, however,  that #6693 replaced the JSON format, these scripts are
truly obsolete and better of deleted.
  • Loading branch information
jakmeier authored May 11, 2022
1 parent d312bad commit 78d418c
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 158 deletions.
6 changes: 0 additions & 6 deletions runtime/runtime-params-estimator/emu-cost/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -130,9 +130,3 @@ To execute commands in already running container first find its id with:
and the use container ID for `docker exec` command, like:

docker exec -it e9dcb52cc91b /host/qemu-linux/bin/qemu-x86_64 -d plugin -plugin file=/host/qemu-linux/plugins/libcounter.so /host/nearcore/runtime/runtime-params-estimator/emu-cost/counter_plugin/test_binary

## Useful scripts

The script `../estimate.sh` will handle all the setup mentioned above. It uses `wasmer` by default, but you can pass `wasmtime` and `lightbeam` as well.

To just test the compile time costs use `../compile.sh`, which also takes an optional vm argument. Its final output is (fee per byte, base fee)\
21 changes: 0 additions & 21 deletions runtime/runtime-params-estimator/emu-cost/ci.sh

This file was deleted.

106 changes: 0 additions & 106 deletions runtime/runtime-params-estimator/emu-cost/compare_costs.py

This file was deleted.

25 changes: 0 additions & 25 deletions runtime/runtime-params-estimator/emu-cost/safety_mutiplier.py

This file was deleted.

0 comments on commit 78d418c

Please sign in to comment.