Skip to content

Commit a5e9e35

Browse files
committed
Fixed path logic in '_xml_file' function and '_grid_square_metadata_file'; added catch to _analyse for ValueError
1 parent 106a91f commit a5e9e35

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

src/murfey/client/analyser.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -302,6 +302,10 @@ def _analyse(self):
302302
f"Metadata gathering failed with a key error for key: {e.args[0]}"
303303
)
304304
raise e
305+
except ValueError as e:
306+
logger.error(
307+
f"Metadata gathering failed with a value error: {e}"
308+
)
305309
if not dc_metadata or not self._force_mdoc_metadata:
306310
self._unseen_xml.append(transferred_file)
307311
else:
@@ -406,8 +410,8 @@ def _xml_file(self, data_file: Path) -> Path:
406410
data_directories = self._murfey_config.get("data_directories", [])
407411
for dd in data_directories:
408412
if str(data_file).startswith(dd):
409-
base_dir = Path(dd)
410-
mid_dir = data_file.relative_to(dd).parent
413+
base_dir = Path(dd).absolute()
414+
mid_dir = data_file.relative_to(base_dir).parent
411415
break
412416
else:
413417
return data_file.with_suffix(".xml")

src/murfey/client/contexts/spa.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,8 @@ def _grid_square_metadata_file(
6161
) -> Path:
6262
for dd in data_directories:
6363
if str(f).startswith(str(dd)):
64-
base_dir = dd
65-
mid_dir = f.relative_to(dd).parent
64+
base_dir = dd.absolute()
65+
mid_dir = f.relative_to(base_dir).parent
6666
break
6767
else:
6868
raise ValueError(f"Could not determine grid square metadata path for {f}")

0 commit comments

Comments
 (0)