tidymodels / tune

Tools for tidy parameter tuning
https://tune.tidymodels.org
Other
276 stars 42 forks source link

`.gitignore` `revdep` #911

Open simonpcouch opened 4 months ago

simonpcouch commented 4 months ago

Tracking the revdep directory makes some global searches really tricky (or sometimes impossible for terms that occur a lot), especially when searching for contents in tests which appear after revdep. e.g.:

https://github.com/tidymodels/tune/assets/35748691/a0849627-e871-49f7-9f6e-47a25a9a3e0e

simonpcouch commented 4 months ago

Ah, looks like we do gitignore revdep but track a couple files in that directory still:

https://github.com/tidymodels/tune/blob/0ea505c1f35268bbba59f0b0695ae65340b781d1/.gitignore#L14

https://github.com/tidymodels/tune/blob/0ea505c1f35268bbba59f0b0695ae65340b781d1/revdep/README.md#L1

I guess the flow that I actually usually have is not to gitignore revdep so that I'm tempted to delete results as soon as I don't need them, but that risks accidentally pushing revdep. Hm.

topepo commented 4 months ago

That's probably what happened here.