Skip to content

Conversation

@awirb
Copy link
Contributor

@awirb awirb commented Sep 9, 2022

No description provided.

@pep8speaks
Copy link

pep8speaks commented Sep 9, 2022

Hello @awirb! Thanks for updating this PR.

Line 489:19: E225 missing whitespace around operator
Line 405:42: E241 multiple spaces after ','

Comment last updated at 2022-09-09 10:37:25 UTC

@awirb awirb requested a review from fso42 September 9, 2022 07:45
@fso42 fso42 added the enhancement New feature or request label Sep 9, 2022
@fso42 fso42 added this to the Version 1.3 milestone Sep 9, 2022
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit f1bab9f and detected 7 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 3
Duplication 4

The test coverage on the diff in this pull request is 9.3% (50% is the threshold).

This pull request will bring the total coverage in the repository to 72.7% (0.0% change).

View more on Code Climate.

@codecov
Copy link

codecov bot commented Sep 9, 2022

Codecov Report

Merging #767 (f1bab9f) into master (c071f11) will decrease coverage by 0.17%.
The diff coverage is 9.37%.

❗ Current head f1bab9f differs from pull request most recent head f3de87c. Consider uploading reports for the commit f3de87c to get more accurate results

@@            Coverage Diff             @@
##           master     #767      +/-   ##
==========================================
- Coverage   74.97%   74.79%   -0.18%     
==========================================
  Files          57       57              
  Lines       12003    12035      +32     
==========================================
+ Hits         8999     9002       +3     
- Misses       3004     3033      +29     
Impacted Files Coverage Δ
avaframe/out3Plot/outAIMEC.py 85.23% <ø> (ø)
avaframe/out3Plot/statsPlots.py 21.39% <9.37%> (-1.83%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@fso42 fso42 merged commit c01c010 into master Sep 9, 2022
@fso42 fso42 deleted the addPlot branch September 9, 2022 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants