Skip to content

Commit

Permalink
cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
sdreyer committed Oct 18, 2024
1 parent de3c28c commit 044f6a3
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 6 deletions.
1 change: 0 additions & 1 deletion arcade/arcade/actor/core/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,6 @@ def register_toolkit(self, toolkit: Toolkit) -> None:
"""
self.catalog.add_toolkit(toolkit)
self._update_catalog_uuid()

Check warning on line 112 in arcade/arcade/actor/core/base.py

View check run for this annotation

Codecov / codecov/patch

arcade/arcade/actor/core/base.py#L112

Added line #L112 was not covered by tests
print(self.uuid)

async def call_tool(self, tool_request: ToolCallRequest) -> ToolCallResponse:
"""
Expand Down
2 changes: 1 addition & 1 deletion arcade/arcade/actor/core/common.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ async def call_tool(self, request: ToolCallRequest) -> ToolCallResponse:
@abstractmethod
async def tool_status(self, request: ToolStatusRequest) -> ToolStatusResponse:

Check warning on line 66 in arcade/arcade/actor/core/common.py

View check run for this annotation

Codecov / codecov/patch

arcade/arcade/actor/core/common.py#L65-L66

Added lines #L65 - L66 were not covered by tests
"""
Send a request to get the last time tools were updated
Send a request to get the last uuid of the toolkits
"""
pass

Check warning on line 70 in arcade/arcade/actor/core/common.py

View check run for this annotation

Codecov / codecov/patch

arcade/arcade/actor/core/common.py#L70

Added line #L70 was not covered by tests

Expand Down
2 changes: 0 additions & 2 deletions arcade/arcade/actor/core/components.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import time
from typing import Any

from opentelemetry import trace
Expand Down Expand Up @@ -56,7 +55,6 @@ async def __call__(self, request: RequestData) -> ToolCallResponse:
class ToolStatusComponent(ActorComponent):
def __init__(self, actor: Actor) -> None:
self.actor = actor

Check warning on line 57 in arcade/arcade/actor/core/components.py

View check run for this annotation

Codecov / codecov/patch

arcade/arcade/actor/core/components.py#L55-L57

Added lines #L55 - L57 were not covered by tests
self.last_update_time = time.time()

def register(self, router: Router) -> None:

Check warning on line 59 in arcade/arcade/actor/core/components.py

View check run for this annotation

Codecov / codecov/patch

arcade/arcade/actor/core/components.py#L59

Added line #L59 was not covered by tests
"""
Expand Down
4 changes: 2 additions & 2 deletions arcade/arcade/core/schema.py
Original file line number Diff line number Diff line change
Expand Up @@ -256,7 +256,7 @@ class ToolStatusRequest(BaseModel):
"""The request to check for updates for tools."""

uuid: str
"""The timestamp to compare the last update against"""
"""The UUID to compare against the current toolkits UUID"""


class ToolCallError(BaseModel):
Expand Down Expand Up @@ -330,4 +330,4 @@ class ToolStatusResponse(BaseModel):
"""The response to a status invocation."""

uuid: str
"""The timestamp when the tools were last updated."""
"""The current UUID of the registered toolkits."""

0 comments on commit 044f6a3

Please sign in to comment.