Taylor Turner

Results 100 comments of Taylor Turner

If no additional thoughts regarding this, @pietz, I'll go ahead and close. Thx!

#1120 fixing but ultimately will need to resolve to not have late vintage version pins

Yep, we noticed this last week. `Requiring tensorflow==2.15.1 is a current workaround.`: yep, temporary fix is the current recommended workaround. Thanks for documenting in an issue @JGSweets 👍

Yeah, I think you are right -- after I saw the errors on the PR checks, looks like the model would need a version update itself

> Any update on this? Thanks! Not yet, @JGSweets. Thanks for the bump though -- haven't forgot about it

Thanks for your patience @gliptak -- we will review your proposal

@gliptak rebase onto `dev` and I'll approve

rebase or `update branch` on this and I'll approve @gliptak

@gliptak are you able to restore your `cramjam1` branch? This change fails tests locally on this command `DATAPROFILER_SEED=0 python3 -m unittest discover -p "test*.py"` First issues is ```dataprofiler/dataprofiler/tests/test_dp_logging.py", line 27,...

Checking out `f8b3e5dbd4b76f0ecc291911ace9e8e21cf1ecb1` and running tests on that commit (one commit prior to this PR) ensures that tests did run on `dev` locally at that point in the history @gliptak