Skip to content

Conversation

@dweindl
Copy link
Member

@dweindl dweindl commented Jul 18, 2023

No description provided.

dweindl and others added 3 commits July 17, 2023 16:26
…215)



Only import plotting functions in `petab.visualize` if matplotlib is installed.

Closes #212



Co-authored-by: Dilan Pathirana <59329744+dilpath@users.noreply.github.com>
@dweindl dweindl requested a review from dilpath July 18, 2023 05:48
@dweindl dweindl requested review from a team and plakrisenko as code owners July 18, 2023 05:48
Copy link
Member

@dilpath dilpath left a comment

Choose a reason for hiding this comment

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

👍

@codecov-commenter
Copy link

codecov-commenter commented Jul 18, 2023

Codecov Report

Merging #216 (5c052f1) into main (058bf1a) will decrease coverage by 0.01%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##             main     #216      +/-   ##
==========================================
- Coverage   76.32%   76.32%   -0.01%     
==========================================
  Files          34       34              
  Lines        3168     3172       +4     
  Branches      768      769       +1     
==========================================
+ Hits         2418     2421       +3     
  Misses        552      552              
- Partials      198      199       +1     
Impacted Files Coverage Δ
petab/visualize/__init__.py 88.88% <85.71%> (-11.12%) ⬇️
petab/version.py 100.00% <100.00%> (ø)
petab/visualize/lint.py 51.42% <100.00%> (ø)

@dweindl dweindl merged commit 1aafa36 into main Jul 18, 2023
@dweindl dweindl deleted the release_0.2.3 branch July 18, 2023 06:41
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.

4 participants