Skip to content

Update analysis utility functions - #67

Open
ezuetell wants to merge 7 commits into
ClimateImpactLab:mainfrom
ezuetell:main
Open

ezuetell wants to merge 7 commits into
ClimateImpactLab:mainfrom
ezuetell:main

Conversation

@ezuetell

Copy link
Copy Markdown
Contributor

Improvements to functionality, structure, and validation. May cause some backwards compatibility issues with existing analysis notebooks.

@codecov-commenter

codecov-commenter commented Sep 25, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.95%. Comparing base (ae9ce36) to head (2abb7bd).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #67   +/-   ##
=======================================
  Coverage   80.95%   80.95%           
=======================================
  Files           3        3           
  Lines          84       84           
=======================================
  Hits           68       68           
  Misses         16       16           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@brews brews left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is failing CI for a number of reasons. I'm guessing because the branch for this PR is using an out-of-date version of the dev dependencies. @ezuetell, could you merge or rebase from the latest version of main into your branch, then rerun ruff check and ruff format (or the appropriate just commands) to ensure they use the latest version?

Edit: Actually, I just looked at the git log/graph. Maybe you just need to run ruff format and ruff check to clean this up.

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@brews

brews commented Sep 28, 2026

Copy link
Copy Markdown
Member

That is looking better, @ezuetell. Thank you. If you can get the files you're changing as part of this PR checked/fixed/formated that's good enough for me.

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.

3 participants