search

databrickslabs / tempo

API for manipulating time series on top of Apache Spark: lagged time values, rolling statistics (mean, avg, sum, count, etc), AS OF joins, downsampling, and interpolation
https://pypi.org/project/dbl-tempo
Other
310 stars 53 forks source link

Fix list-to-set syntax #338

Closed tnixon closed 1 year ago

tnixon commented 1 year ago

Turns out we introduced a bug in #337.

You can't convert a list to a set with the curly-brace set literal syntax. This is correct for defining a set literal: {1, 2, 3, 4} But this is not correct for converting a list to a set: {[1, 2, 3, 4]} Instead you must explicitly use the set function: set([1, 2, 3, 4])

codecov[bot] commented 1 year ago

Codecov Report

Merging #338 (ee1d1f7) into master (5198312) will decrease coverage by 0.11%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #338      +/-   ##
==========================================
- Coverage   87.83%   87.72%   -0.11%     
==========================================
  Files           6        6              
  Lines         871      872       +1     
  Branches      176      177       +1     
==========================================
  Hits          765      765              
  Misses         81       81              
- Partials       25       26       +1

see 1 file with indirect coverage changes