search

cytomining / profiling-recipe

Image-based Profiling Recipe
BSD 3-Clause "New" or "Revised" License
8 stars 26 forks source link

Switch LFS to DVC #25

Closed ErinWeisbart closed 2 years ago

ErinWeisbart commented 2 years ago

Update README.md and environment.yaml so that large file storage uses DVC not Git LFS.

ErinWeisbart commented 2 years ago

I believe this is good to go @shntnu. I could ask for the next person in the Cimini lab who uses the profiling recipe to follow these steps as a final test? (I haven't followed the instructions from start to finish on a new repo since making the few changes in this PR).

shntnu commented 2 years ago

@ErinWeisbart would you like to go ahead and merge?

ErinWeisbart commented 2 years ago

@shntnu I think it's fine to merge. As I mentioned, I haven't run through my instructions start to finish since all the final edits, but I suppose if someone finds something unclear/wrong with a run-through, they can always start a new issue/PR. (I don't have write access so you'll have to be the one to pull the trigger).

shntnu commented 2 years ago

Thanks, Erin!

I've made you a maintainer now FWIW