Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release q42021 #121

Merged
merged 42 commits into from
Jan 28, 2022
Merged

Release q42021 #121

merged 42 commits into from
Jan 28, 2022

Conversation

rportilla-databricks
Copy link
Contributor

-- Added FFT (already reviewed)
-- Added new release for pypi
-- Added interpolation (already reviewed)

rportilla-databricks and others added 30 commits August 10, 2021 12:19
* fourier transformation functionality in tempo

* fourier transform method docstrings added

* fourier transform unit test added

* updating readme with the fourier transform usage and the fourier function's variable naming pattern standard

* Updating requirements

* minor logic correction of naming the data column as 'val'

* adding the corrected buildTestDF and also adding pyarrow in requirements.txt

* Fourier unit test fixed and contributions information updated

* data column in tests and logic is corrected with the name changed to tdval

* original contribution restoration

* bringing the pandas_udf method inside the calling method to ensure the reference is not lost in the executors

* committing the correct timestep variable position

* adding self to timestep

* inherit timestep directly from parameter

* tidying up the codebase

* removed the set_timestep method as it is no longer required

* removing the unnecessary orderby

* adding order by inside the pandas function

* removed the redundant imports
guanjieshen and others added 2 commits January 20, 2022 16:22
* feat: add interpolation

* feat(interpolation): add support for multuple partitions, and target columns

* test: add interpolation zero fill test

* test: add additional interpolation tests

* chore: convert linear interpolation to use spark native functions

* chore: allow for interpolation to be called directly from the TSDF object

* chore: update series fill logic

* chore: change default behaviour for target_cols

* chore: rename to be more consistent with pandas and the tsdf class

* chore(interpolation): make show if interpolated column optional

* chore(interpolation): remove caching

* Troubleshooting (#2)

* Refactor interpolation code to remove joins, and double` resample`
* Added additional test coverage to interpolation code
* Updated `test` folder structure

Co-authored-by: Guanjie Shen <guanjie.shen@databricks.com>

* chore: add additional comments

* chore: update branches in test.yml

* fix: update interpolate_column params

* chore: add interpolation details in readme.md

* chore: update main readme.md

* chore: update main readme.md

* Merge branch 'master' of github.com:guanjieshen/tempo

* chore: make readme more consistent

* chore: add build and downloads badge to readme

* changes

* fix: fourier test java error

* fix: try to configure netty changes so tests for fourier will work

* change

* housekeeping: organize imports on tsdf.py

* chore(interpolation): change back to bfill, change forward to ffill

* interpolation: add the ability to call interpolate after resample

* housekeeping: add missing type hint

* chore(interpolate): update readme

* chore: update interpolation documentation to be more clear

* adding one unit test

Co-authored-by: Guanjie Shen <guanjie.shen@databricks.com>
Co-authored-by: Ricardo Portilla <ricardo.portilla@databricks.com>
@lgtm-com
Copy link

lgtm-com bot commented Jan 26, 2022

This pull request introduces 2 alerts when merging ddb8135 into 2161b60 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for Variable defined multiple times

Copy link
Contributor

@guanjieshen guanjieshen left a comment

Choose a reason for hiding this comment

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

I noticed, there are a lot new metadata files added from scala/project/target/streams and from scala/project/target/scala-2.12/sbt-1.0/update/update_cache_2.12/output are these needed?

Otherwise LGTM!

@lgtm-com
Copy link

lgtm-com bot commented Jan 27, 2022

This pull request introduces 2 alerts when merging fbb5519 into 2161b60 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for Variable defined multiple times

@lgtm-com
Copy link

lgtm-com bot commented Jan 27, 2022

This pull request introduces 2 alerts when merging 784e1d8 into 2161b60 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for Variable defined multiple times

@lgtm-com
Copy link

lgtm-com bot commented Jan 28, 2022

This pull request introduces 2 alerts when merging 298886c into 2161b60 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for Variable defined multiple times

@lgtm-com
Copy link

lgtm-com bot commented Jan 28, 2022

This pull request introduces 1 alert when merging f5d92d9 into 2161b60 - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@rportilla-databricks rportilla-databricks merged commit f20d73f into master Jan 28, 2022
Sonali-guleria added a commit to Sonali-guleria/tempo that referenced this pull request Jan 28, 2022
@rportilla-databricks rportilla-databricks deleted the release_Q42021 branch February 1, 2022 19:23
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.

4 participants