op-analytics icon indicating copy to clipboard operation
op-analytics copied to clipboard

Remove converted .py file and update ipynb so results are not written to txt (already going to BQ)

Open lithium323 opened this issue 1 year ago • 2 comments

Description

Remove converted .py file and update ipynb so results are not written to txt (already going to BQ). I believe the txt files are no longer being used anywhere else in the repo.

lithium323 avatar Sep 17 '24 21:09 lithium323

Hi, the use case for the .txt files is so we can read the latest gas prices in to google sheets for public chain parameter calculators.

  • Google Sheet | Docs Reference
  • Formula to call this txt file is =IMPORTDATA("https://static.optimism.io/op-analytics/reference_data/market_data/outputs/suggest_base_fee.txt")

Open to other ideas, but we followed this pattern, since FND also does it to create a circulating OP token supply endpoint (but that only updates monthly).

But to reduce commits, maybe we can either: 1 Make this update once per day or so, rather than every hour. or 2 Push things like this in to a separate repo, so that it doesn't clog things.

MSilb7 avatar Sep 18 '24 13:09 MSilb7

Hi, the use case for the .txt files is so we can read the latest gas prices in to google sheets for public chain parameter calculators.

  • Google Sheet | Docs Reference
  • Formula to call this txt file is =IMPORTDATA("https://static.optimism.io/op-analytics/reference_data/market_data/outputs/suggest_base_fee.txt")

Open to other ideas, but we followed this pattern, since FND also does it to create a circulating OP token supply endpoint (but that only updates monthly).

But to reduce commits, maybe we can either: 1 Make this update once per day or so, rather than every hour. or 2 Push things like this in to a separate repo, so that it doesn't clog things.

ah that's good to know! To make the dependency more explicit something we could do is have the github action update a specific cell in the destination spreadsheet. If that sounds good I can look into it.

lithium323 avatar Sep 18 '24 13:09 lithium323