arfc / annsa

Artificial neural networks for spectral analysis.
BSD 3-Clause "New" or "Revised" License
12 stars 4 forks source link

updated CNN and DNN unit tests #80

Closed kamuda1 closed 5 years ago

kamuda1 commented 5 years ago

This PR updates the CNN and DNN unit tests for the Tensorflow classes in the model_classes.py module. Tests use fixtures to create models with dense connections and convolutional filters of all ones. Training tests and tests for the CAE and DAE to come.

Tests include checking if:

pep8speaks commented 5 years ago

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

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2019-09-18 23:33:30 UTC
samgdotson commented 5 years ago

Everything looks good, merging PR