Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lfs => dvc #64

Merged
merged 1 commit into from
Mar 9, 2022
Merged

lfs => dvc #64

merged 1 commit into from
Mar 9, 2022

Conversation

ErinWeisbart
Copy link
Member

Updates the handbook to reflect recent changes to the profiling-recipe wherein we switch from the use of Git LFS to DVC for large file management.

@shntnu
Copy link
Member

shntnu commented Mar 8, 2022

I can fix this build error, but I wonder if this is the right time to do the thing @bethac07 suggested below.

Over to you two to decide @ErinWeisbart @bethac07


From: Beth Cimini bcimini@broadinstitute.org
Date: Tue, Nov 30, 2021 at 6:49 AM
To: Shantanu Singh shsingh@broadinstitute.org

FWIW, I don't think the "lift" is currently probably worth it, but if we did want to switch to JupyterBook somewhere down the road, it looks like it takes Rmd pages so it would be a fast start-up.

https://jupyterbook.org/file-types/jupytext.html?highlight=rmd


From: Shantanu Singh shsingh@broadinstitute.org
Date: Tue, Nov 30, 2021 at 6:56 AM
To: Beth Cimini bcimini@broadinstitute.org

Neat! The conversion would be a nice project (~2-3 days?) for someone who is interested in creating training material and learning about GitHub Actions.


From: Beth Cimini bcimini@broadinstitute.org
Date: Tue, Nov 30, 2021 at 7:11 AM
To: Shantanu Singh shsingh@broadinstitute.org

Since it can read Rmd files directly, and GH actions is already documented, assuming we stuck with the Rmd files we have in theory it's a 30 minute project; someone with not a ton of experience still could probably get it up and running in a day.

It doesn't look like there are any good tools (ala pandoc) for a straight Rmd-to-MyST conversion, but with the VSCode extensions for Rmd and for MyST both installed and a MyST cheat sheet, I still can't see it taking much more than a few hours.


From: Shantanu Singh shsingh@broadinstitute.org
Date: Tue, Nov 30, 2021 at 7:15 AM
To: Beth Cimini bcimini@broadinstitute.org

Good point – just 6 Rmd files, and it's mostly just the code blocks that will need conversion (if done by hand)

@bethac07
Copy link
Member

bethac07 commented Mar 8, 2022

@ErinWeisbart your call; we can discuss at checkin today if you need more info

@bethac07
Copy link
Member

bethac07 commented Mar 8, 2022

Otherwise, I would say might be a good project for Callum since a) he's working on CellProfiler so he's going to need to know a bit about GHActions for build stuff and b) he's planning on putting together the documentation for Piximi in Jupyter book and so having an easy starter project for that might be good practice.

@bethac07
Copy link
Member

bethac07 commented Mar 8, 2022

Callum has said he's happy to work on this next week, so let's maybe just pull this and make a separate GH issue for the migration?

@bethac07
Copy link
Member

bethac07 commented Mar 8, 2022

@ErinWeisbart you're also welcome to work on this too, or we can all work on it together, not trying to take it if you're interested :)

@ErinWeisbart
Copy link
Member Author

I've got plenty on my plate at the moment and I'm happy to have Callum handle the integration.

@bethac07
Copy link
Member

bethac07 commented Mar 9, 2022

I'm going to go ahead and merge then, and move the changing the build to a new issue.

@bethac07 bethac07 merged commit d9b9f5a into cytomining:master Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants