Skip to content

Fixes/fix warnings#58

Merged
jose-moran merged 2 commits intomainfrom
fixes/fix_warnings
Feb 6, 2026
Merged

Fixes/fix warnings#58
jose-moran merged 2 commits intomainfrom
fixes/fix_warnings

Conversation

@jose-moran
Copy link
Member

This fix silences warnings.

When running the tests we were getting several warnings (more than 5k total I think). This was a mixture of two things:

  • sometimes, we had multiplications with infinities or divisions by 0. These are well handled by the code so I just added them to the ignore in pyproject.toml
  • Other warnings were about deprecation warnings (things that will break when we update to future versions of pandas or pydantic). These were fixed with the changes suggested by the warnings themselves.

Now tests pass without warnings.

@jose-moran jose-moran requested a review from t-youngman February 5, 2026 18:01
Copy link
Collaborator

@t-youngman t-youngman left a comment

Choose a reason for hiding this comment

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

Looks fine!

@jose-moran jose-moran merged commit 2d09db6 into main Feb 6, 2026
4 checks passed
@jose-moran jose-moran deleted the fixes/fix_warnings branch February 6, 2026 09:53
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