Taylor Turner
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