Uploaded image for project: 'Portal'
  1. Portal
  2. POR-2120

Event date is interpreted as January 1st when month or day omitted in the verbatim record

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      An occurrence record like [gbifid: 500544056, v_eventdate: 1979, eventdate : 01/01/79 00:00, v_year: 1979, year:1979, v_month:0, month:1, v_day:0, day:1] contains an incorrect interpretation of the eventdate field; the month and day have been interpreted as Jan 1st. The eventdate shouldn't be interpreted when there's an omission of year,month or day in the verbatim version of the record.

      Note: an attachment contains sample records that show the issue.

        Gliffy Diagrams

        Issue Links

          Activity

          Hide
          Oliver Meyn (Inactive) added a comment -

          More analysis reveals that approx 8.1M records have an eventDate of Jan 1, but approx 7.9M of those are Jan 1 in the verbatim record (given either as day and month or as part of the verbatim dateString). There is definitely a bug in our handling of incomplete dateStrings (i.e. year only or year-month only) but is probably only responsible for approx 200k of the Jan 1 spike.

          Show
          Oliver Meyn (Inactive) added a comment - More analysis reveals that approx 8.1M records have an eventDate of Jan 1, but approx 7.9M of those are Jan 1 in the verbatim record (given either as day and month or as part of the verbatim dateString). There is definitely a bug in our handling of incomplete dateStrings (i.e. year only or year-month only) but is probably only responsible for approx 200k of the Jan 1 spike.
          Hide
          Oliver Meyn (Inactive) added a comment -

          I've added tests to the TemporalInterpreterTest of occ project that show this problem. Fix is TODO.

          Show
          Oliver Meyn (Inactive) added a comment - I've added tests to the TemporalInterpreterTest of occ project that show this problem. Fix is TODO.

            People

            • Assignee:
              Christian Gendreau
              Reporter:
              Federico Mendez
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated: