Skip to content

Commit 974f581

Browse files
changed how things are laid out to work with pypi installation
1 parent b55cdb7 commit 974f581

5 files changed

Lines changed: 15 additions & 11 deletions

File tree

setup.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,12 @@
1111
author="Science Team",
1212
author_email="team@science.xyz",
1313
packages=find_packages(include=["synapse", "synapse.*"]),
14+
package_data={
15+
"synapse": [
16+
"utils/model_converter/docker/Dockerfile",
17+
"utils/model_converter/docker/convert.py",
18+
],
19+
},
1420
long_description=long_description,
1521
long_description_content_type="text/markdown",
1622
python_requires=">=3.9",

synapse/client/sftp.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ def connect_sftp(hostname, username, password=None, pass_filename=None, key_file
2929
logging.error(f"Failed to read password file: {e}")
3030
return None, None
3131
try:
32-
logging.debug(f"Connecting to {hostname}:{port} as {username}")
3332
ssh.connect(
3433
hostname=hostname,
3534
port=port,

model-converter/Dockerfile renamed to synapse/utils/model_converter/docker/Dockerfile

File renamed without changes.

model-converter/convert.py renamed to synapse/utils/model_converter/docker/convert.py

File renamed without changes.

synapse/utils/model_converter/onnx_to_dlc.py

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -18,18 +18,17 @@
1818

1919

2020
def _find_model_converter_dir() -> str:
21-
"""Locate the model-converter/ directory containing the Dockerfile."""
22-
# Walk up from this file to the repo root
21+
"""Locate the directory containing the Dockerfile for model conversion."""
2322
here = os.path.dirname(os.path.abspath(__file__))
24-
repo_root = os.path.dirname(os.path.dirname(os.path.dirname(here)))
25-
candidate = os.path.join(repo_root, "model-converter")
26-
if os.path.isdir(candidate) and os.path.isfile(
27-
os.path.join(candidate, "Dockerfile")
23+
docker_dir = os.path.join(here, "docker")
24+
if os.path.isdir(docker_dir) and os.path.isfile(
25+
os.path.join(docker_dir, "Dockerfile")
2826
):
29-
return candidate
27+
return docker_dir
28+
3029
raise FileNotFoundError(
31-
f"model-converter/ directory not found at {candidate}. "
32-
"Make sure you are running from the synapse-python repository."
30+
"Model converter Dockerfile not found. "
31+
"Try reinstalling: pip install science-synapse"
3332
)
3433

3534

@@ -72,7 +71,7 @@ def _build_image(console: Optional[Console] = None) -> bool:
7271
return False
7372

7473
if console:
75-
console.print(f"[green]Docker image {DOCKER_IMAGE} built successfully[/green]")
74+
console.print(f"[green]Docker image built successfully[/green]")
7675
return True
7776

7877

0 commit comments

Comments
 (0)