-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Times out #2
Comments
@wongjustin99: can you tell me which command timed out? every command on applescript has a maximum execution time of 2 minutes, it's weird that the cli is exceeding it. |
I was trying reminders --list. Here's a fuller stack trace:
|
I just ran the dev version and added a 1-hour timeout instead and that got it to work. I think something about your get_reminders.applescript is timing out with a lot of reminders. |
@wongjustin99: hmm, how many reminders do you have? Did you see how long it takes to list all of them after you updating the timeout? |
Counting completed? Spread around 10 or so lists, 1000-1300. Not completed?
Less than 100.
Yeah, it took around 15 minutes, and it locked up the native Reminders
application as well. I think that loop may not be the most efficient.
Here's how I added that timeout:
```
with timeout of (30 * 60) seconds
tell application "Reminders"
set reminderList to {}
if (count of (reminders whose completed is false)) > 0 then
set reminderList to name of reminders whose completed is false
end if
quit
return reminderList
end tell
end timeout
```
…On 25 October 2017 at 02:57, Lucas Bento ***@***.***> wrote:
@wongjustin99 <https://github.com/wongjustin99>: hmm, how many reminders
do you have? Did you see how long it takes to list all of them after you
updating the timeout?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABgsDjQKF8JpTnJUTRtE9sjf-uJW5qxiks5svtvQgaJpZM4QDO5D>
.
|
Yeah, gotta improve the loop perf, I'll try to have a look on it once I have some free time. Thank you for reporting it, @wongjustin99! |
On a couple of computers, one El Capitan, one Sierra, I get an AppleScript timeout on both.
script:64:109: execution error: Reminders got an error: AppleEvent timed out. (-1712)
Is there a way to fix this or extend the timeout?
The text was updated successfully, but these errors were encountered: