Skip to content

Commit

Permalink
Move new deps from configure.py to pyproject.toml (#55)
Browse files Browse the repository at this point in the history
The new dependencies added by cc88906 were in the old location,
`braingeneers.utils.configure.DEPENDENCIES`, so I wasn't able to load
any IOT functionality. Add those dependencies to the correct place in
`pyproject.toml`, and completely remove the confusing and redundant
`DEPENDENCIES` variable.
  • Loading branch information
atspaeth authored Nov 30, 2023
1 parent 18048a8 commit 86f55b6
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 55 deletions.
4 changes: 3 additions & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,9 @@ dependencies = [
'schedule',
'scipy>=1.10.0',
'tenacity',
"typing_extensions>=4.6; python_version<'3.11'"
"typing_extensions>=4.6; python_version<'3.11'",
'diskcache',
'pytz'
]

[tool.hatch.build.hooks.vcs]
Expand Down
54 changes: 0 additions & 54 deletions src/braingeneers/utils/configure.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,60 +13,6 @@
CURRENT_ENDPOINT = None
_open = None # a reference to the current smart_open function that is configured this should not be used directly

"""
Define all groups of dependencies here. `minimal` dependencies are always installed with
the braingeneerspy package, all other groups of dependencies will be installed
braingeneerspy[all] or braingeneerspy[specific_dependency_group]
Maintainers are encouraged to move dependencies to specific dependency groups and keep
`minimal` as small as possible. `minimal` would typically be used in a RaspberryPI
environment for example, where large dependencies can be very slow to build.
"""
DEPENDENCIES = {
# Minimal dependencies are always installed with a pip install of the repo
'minimal': [
'deprecated',
'requests',
'numpy',
'tenacity',
# 'sortedcontainers',
'boto3',
'joblib>=1.3.0,<2',
'braingeneers_smart_open', # 'smart_open>=5.1.0', the hash version fixes the bytes from-to range header issue.
'awswrangler==3.*',
],
'data': [
'h5py',
'pandas',
'nptyping',
'paho-mqtt',
],
# Specific dependency groups allow unnecessary (and often large) dependencies to be skipped
# add dependency groups here, changes will be dynamically added to setup(...)
'iot': [
# 'awsiotsdk==1.6.0', # dependency issues occur when the current version is installed, that may be resolvable
'redis',
'schedule',
'paho-mqtt',
'diskcache',
'pytz',
'tzlocal',
],
'analysis': [
'scipy>=1.10.0',
'pandas',
'powerlaw',
'matplotlib',
],
'ml': [
'torch',
'scikit-learn',
],
'hengenlab': [
'neuraltoolkit @ git+https://github.com/hengenlab/neuraltoolkit.git', # channel mapping information
],
}


def get_default_endpoint() -> str:
"""
Returns the current default (S3) endpoint. By default this will point to the standard
Expand Down

0 comments on commit 86f55b6

Please sign in to comment.