Skip to content

Conversation

@ZenGround0
Copy link
Collaborator

Not convinced we should commit this but here is a demo of how to interact with ethcalls to get a complete fault report. State viewer has a few basic methods which make checking this information simple.

This is a claude heavy PR so some mistakes are going to be in here and the writing is a little AI-y. But I've tested it out and it works pretty nicely.

@FilOzzy FilOzzy added this to FS Nov 5, 2025
@github-project-automation github-project-automation bot moved this to 📌 Triage in FS Nov 5, 2025
@ZenGround0
Copy link
Collaborator Author

@rvagg I'm considering putting something like this into filecoin-pin datasets command. Any thoughts?

Comment on lines +134 to +135
3. `getMaxProvingPeriod()` - Length of each proving period
4. `challengeWindow()` - Size of the challenge window
Copy link
Collaborator

Choose a reason for hiding this comment

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

these two can and should be getPDPConfig() where they are both available on the returned struct; we've just been discussing removing these methods because they should have been cleaned up when we made it a single call to get everything.

@rvagg
Copy link
Collaborator

rvagg commented Nov 5, 2025

Yeah, very nice. Would be good to have a nice filecoin-pin tool to do this, filecoin-pin data-set 123 --check-proofs 10 or something like that. If you can also look for opportunities to put nice utilities into Synapse for other devs to use while you're at it then that's good too. It could start out as just a separate function or two in filecoin-pin and then we lift it up. We need to do something similar with payments in there now.

@rjan90 rjan90 moved this from 📌 Triage to ⌨️ In Progress in FS Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ⌨️ In Progress

Development

Successfully merging this pull request may close these issues.

3 participants