ChunkedHashingVectorizer is not compatible with sklearn 0.19.0
Closed this issue · 2 comments
ebolyen commented
This testing log shows the issue. It looks like a parameter was removed (or perhaps it became keyword only?). In any case, a super().transform method has a different signature now.
thermokarst commented
Hey @BenKaehler, do you have time to look at this for this release (2017.8) or the next release (2017.9)? If we tried to get this in for 2017.8, a PR would need to get in this week some time (we are shooting for a release on August 30th). No worries if you don't have the bandwidth to look into this right now, please just let us know either way. Thanks!
BenKaehler commented
Thanks Matt, I'll try to get this done this week.
… On 22 Aug 2017, at 8:20 am, Matthew Dillon ***@***.***> wrote:
Hey @BenKaehler, do you have time to look at this for this release (2017.8) or the next release (2017.9)? If we tried to get this in for 2017.8, a PR would need to get in this week some time (we are shooting for a release on August 30th). No worries if you don't have the bandwidth to look into this right now, please just let us know either way. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.