galaxyproject / training-material

A collection of Galaxy-related training material
https://training.galaxyproject.org
MIT License
294 stars 846 forks source link

CRISPR finalise #3219

Closed mblue9 closed 2 years ago

mblue9 commented 2 years ago

Few things to fix in slides

mblue9 commented 2 years ago

Ready for review for Tapas if anyone can do it

hexylena commented 2 years ago

@mblue9 I'm looking at this and #3220 and wondering if there's not some organisation that might make more sense, than having both in their own section.

What would you think about renaming genome annotation to genomics? Would this fit appropriately under genomics, if we made a subtopic for genome editing?

hexylena commented 2 years ago

@mblue9 I'm working on this now! I'm made one small change, I'm replacing your **C**lustered **R**egularly... with {CRISPR}, and using the built in abbreviation plugin. This should be more friendly for screen-readers, as some will read emphasized letters separately from the rest of the word.

It looks like you're missing some alt-text for a few of the images. This was not caught in the initial round of linting due to an issue in the plugin which I've now fixed. Could you please provide alt text there?

mblue9 commented 2 years ago

Thanks for working on this @hexylena !

@mblue9 I'm looking at this and https://github.com/galaxyproject/training-material/pull/3220 and wondering if there's not some organisation that might make more sense, than having both in their own section.

What would you think about renaming genome annotation to genomics? Would this fit appropriately under genomics, if we made a subtopic for genome editing?

Yes that makes sense to me, should I go ahead and create a topic for genomics with subtopic genome-editing?

@mblue9 I'm working on this now! I'm made one small change, I'm replacing your Clustered Regularly... with {CRISPR}, and using the built in abbreviation plugin. This should be more friendly for screen-readers, as some will read emphasized letters separately from the rest of the word.

Great thanks!

It looks like you're missing some alt-text for a few of the images. This was not caught in the initial round of linting due to an issue in the plugin which I've now fixed. Could you please provide alt text there?

Will do

hexylena commented 2 years ago

I can handle the reorganisation, no dramas. It's pretty involved since we'll need to redirect all of the genome-annotation tutorials.

hexylena commented 2 years ago

Question: are you planning more CRISPR tutorials? (We've had some internal discussion about the rearranging and struggling to find a good solution, I was too optimistic.)

mblue9 commented 2 years ago

Question: are you planning more CRISPR tutorials? (We've had some internal discussion about the rearranging and struggling to find a good solution, I was too optimistic.)

I don't currently have plans for more. Was just looking at TN-seq tutorial and that's a similar concept (identifying essential genes) so maybe we put this under genome-annotation too as CRISPR can be used for identifying/annotating gene functions. This tutorial is not really about annotating genomes (it's determining action of a drug) but if it's better to put it there rather than in a topic on it's own we could do that?

hexylena commented 2 years ago

Ok, that works for now then

mblue9 commented 2 years ago

Thanks a lot @hexylena !

hexylena commented 2 years ago

Sure thing @mblue9 !! Great to see more tutorials :)

And one more thing: I'm sorry I missed this @mblue9, it looks like one slide is missing speaker notes:

image

unfortunately that will break the autogenerated video. Would you mind adding notes for that?

mblue9 commented 2 years ago

Fixed here @hexylena

mblue9 commented 2 years ago

Hi @hexylena just wondering if I need to do anything to get the autogenerated video to show up here? https://training.galaxyproject.org/training-material/videos/watch.html?v=/genome-annotation/tutorials/crispr-screen/slides

hexylena commented 2 years ago

No, nothing you need to do. Look like something's gone wrong with the video generation. I'll look into it. https://github.com/galaxyproject/training-material/actions/workflows/video.yml

mblue9 commented 2 years ago

Ok great thanks!

hexylena commented 2 years ago

I'm struggling to get the video building working again, something is wrong with the dependencies and I don't know what, I can't reproduce it locally. I'll build the video manually, locally. and upload it.

mblue9 commented 2 years ago

Ok thanks @hexylena ! I just realised I should probably add some pronounciations into https://github.com/galaxyproject/training-material/blob/main/bin/ari-map.yml e.g CRISPR = crisper have you already made the video or will I do that first?

hexylena commented 2 years ago

I've made it, https://training.galaxyproject.org/training-material/videos/watch.html?v=/genome-annotation/tutorials/crispr-screen/slides you can use it to check pronunciation and I'll re-make it whenever you're done :)

mblue9 commented 2 years ago

I've made it, https://training.galaxyproject.org/training-material/videos/watch.html?v=/genome-annotation/tutorials/crispr-screen/slides you can use it to check pronunciation and I'll re-make it whenever you're done :)

Great, done here https://github.com/galaxyproject/training-material/pull/3236

mblue9 commented 2 years ago

Forgot to say I got asked if the slides robot voice could have an Oz accent but I assume that's not possible 😆

hexylena commented 2 years ago

Oh defo possible! Only the woman's voice is good, the man's a bit synthetic sounding. I'll get that swapped in for ya. We can have her goin on about how grouse crispr is in no time 🤣

On Wed, Mar 9, 2022, 21:51 mblue9 @.***> wrote:

Forgot to say I got asked if the slides robot voice could have an Oz accent but I assume that's not possible 😆

— Reply to this email directly, view it on GitHub https://github.com/galaxyproject/training-material/pull/3219#issuecomment-1063359801, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADP7O467DQUINBRAJX77XDU7EFOPANCNFSM5P4CNXGQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

mblue9 commented 2 years ago

Oh defo possible! Only the woman's voice is good, the man's a bit synthetic sounding. I'll get that swapped in for ya. We can have her goin on about how grouse crispr is in no time 🤣

😃 Ohh brilliant!! 🙌 🎉

shiltemann commented 2 years ago

This is so cool! Thanks @mblue9! I think I am going to do this tutorial myself during Smorg next week, this looks very intersting :)

mblue9 commented 2 years ago

Aw thanks @shiltemann 🥰