CannyLab / vdtk

Visual Description Dataset Analysis Toolkit
MIT License
10 stars 2 forks source link

Fix tf in bleurt takes all the memory (--amend) #7

Closed xk-huang closed 1 year ago

xk-huang commented 1 year ago

Sorry that I accidentally missed the latter d in def :scream::scream::scream:. Here is the emergent fix patch.

DavidMChan commented 1 year ago

I just pushed a commit which fixes the typo. I should have looked closer when merging, so this is my bad. This PR looks like it is for #8, so we should merge it separately. I should probably fix the CI/CD as well, but as you mentioned, we're having trouble with the LFS assets, so I have to figure out how to get that sorted before the next few steps. I'll take care of this tomorrow, when it's not a holiday!