Building-ML-Pipelines / building-machine-learning-pipelines

Code repository for the O'Reilly publication "Building Machine Learning Pipelines" by Hannes Hapke & Catherine Nelson
MIT License
584 stars 250 forks source link

Match tfx version in notebook with the one pinned in requirements.txt #10

Closed BioGeek closed 4 years ago

BioGeek commented 4 years ago

In requirements/requirements.txt the tfx version is pinned to version 0.22.0, but in the notebook, the version is pinned to 0.21.4.

hanneshapke commented 4 years ago

Hi @BioGeek, Thank you for your PR. Can you please confirm that the notebook worked on your end with the 0.22.0 version of TFX? When I upgraded my env and reran the notebook, I noticed that the SchemaGen component failed. When I tried to reproduce it in a different env, it worked fine.

@drcat101 Are you ok with upgrading the version?

hanneshapke commented 4 years ago

Thank you @BioGeek for your PR.