Skip to content

Commit

Permalink
ruff cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
kebwi committed Dec 16, 2024
1 parent 2ff9db6 commit a171590
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions caveclient/skeletonservice.py
Original file line number Diff line number Diff line change
Expand Up @@ -674,7 +674,7 @@ def generate_bulk_skeletons_async(
The name of the datastack to check
skeleton_version : int
The skeleton version to generate. Use 0 for Neuroglancer-compatibility. Use -1 for latest.
Returns
-------
float
Expand All @@ -701,7 +701,7 @@ def generate_bulk_skeletons_async(
f"The number of root_ids exceeds the current limit of {MAX_BULK_ASYNCHRONOUS_SKELETONS}. Only the first {MAX_BULK_ASYNCHRONOUS_SKELETONS} will be processed."
)
root_ids = root_ids[:MAX_BULK_ASYNCHRONOUS_SKELETONS]

# TODO: I recently converted this function to a batched approach to alleviate sending a long URL of root_ids via GET,
# but has since converted the call to POST, which probably obviates the need for the considerably more complex batch handling.
# So consider reverting to the unbatched approach in the future.
Expand Down Expand Up @@ -750,5 +750,7 @@ def generate_bulk_skeletons_async(
# else:
# estimate_time_str = f"{(estimated_async_time_secs_upper_bound_sum / 86400):.2f} days"

logging.info(f"Upper estimate to generate all {len(root_ids)} skeletons: {estimate_time_str}")
logging.info(

Check warning on line 753 in caveclient/skeletonservice.py

View check run for this annotation

Codecov / codecov/patch

caveclient/skeletonservice.py#L753

Added line #L753 was not covered by tests
f"Upper estimate to generate all {len(root_ids)} skeletons: {estimate_time_str}"
)
return estimated_async_time_secs_upper_bound_sum

Check warning on line 756 in caveclient/skeletonservice.py

View check run for this annotation

Codecov / codecov/patch

caveclient/skeletonservice.py#L756

Added line #L756 was not covered by tests

0 comments on commit a171590

Please sign in to comment.