OpenAdaptAI / OpenAdapt

AI-First Process Automation with Large ([Language (LLMs) / Action (LAMs) / Multimodal (LMMs)] / Visual Language (VLMs)) Models
https://www.OpenAdapt.AI
MIT License
735 stars 95 forks source link

fix:windows file permission #791

Closed Animesh404 closed 1 week ago

Animesh404 commented 1 week ago

What kind of change does this PR introduce?

a bug fix for windows file permission

Summary image

added a retry mechanism (max_retries and retry_delay_seconds) to handle cases where the file deletion fails due to the file being locked or accessed by another process. you can find do_fastsam function in openadapt/adapters/ultralytics.py

Checklist

How can your code be run and tested?

python -m openadapt.replay VisualReplayStrategy --instructions "<your instructions>"

abrichr commented 1 week ago

Looks like the build is failling (clck "Details"):

https://github.com/OpenAdaptAI/OpenAdapt/actions/runs/9628215324/job/26557412749?pr=791

image
Run poetry run black --preview --check . --exclude '/(alembic|\.cache|\.venv|venv|contrib|__pycache__)/'
would reformat /Users/runner/work/OpenAdapt/OpenAdapt/OpenAdapt/openadapt/adapters/ultralytics.py
would reformat /Users/runner/work/OpenAdapt/OpenAdapt/OpenAdapt-2024-06-22_22-43-54/adapters/ultralytics.py
Animesh404 commented 1 week ago

Looks like the build is failling (clck "Details"):

https://github.com/OpenAdaptAI/OpenAdapt/actions/runs/9628215324/job/26557412749?pr=791

image
Run poetry run black --preview --check . --exclude '/(alembic|\.cache|\.venv|venv|contrib|__pycache__)/'
would reformat /Users/runner/work/OpenAdapt/OpenAdapt/OpenAdapt/openadapt/adapters/ultralytics.py
would reformat /Users/runner/work/OpenAdapt/OpenAdapt/OpenAdapt-2024-06-22_22-43-54/adapters/ultralytics.py

@abrichr this 5bc685fedcdb7bc44a540d67e90a6f11152e0c40 should fix build issue 🙏