BAMT icon indicating copy to clipboard operation
BAMT copied to clipboard

Integrational tests

Open PabloKarpacho opened this issue 2 years ago • 3 comments

Added 2 benchmarks and integrational tests with 4 scripts, covering all networks hyperparameters.

PabloKarpacho avatar Feb 15 '24 12:02 PabloKarpacho

Hello @PabloKarpacho! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 146:121: E501 line too long (156 > 120 characters) Line 187:121: E501 line too long (156 > 120 characters) Line 229:121: E501 line too long (156 > 120 characters) Line 242:24: E712 comparison to False should be 'if cond is False:' or 'if not cond:' Line 273:121: E501 line too long (160 > 120 characters)

Comment last updated at 2024-04-06 09:33:28 UTC

pep8speaks avatar Feb 15 '24 12:02 pep8speaks

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 71.09%. Comparing base (21fdd10) to head (88eae12). Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #105      +/-   ##
==========================================
+ Coverage   65.48%   71.09%   +5.61%     
==========================================
  Files          42       42              
  Lines        2903     2889      -14     
==========================================
+ Hits         1901     2054     +153     
+ Misses       1002      835     -167     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Feb 15 '24 13:02 codecov-commenter

Since we have a black code style in the repo, please do

pip install black
black tests/test_Integrational.py

jrzkaminski avatar Feb 16 '24 11:02 jrzkaminski