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
Assets azure #121
Assets azure #121
Conversation
f75729e
to
9970d96
Compare
Let's wait until https://github.com/Cornerstone-OnDemand/modelkit/pull/120/files is merged first so I can correctly update requirements |
Nice! |
Definitely, let me add this to the documentation! |
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.
Nice to see that more and more providers are available in modelkit!
A few remarks, but awesome as usual!
they are internally incorrect, and will lead to mypy errors see Azure/azure-sdk-for-python#20083
4dfef52
to
f9e4d61
Compare
PTAL @antoinejeannot |
4adb6de
to
d40f61a
Compare
|
In this PR I add support for assets stored in Azure blob storage. It's relatively simple, since I only need to correctly implement the driver in
assets/drivers/azure.py
.The rest follows: I add an assets manager fixture in
tests/assets/conftest.py
, and then duplicate a bunch of tests using the same approach as for other drivers.