Benjamin Bossan

I think we can consider this being solved by #203. The parsing of model cards is yet to come, but that was not part of the initial issue.

Let's close this for now after #212 has been merged.

@kamalabdul any updates?

Since there is no more reply, I consider this solved. Feel free to re-open if there are still issues.

> Would `my-app` break ? If yes, then what's the benefit of introducing the `skops` format apart from the ability of storing the metadata on the hub to make loading...

Yep

> Could you try the code from https://github.com/skops-dev/skops/pull/204 ? For me your test doesn't fail, but I haven't tested if the loaded object is actually valid. It's probably a bug...

@blaisethom Thanks for reporting. Please note that keras is not supported at the moment, so I wouldn't expect it to work.

> I'm not sure what's happening here, because I haven't touched `conftest.py`. My suspicion is that the line `from sklearn.utils import check_array` that I added to `_hf_hub.py` could be the...

@anferico I figured out the recursion error. Honestly, I'm a bit surprised that it didn't occur previously, but patching can be tricky. Please change the `pandas_not_installed` fixture like so: ```python...