-
Notifications
You must be signed in to change notification settings - Fork 22
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
Unable to include 2 calendars #24
Comments
Unfortunately, the libraries I'm using to parse the calendars are very picky. If the validator shows errors, it's unlikely the calendars will work. However, you might try both the beta and non-beta versions and see if one works. If one of them works, please report back here, just for future reference. |
@franc6 yes I noticed that aswell while debugging. I was thinking of replacing that library with something more modern and battle tested, any objections? If not, then I'll try and clobber together and PR this weekend |
That’s the library used by the non-beta version. :) It has different bugs than the one in the beta (whose bugs are mostly in a third library for parsing recurring event rules), but the worst is that it’s unusable if HA is running on a small system like a raspberry pi — it uses too much ram and processor for any decently sized calendars (look at the other open issues).
If you know of other libraries, please let me know, and feel free to try them and let me know how they go. I have a few calendar files from other users that I regularly work with to check for problems, especially to try resolving performance issues that afflict the smaller systems.
… On Jan 9, 2021, at 5:00 AM, Joel Kåberg ***@***.***> wrote:
@franc6 yes I noticed that aswell while debugging.
I was thinking of replacing that library with something more modern and battle tested, any objections? If not, then I'll try and clobber together and PR this weekend
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
@jkaberg I just noticed the first error in your logs is a failure to download the URL. I tried and both URLs appear to work. Could it have been a temporary issue, or is there a firewall in your HA system that would prevent accessing those URLs? The second one seems to parse fine with the non-BETA version (exclusive of its bugs regarding recurring events). The first one doesn't have any events in it. I should be releasing v2.0 either today or early next week which will let you switch parsers on a per-calendar basis. |
@jkaberg Is this still a problem? |
@franc6 sorry for late reply, I'm not sure. What I did was let Google (calendar) proxy the calendars which works fine (orignal calendar->google calendar->ics_calendar/home assistant), so I sorta worked around it. I'll give it a go again and let you know. |
The first calendar won't download because of the user agent string, note the Forbidden in the first error message for it. The new user_agent option will work around that (I checked). |
Spoke too soon; setting the user_agent didn't resolve the problem for the first calendar. |
@jkaberg Were you using the user name and password features for any of your calendars? |
First, thanks for this awesome addon! 😄
I'm haveing some trouble adding the following calendars:
https://connect.garmin.com/modern/calendar/export/4ec9e47abf9447f19acd6a279823f3e7 results in:
and https://ukenr.no/ics/helligdager/no.ics results in:
However both these parses fine in Nextcloud? I did an reference check with the iCalendar validator, and indeed they show some errors. I have 4 other ics calendars that works just fine so something with these two. The fact that NC parses them fine suggest the exception handling might a bit to strict?
Any way we could work around this @franc6 ?
Tested with 2.0.0-beta7 and 1.0.6 - both show the same error in the logs
The text was updated successfully, but these errors were encountered: