keras-molecules icon indicating copy to clipboard operation
keras-molecules copied to clipboard

Top-level dir is not a module

Open dakoner opened this issue 8 years ago • 4 comments

I've been trying to package up keras-molecules as a module so I can run it on Google Cloud Machine Learning. Unfortunately, the top-level of keras-molecules (with train.py etc) isn't really module ready. The repo name has a "-" in it, and there's no top-level init.py.

dakoner avatar Nov 18 '16 21:11 dakoner

What did you end up doing for this? Intuitively it doesn't feel like making the top level a module is the right thing to do but maybe having an invokable module in the molecules dir rather than having the logic in a top level script, so we could do python -m molecules.train and have a shell script or something in the top level as an alias that passes arguments through.

maxhodak avatar Nov 29 '16 18:11 maxhodak

Can you point me to documentation for what Google Cloud Machine Learning expects?

maxhodak avatar Nov 29 '16 18:11 maxhodak

I just renamed the parent dir to keras_molecules, and added an init.py.

The way I usually like to structure this is:

toplevel/ toplevel/src/<all source code such as molecules/ goes here> toplevel/scripts/

and then no code goes in toplevel/, just README, LICENSE, setup.py.

On Tue, Nov 29, 2016 at 10:59 AM, Max Hodak [email protected] wrote:

What did you end up doing for this? Intuitively it doesn't feel like making the top level a module is the right thing to do but maybe having an invokable module in the molecules dir rather than having the logic in a top level script, so we could do python -m molecules.train and have a shell script or something in the top level as an alias that passes arguments through.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/maxhodak/keras-molecules/issues/40#issuecomment-263663988, or mute the thread https://github.com/notifications/unsubscribe-auth/AHtyQIdXymer-3F30EiqkEq0gF6IfirUks5rDHYDgaJpZM4K2724 .

dakoner avatar Nov 29 '16 19:11 dakoner

The docs for Cloud ML aren't great for custom jobs right now. Most of them are here: https://cloud.google.com/ml/docs/how-tos/preparing-models At the bottom it mentions setup.py.

I have a working example for Google Cloud Machine Learning. I could reformat it into a pull request if you want the recipe.

On Tue, Nov 29, 2016 at 10:59 AM, Max Hodak [email protected] wrote:

Can you point me to documentation for what Google Cloud Machine Learning expects?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/maxhodak/keras-molecules/issues/40#issuecomment-263664060, or mute the thread https://github.com/notifications/unsubscribe-auth/AHtyQOJrY-ZNIMipnvNl7KFUfkx4GQhiks5rDHYkgaJpZM4K2724 .

dakoner avatar Nov 29 '16 19:11 dakoner