Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kch/fix fee history #1

Open
wants to merge 85 commits into
base: main
Choose a base branch
from

Conversation

kamilchodola
Copy link

Motivation

Solution

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

sslivkoff and others added 30 commits July 11, 2023 17:50
The results.json was not generated because vegeta was not found.
For some reason the error was caught and silenced which made it harder
to debug.
1) adding `ipykernel` fixed:
```
jupyter_client.kernelspec.NoSuchKernel: No such kernel named python3
```

2) adding `ipython_genutils` fixed:
```
ModuleNotFoundError: No module named 'ipython_genutils'
```

3) bumping the `nbconvert` version fixed:
```
ImportError: cannot import name 'contextfilter' from 'jinja2'
(/home/flood/.local/lib/python3.11/site-packages/jinja2/__init__.py)
```

This was tested within a Docker container, built with:
```
docker build --tag paradigmxyz/flood:latest .
```
Ran with:
```
docker run --rm --volume ${PWD}/output:/output paradigmxyz/flood:latest \
eth_getBlockByNumber NODE:8545 --output /output/
```
sslivkoff and others added 30 commits July 27, 2023 17:20
Add existing report example link in report generation section
Fix parquet file download error because of a missing curl binary.
This was happening on the `eth_getBalance` test for instance.
Note that flood swallowed the exception making the error harder to spot.

Update the CI to also test against eth_getBalance which triggers a
parquet file download.
Also removes the redundant node2, this was a workaround as flood used to
crash with only one node.

Also unpin the patch version of Python and address a small DRY on the
Vegeta version.
Add possibility to manipulate start/end block
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants