Skip to content

Commit

Permalink
Small tune ups from code review
Browse files Browse the repository at this point in the history
Co-authored-by: John T. Wodder II <[email protected]>
  • Loading branch information
yarikoptic and jwodder authored Aug 20, 2024
1 parent c45ffe2 commit c47269b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions dandi/organize.py
Original file line number Diff line number Diff line change
Expand Up @@ -308,7 +308,7 @@ def _assign_obj_id(metadata, non_unique):
# Avoid heavy import by importing within function:
from .pynwb_utils import get_object_id

msg = "%d out of %d paths are not unique." % (len(non_unique), len(metadata))
msg = f"{len(non_unique)} out of {len(metadata)} paths are not unique."
non_unique_paths = sorted(non_unique)

# provide more information to the user
Expand All @@ -329,7 +329,7 @@ def get_msg(path, indent=" "):
else:
for ex_path in non_unique_paths[1:]:
msg += "\n " + get_msg(ex_path)

Check warning on line 331 in dandi/organize.py

View check run for this annotation

Codecov / codecov/patch

dandi/organize.py#L330-L331

Added lines #L330 - L331 were not covered by tests
lgr.info(msg + "\n We will try adding _obj- based on crc32 of object_id")
lgr.info("%s\n We will try adding _obj- based on crc32 of object_id", msg)
seen_obj_ids = {} # obj_id: object_id
seen_object_ids = {} # object_id: path
recent_nwb_msg = "NWB>=2.1.0 standard (supported by pynwb>=1.1.0)."
Expand Down

0 comments on commit c47269b

Please sign in to comment.