Skip to content

Commit 73d6fba

Browse files
committed
fix: repair an issue with the use of "name" as a parameterized field
1 parent b250180 commit 73d6fba

File tree

4 files changed

+3326
-21
lines changed

4 files changed

+3326
-21
lines changed

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
FROM python:3.11-slim
1+
FROM python:3.10-slim
22

33
WORKDIR /app
44

src/aind_metadata_viz/database.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -334,8 +334,6 @@ def _get_metadata(test_mode=False) -> pd.DataFrame:
334334

335335
records.append(info_data)
336336

337-
print(records)
338-
339337
return pd.DataFrame(
340338
records,
341339
columns=[

src/aind_metadata_viz/view.py

Lines changed: 11 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
"""App for viewing individual metadata assets"""
22

3-
import os
4-
3+
from typing import Optional
54
import panel as pn
65
import param
76

@@ -36,32 +35,23 @@
3635
)
3736

3837

39-
# State sync
40-
class Settings(param.Parameterized):
41-
"""Top-level settings for QC app"""
42-
43-
name = param.String(default="")
44-
45-
46-
settings = Settings()
47-
pn.state.location.sync(settings, {"name": "name"})
48-
49-
50-
def get_record(name):
38+
def get_record(asset_name: str) -> Optional[dict]:
5139
"""Get a record from the database by name"""
5240
records = docdb_api_client.retrieve_docdb_records(
53-
filter_query={"name": name},
41+
filter_query={"name": asset_name},
5442
limit=1,
5543
)
5644

5745
if len(records) == 0:
46+
print(f"Record with name {asset_name} not found.")
5847
return None
5948
return records[0]
6049

6150

6251
class MetadataView(param.Parameterized):
6352
"""Class for viewing metadata records"""
6453

54+
asset_name = param.String(default="")
6555
record = param.Dict(default=None)
6656
files_present = param.List(default=[])
6757
describedBys = param.Dict(default={})
@@ -203,15 +193,18 @@ def panel(self):
203193

204194

205195
metadata_view = MetadataView()
206-
metadata_view.set_record(get_record(settings.name))
196+
197+
pn.state.location.sync(metadata_view, {"asset_name": "name"})
198+
199+
metadata_view.set_record(get_record(metadata_view.asset_name))
207200
metadata_view_pane = metadata_view.panel()
208201

209202
print(f"AIND Metadata Viz version: {__version__}")
210-
print(f"Viewing record: {settings.name}")
203+
print(f"Viewing record: {metadata_view.asset_name}")
211204

212205
main_row = pn.Row(
213206
pn.HSpacer(),
214207
metadata_view_pane,
215208
pn.HSpacer(),
216209
)
217-
main_row.servable(title="Metadata View")
210+
main_row.servable(title="Metadata View")

0 commit comments

Comments
 (0)