-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add basic deal stats api server for spacerace slingshot #3963
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OMG @whyrusleeping you have totally made my day. Thank you for this one!!!!
var filteredClients map[address.Address]bool | ||
|
||
func init() { | ||
filteredClients = make(map[address.Address]bool) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a way we can filter out miners as well? E.g. there are terabytes of data from Travis' miners that were stored at genesis on Space Race net, but this isn't legit usage data.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, i'm only looking at deals on chain. Do the genesis miners have deals for their data?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I believe they do have deals on chain!
cmd/lotus-shed/dealtracker.go
Outdated
|
||
func init() { | ||
filteredClients = make(map[address.Address]bool) | ||
for _, a := range []string{"t0112", "t0113", "t0114", "t010089"} { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And is there some way that we can make this configurable? E.g. if we want to add more addresses to filter out of the search down the road?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, I can make it into an environment variable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome!
🤯 This will totally work, and obviates need for chainwatch entirely. Thank you @whyrusleeping, you totally saved the day! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(pushed the env var thing)
No description provided.