Open petermeissner opened 5 years ago
Good question!
There seems to be multiple things going on...
DTSTART;TZID=Greenwich Standard Time
"# data
u = "https://github.com/ATFutures/calendar/releases/download/0.0.1/d-mini.ics"
download.file(url = u, destfile = "d-mini.ics")
# calendars
c1 = calendar::ic_read("d-mini.ics")
c2 = ical::ical_parse_df("d-mini.ics")
# comparison
names(c1)
#> [1] "DESCRIPTION"
#> [2] "UID"
#> [3] "SUMMARY"
#> [4] "DTSTART;TZID=GMT Standard Time"
#> [5] "DTEND;TZID=GMT Standard Time"
#> [6] "CLASS"
#> [7] "PRIORITY"
#> [8] "DTSTAMP"
#> [9] "TRANSP"
#> [10] "STATUS"
#> [11] "SEQUENCE"
#> [12] "LOCATION"
#> [13] "X-MICROSOFT-CDO-APPT-SEQUENCE"
#> [14] "X-MICROSOFT-CDO-BUSYSTATUS"
#> [15] "X-MICROSOFT-CDO-INTENDEDSTATUS"
#> [16] "X-MICROSOFT-CDO-ALLDAYEVENT"
#> [17] "X-MICROSOFT-CDO-IMPORTANCE"
#> [18] "X-MICROSOFT-CDO-INSTTYPE"
#> [19] "X-MICROSOFT-DONOTFORWARDMEETING"
#> [20] "X-MICROSOFT-DISALLOW-COUNTER"
#> [21] "DTSTART;TZID=Greenwich Standard Time"
#> [22] "DTEND;TZID=Greenwich Standard Time"
#> [23] "DTSTART;VALUE=DATE"
#> [24] "DTEND;VALUE=DATE"
#> [25] "DTSTART;TZID=Romance Standard Time"
#> [26] "DTEND;TZID=Romance Standard Time"
names(c2)
#> [1] "uid" "summary" "start" "end"
#> [5] "description" "location" "class" "priority"
#> [9] "transp" "dtstamp" "last_modified" "status"
Created on 2019-09-10 by the reprex package (v0.3.0)
I browsed through my code ...
ical_parse_df()
still does only parse specific columns, namely those that are shown in the example above.
... but ...
There is now a function called ical_parse_jcal()
which parses the calendar into so called jcal (Javascript representation of ical data) which basically is a stupid very nested and long and ugly list of lists of lists .... that entails all the data, including meta data.
Now this format has to be made useful --> v0.3.0
Hi @petermeissner, fantastic work and I understand the logic: best start conservative, exporting only key variables by default. Still I think enabling users to decide which columns they can see will be very useful.
Confirmed, the datetime issue is now fixed. Interesting that
ical
generates fewer columns:Created on 2019-09-09 by the reprex package (v0.3.0)
Originally posted by @Robinlovelace in https://github.com/petermeissner/ical/issues/2#issuecomment-529361734