Calendar appears to be fetching the last calendar in the list multiple times.

sdetweil

@vvrangler but the events numbers match

sdetweil

@sdetweil looks like a timing bug… will submit a fix

https://github.com/MichMich/MagicMirror/issues/2160
fix
https://github.com/MichMich/MagicMirror/pull/2162

sdetweil

@vvrangler the fix i proposed has been accepted and is part of the develop branch, if you would like to use it

to get there

if you did manual install

cd ~/MagicMirror
git checkout develop
git pull

restart MagicMirror as normal

if you used my install script

cd ~/MagicMirror
git fetch origin develop:develop
git checkout develop

restart MagicMirror as normal

vvrangler

@sdetweil Yes, the number of events look to be good, and I’m not really seeing any issues because of this, it’s just something that I noticed in the logs.

sdetweil

@vvrangler yeh, it seems to luck out currently

the top code starts the fetcher and a starts a timer for each calendar entry
the fetcher ALSO starts a timer, so if its never called again, it still returns a refreshed list

I changed it so the top code starts the fetcher for each calendar entry
and let the fetcher handle its configured timer cycle

vvrangler

@sdetweil said in Calendar appears to be fetching the last calendar in the list multiple times.:

@vvrangler yeh, it seems to luck out currently

the top code starts the fetcher and a starts a timer for each calendar entry
the fetcher ALSO starts a timer, so if its never called again, it still returns a refreshed list

I changed it so the top code starts the fetcher for each calendar entry
and let the fetcher handle its configured timer cycle

The change definitely fixed the issue and everything looks good now during the subsequent polls, thank you again.

I also have a couple of other calendar related issues that I’ve seen in the log, would it be best if I closed this topic and post a new one?

sdetweil

@vvrangler said in Calendar appears to be fetching the last calendar in the list multiple times.:

I also have a cope of other calendar related issue that I’ve seen in the log, would it be best if I close this topic and posted a new one?

yes please