Fix timespan subfield references in 'where' and 'order_by' arguments.

Description

Something is apparently going wrong with parsing or otherwise interpreting identifiers like "timespan.end" or "exposure.timespan.end" in where/order_by strings.

See https://lsstc.slack.com/archives/C01FBUGM2CV/p1676992334505879

Activity

Show:
Tim Jenness
July 14, 2023 at 4:10 PM

Thanks for sorting out this confusion.

Jim Bosch
July 14, 2023 at 3:40 PM

, here's a small review that's just documentation fixes and an attempt to make error messages more helpful:

https://github.com/lsst/daf_butler/pull/865

I'm fairly confident there was no actual missing functionality in the misadventures that spawned this ticket, but some of the CLI docs were definitely wrong.

Jim Bosch
June 27, 2023 at 2:54 PM

I've just tried to create a unit test that would reproduce the problem, and everything seemed to work.  I didn't find an exact how-to-reproduce in that Slack thread, but I did try both command-line and in-Python versions of queryDimensionRecords with order_by="timespan.end" and order_by="exposure.timespan.end".  Let's try to reproduce it again in a week when USDF is back up.

Jim Bosch
March 22, 2023 at 2:08 PM

Afraid there's been no progress yet; I've just been busy with other things.

Sreevani Jarugula
March 21, 2023 at 3:04 PM

Hello Jim,

Is this issue been tested? any help with the where cause is helpful. 

Done
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Labels

Reviewers

Tim Jenness

Story Points

RubinTeam

Components

Checklist

Created February 21, 2023 at 4:51 PM
Updated November 7, 2023 at 3:51 AM
Resolved July 14, 2023 at 6:00 PM