Skip to content

Woodwork version 0.16 fixes#2030

Merged
thehomebrewnerd merged 17 commits intomainfrom
ww-0.16-fixes
Apr 26, 2022
Merged

Woodwork version 0.16 fixes#2030
thehomebrewnerd merged 17 commits intomainfrom
ww-0.16-fixes

Conversation

@thehomebrewnerd
Copy link
Contributor

Updates for compatibility for WW version 0.16

  • bump min WW requirement to 0.16.1
  • Update serialization to specify filename when deserializing from parquet
  • Fix test in test_transform_features.py

@rwedge
Copy link
Contributor

rwedge commented Apr 25, 2022

Docs error is probably due to woodwork needing python-dateutil 2.8.1 or higher and autonormalize pinning it below 2.8.1

I can do a release of autonormalize to fix

@codecov
Copy link

codecov bot commented Apr 25, 2022

Codecov Report

Merging #2030 (e59dfac) into main (6e81933) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2030   +/-   ##
=======================================
  Coverage   99.09%   99.09%           
=======================================
  Files         143      143           
  Lines       16505    16511    +6     
=======================================
+ Hits        16355    16361    +6     
  Misses        150      150           
Impacted Files Coverage Δ
featuretools/entityset/deserialize.py 100.00% <100.00%> (ø)
featuretools/entityset/serialize.py 100.00% <100.00%> (ø)
...etools/tests/entityset_tests/test_serialization.py 99.67% <100.00%> (ø)
...ests/primitive_tests/test_feature_serialization.py 99.36% <100.00%> (ø)
...s/tests/primitive_tests/test_transform_features.py 99.85% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6e81933...e59dfac. Read the comment docs.

@thehomebrewnerd thehomebrewnerd marked this pull request as draft April 25, 2022 18:58
@thehomebrewnerd thehomebrewnerd marked this pull request as ready for review April 26, 2022 12:16
@thehomebrewnerd thehomebrewnerd requested a review from rwedge April 26, 2022 12:18
@thehomebrewnerd thehomebrewnerd requested a review from rwedge April 26, 2022 15:52
Copy link
Contributor

@rwedge rwedge left a comment

Choose a reason for hiding this comment

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

LGTM

@thehomebrewnerd thehomebrewnerd enabled auto-merge (squash) April 26, 2022 16:07
@thehomebrewnerd thehomebrewnerd merged commit 15e40b9 into main Apr 26, 2022
@thehomebrewnerd thehomebrewnerd deleted the ww-0.16-fixes branch April 26, 2022 16:10
@thehomebrewnerd thehomebrewnerd mentioned this pull request Apr 27, 2022
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