Skip to content
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

start working on dask compatible loo #2205

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

OriolAbril
Copy link
Member

@OriolAbril OriolAbril commented Feb 9, 2023

Description

Initial refactoring to get loo working with dask arrays

Checklist

  • Does the PR follow official PR format?
  • Does the PR include new or updated tests to cover the new feature (using pytest fixture pattern)?
  • Is the code style correct (follows pylint and black guidelines)?
  • Is the new feature listed in the New features section of the changelog?

📚 Documentation preview 📚: https://arviz--2205.org.readthedocs.build/en/2205/

@codecov
Copy link

codecov bot commented Apr 5, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (595fd7f) 86.70% compared to head (b565643) 86.70%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2205   +/-   ##
=======================================
  Coverage   86.70%   86.70%           
=======================================
  Files         122      122           
  Lines       12640    12640           
=======================================
  Hits        10959    10959           
  Misses       1681     1681           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cluhmann
Copy link

Hey @OriolAbril . Can you let me know how I can help test this? You mentioned that it probably isn't working in its current state, but I'm happy to see what I can do to help out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants