Thomas Lazarus

Results 23 comments of Thomas Lazarus

I'll work on sklearn.utils.parallel.Parallel sklearn.utils.parallel.delayed

Hi! I've added an example of tabular regression in #254. Let me know if I did anything wrong!

@merveenoyan I think the `text-classification` task is already done in `/examples/plot_text_classification.py` unless I misunderstand the task? It looks like the `text-classification` example is still using `pickle` instead of `skops.io` so...

@adrinjalali Is this issue still something that needs to be done? I'm just asking since all the comments are from over a year ago. If so, just so I'm clear...

It looks like all the pytest tests are failing with `You have exceeded our daily quotas for action: createRepo. We invite you to retry later.`

@BenjaminBossan I've fixed the merge conflict. Yeah, I believe it would be but I was unsure if we wanted to have tests that included another library like that. Should I...

@BenjaminBossan Sorry this took me so long to get back to. I've added a test to hit that line.

@BenjaminBossan Yeah, sorry I realized I had marked the test method as a `@pytest.fixture` 🤦🏽, so the test wasn't ever running (which is why it wasn't erroring when switching from...