Skip to content

Cleanup#11

Merged
someshsingh22 merged 4 commits intoAdversarial-Deep-Learning:mainfrom
Devanshu24:cleanup
Aug 7, 2021
Merged

Cleanup#11
someshsingh22 merged 4 commits intoAdversarial-Deep-Learning:mainfrom
Devanshu24:cleanup

Conversation

@Devanshu24
Copy link
Copy Markdown
Contributor

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 7, 2021

Codecov Report

Merging #11 (24a07a2) into main (09d01a5) will decrease coverage by 64.71%.
The diff coverage is 53.84%.

@@             Coverage Diff             @@
##             main      #11       +/-   ##
===========================================
- Coverage   96.96%   32.25%   -64.72%     
===========================================
  Files           7        7               
  Lines          33       93       +60     
===========================================
- Hits           32       30        -2     
- Misses          1       63       +62     
Impacted Files Coverage Δ
code_soup/ch5/mnist_gan.py 0.00% <0.00%> (ø)
code_soup/ch5/models/discriminator.py 100.00% <ø> (ø)
code_soup/ch5/models/generator.py 100.00% <ø> (ø)
code_soup/ch5/__init__.py 100.00% <100.00%> (ø)
code_soup/ch5/datasets/__init__.py 100.00% <100.00%> (ø)
code_soup/ch5/datasets/mnist.py 100.00% <100.00%> (ø)
code_soup/ch5/models/__init__.py 100.00% <100.00%> (ø)

@Devanshu24
Copy link
Copy Markdown
Contributor Author

The imports don't work as I've changed them to absolute imports, and some things broke.
Working on it.

@someshsingh22 someshsingh22 linked an issue Aug 7, 2021 that may be closed by this pull request
12 tasks
@Devanshu24
Copy link
Copy Markdown
Contributor Author

@someshsingh22 Have a look, coverage is failing but rest seems fine.
I'll remove the redundant python versions in a separate PR

Copy link
Copy Markdown
Member

@someshsingh22 someshsingh22 left a comment

Choose a reason for hiding this comment

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

Great, Coverage we can improve in the GAN PR

@someshsingh22 someshsingh22 linked an issue Aug 7, 2021 that may be closed by this pull request
@someshsingh22 someshsingh22 merged commit 9cc5627 into Adversarial-Deep-Learning:main Aug 7, 2021
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.

CodeCoverage Report Missing

2 participants