Fixed pip installation for submodules #9
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
Installing from git+https://github.com/braun-steven/simple-einet doesn't install the submodules "layers" and "layers.distribution" wouldn't add simple_einet.layers to the environment, upon checking the setup for package, it wasn't included in the setup file hence the sub-directories weren't bundled with the main package.
Solution
Add the submodules to the package list, if it was a .py file using setuptools we could also use setuptools.find_packages(), but for toml I believe we need to list all sub-modules to be in pip installable package
Changes
Included "simple_einet.layers", "simple_einet.layers.distributions" in the list of modules to be packaged