After recent refactorings, the remaining top-level modules are only for the server processes, so move them all to a new sub-package server. Only the grid_search, hp_optimization and parallel_executor modules are still kept at top level for now, as they are called by the users via python -m cluster_utils.X (they should be handled in a later PR).
Move the cluster_utils and cluster packages to a sub-direcotry src/. This allows setuptools to easily auto-detect them, ridding us of the need to list all the (sub-)packages in pyproject.toml.
Further changes:
I remove the convenience imports of grid_search and hp_optimization functions in top-level __init__.py. They are not supposed to be called by the regular user directly, so I don't think they need to be there. Further, they were shadowing the modules of the same name, which can be confusing.
Let black simply check all Python files in stead of maintaining a list of files and directories.
This covers part of #45, but we may still want to add some further structure inside server (I just didn't want to do too many changes at once). Likewise the entrypoint scripts are left where they are for now (i.e. no change in the cli).
Moving modules around:
server
. Only thegrid_search
,hp_optimization
andparallel_executor
modules are still kept at top level for now, as they are called by the users viapython -m cluster_utils.X
(they should be handled in a later PR).cluster_utils
andcluster
packages to a sub-direcotrysrc/
. This allows setuptools to easily auto-detect them, ridding us of the need to list all the (sub-)packages in pyproject.toml.Further changes:
grid_search
andhp_optimization
functions in top-level__init__.py
. They are not supposed to be called by the regular user directly, so I don't think they need to be there. Further, they were shadowing the modules of the same name, which can be confusing.This covers part of #45, but we may still want to add some further structure inside
server
(I just didn't want to do too many changes at once). Likewise the entrypoint scripts are left where they are for now (i.e. no change in the cli).