-
Notifications
You must be signed in to change notification settings - Fork 8
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
feat: use lassie for retrievals #149
Conversation
Codecov ReportBase: 20.90% // Head: 5.70% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #149 +/- ##
==========================================
- Coverage 20.90% 5.70% -15.21%
==========================================
Files 16 10 -6
Lines 2334 1490 -844
==========================================
- Hits 488 85 -403
+ Misses 1816 1401 -415
+ Partials 30 4 -26
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@rvagg @kylehuntsman 's PR is merged can we move this forward? Any reason not to? |
application-research/filclient#112 is the blocker, and it's blocked by having a tagged boost—we could just use an untagged master over there but I don't think we need to rush this, I'm happy to keep this branch updated on both ends for now |
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.
looks great! dropping-in the current filclient on the retriever interface is a very clean transition to get started. if it's working, you have my blessing!
Not intended to be merged (yet anyway, we can have that conversation later). This is for dev purposes to make sure we get interfaces right and give me something functional so I can actually use code over in lassie for something real.
Depends on: