Model Loading is now broken. about aestheta

[kkmcgg]

Perhaps we can use your branch to try and implement the sub-package structure while I roll back to the original setup in our main branch?

[donuty-party]

@kevinkmcguigan, this is because of the relative import (from .module import Class). You need to run the script as python -m Library.core instead of python Library/core.py. I want to break code out of core.py to prevent it from getting overwhelming. Do you of a way to do that without requiring relative imports?

By the way, this concerns my pull request, too. If you merge that, the same thing will happen.

[donuty-party]

@kevinkmcguigan Oh, never mind. Absolute imports work fine and don't require us to change how we run the script. I have fixed this in my branch.

[kkmcgg]

oh very good. We will more than likely merge branches shortly I just keep getting pulled away from a deep dive sorry about that.
I'm going to take this opportunity to try and set up the library as a proper python package as well - something I don't typically do myself- which should help with handling dependencies and such. l'll leave this open until after the merge

[donuty-party]

Fixed in 02a5267. This restores the original loadModel() and saveModel() functions, but in so doing, removes the model load by name functionality. I have moved that to the 82-load-model-by-name branch and reopened issue #82.

[kkmcgg]

Thanks for resolving this - we'll sort things out in your branch and move them over to the main.