Chainsync cleanup

[slundqui]

General cleanup of chainsync (that touches on a couple of other packages). Here's a rundown:

Major changes:

• Moving crash reporting and test from chainsync to agent0
  • This requires some adjustments, this crash report is using postgres data structures, where it should be using a more generic data structure
  • After discussion, removed dead code and referenced in crash-reporting: human-readable crash report on revert #479
• Changing chainsync imports from within a subpackage to be relative imports. This fixes circular import issues
  • Need to go through other packages to match this import structure
• Changing testing structure of chainsync to live close to the file it's testing, and renaming to be <file_to_be_tested>_test.py
  • elfpy still follows old testing structure, but the pytest config allows for both

Minor changes:

• Moving chainsync base/hyperdrive to an outer db directory
• Renaming postgres to db_interface
• Moving hyperdrive specific transactions to hyperdrive dir, and renaming to HyperdriveTransactions
• Moving get_hyperdrive_contract to ethpy
• Moving helper functions for dashboard from the bin script to analysis
• Adding checks to dashboard so it runs cleanly when there's no data
• Removing pytest.ini in favor of defining in pyproject.toml

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
elf-simulations	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 9, 2023 9:38pm

[github-actions]

Remaining rate limit: 4998 🚀

[github-actions]

Remaining rate limit: 4998 🚀

[github-actions]

Remaining rate limit: 4997 🚀

[github-actions]

Remaining rate limit: 4996 🚀

[github-actions]

Remaining rate limit: 5000 🚀

[github-actions]

Remaining rate limit: 4988 🚀

[github-actions]

Remaining rate limit: 4972 🚀

[github-actions]

Remaining rate limit: 4958 🚀

[github-actions]

Remaining rate limit: 5000 🚀

[github-actions]

Remaining rate limit: 4992 🚀

[github-actions]

Remaining rate limit: 4992 🚀

[github-actions]

Remaining rate limit: 4991 🚀

[dpaiton]

might as well remove the lib/chainsync/tests folder

[dpaiton]

What is the intended difference between scripts and bin?

[dpaiton]

I think it is better to remove the db_ prefix in front of the files in the db folders. e.g. chainsync/db/base/db_schema.py --> chainsync/db/base/schema.py. Same for the ones in the db/hyperdrive folder.

[slundqui]

What is the intended difference between scripts and bin?

bin are entrypoints to various services, e.g., from docker. scripts are one-offs, probably should be in jupytext format, and likely shouldn't really be tested or maintained, e.g., calculating previous leaderboards from trading competitions.

I don't know what the best way to keep something like scripts around, because for the time being, some of the functionality in those scripts can definitely be generalized once we get some time to look at it.

[dpaiton]

lgtm thanks for doing this & dealing with my many nits!