Skip to content

feat(shed): lotus-shed miner fees-inspect improvements#13212

Merged
rjan90 merged 1 commit intomasterfrom
rvagg/fees-inspect-improve
Jul 15, 2025
Merged

feat(shed): lotus-shed miner fees-inspect improvements#13212
rjan90 merged 1 commit intomasterfrom
rvagg/fees-inspect-improve

Conversation

@rvagg
Copy link
Copy Markdown
Member

@rvagg rvagg commented Jul 15, 2025

  • Proper CSV output
  • Add --output to write to a file
  • Add resume capability from last processed epoch when using --output
  • --tipset only required if not resuming, now means "start"
  • --count is optional, will default to chain head
  • Always walk from oldest to newest so keeping a file updated is easier
  • Nice progress indicator when writing to a file
  • Improve efficiency comment for inspectMiner function

This does break the command for existing users, but since that's probably just me I think that's OK. The main breakage is in the ordering of the output and the fact that --tipset is that start and we go forward for --count or until head.

@rvagg rvagg requested a review from rjan90 July 15, 2025 11:05
@rvagg rvagg added the skip/changelog This change does not require CHANGELOG.md update label Jul 15, 2025
@github-project-automation github-project-automation bot moved this to 📌 Triage in FilOz Jul 15, 2025
- Proper CSV output
- Add --output to write to a file
- Add resume capability from last processed epoch when using --output
- --tipset only required if not resuming, now means "start"
- --count is optional, will default to chain head
- Always walk from oldest to newest so keeping a file updated is easier
- Nice progress indicator when writing to a file
- Improve efficiency comment for inspectMiner function
@rvagg rvagg force-pushed the rvagg/fees-inspect-improve branch from 65c282f to 67f19b1 Compare July 15, 2025 11:32
Copy link
Copy Markdown
Contributor

@rjan90 rjan90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does break the command for existing users, but since that's probably just me I think that's OK.

That is probably fine. Looks good to me as long as tests pass

@github-project-automation github-project-automation bot moved this from 📌 Triage to ✔️ Approved by reviewer in FilOz Jul 15, 2025
@rvagg
Copy link
Copy Markdown
Member Author

rvagg commented Jul 15, 2025

This is so much better than my previous workflow .. catch up in progress using an existing data file (which I've had to reformat a bit to make compatible):

Last processed epoch: 5058269
Resuming from epoch 5058270
Processing 84176 epochs from 5058270 to 5142445
Processing epoch 5058278 [443/84176] (0%)...

@rjan90 rjan90 merged commit af920e5 into master Jul 15, 2025
176 of 177 checks passed
@rjan90 rjan90 deleted the rvagg/fees-inspect-improve branch July 15, 2025 12:48
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip/changelog This change does not require CHANGELOG.md update

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

2 participants