Closed Imraj-Singh closed 2 years ago
Hi @KrisThielemans, I've done this now and there were no changes. I did strip the notebook outputs by hand previously and wasn't aware of the nbstripout
, my apologies. Is this okay now? Thanks, Imraj
@KrisThielemans made the changes you suggested in our chat. Okay to merge?
Possibly you address a few of the Codacy issues. This link might work https://app.codacy.com/gh/SyneRBI/SIRF-Exercises/pullRequest?_ga=2.232672095.1548072879.1653313615-268825818.1653313615
a lot is trailing white space or docstring stuff. Addressing a few would be good, but not essential
I changed my trained network to a 5 iteration case, also implemented the changes suggested by you and codacy.
The main problem is the computational time needed to train. Perhaps worth giving participants a pre-trained model (or quasi-trained), and during the training session we can get them to run inference of the model.