Skip to content

Conversation

@davydog187
Copy link
Owner

No description provided.

[Documentation](https://docs.timescale.com/api/latest/hyperfunctions/gapfilling-interpolation/time_bucket_gapfill/)
"""
defmacro time_bucket_gapfill(field, time_bucket) do

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would love to see time_bucket_gapfill/2 get a time_bucket_gapfill/3 sibling that takes options in parity with time_bucket/3, as I have a use case of passing timezone to my fragments 🙏

Copy link

@vanderhoop vanderhoop May 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, turns out time_bucket/3 doesn't actually support the timescale implementation's timezone option. my use case calls for timezone to be passed as a positional argument to time_bucket_gapfill. gonna play around with a fragment macro and see if I can get it to blend locally

Copy link

@vanderhoop vanderhoop May 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, even with a working fragment, postgres barfs because my timestamp is a timestamp without timezone, which I'd forgotten about, and which has sent me down a rabbit hole of postgres/ecto timestamp storage/casting articles.

Disregard, though I may report back!

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vanderhoop now that I know you're looking for this, I'll try to get this cleaned up and merged in

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants