laurenrw12 / Demultiplex

0 stars 0 forks source link

Pseudocode Review #1

Open daytonamelia opened 1 month ago

daytonamelia commented 1 month ago

Very clean pseudocode! It makes a lot of sense to me.

My feedback:

Great work! Makes a lot of sense to me :)

calzamora commented 1 month ago

Your code is beautiful!

Pseudocode: I think the way that you're populating the mismatched dict in order to track the index combinations is really smart and I'll probably steal that idea! I think your idea to edit the headers before looping through will definitely make populating headers down the line easier. My only question is if this will take extra time? I actually am not sure if its more efficient to do it here or when youre populating files.

Planned functions: These are all great ideas and the same functions I had thought about making as well.

Do you think that the convert qscore function that you're wrapping in check_qscore should work for +64 and +33? This is something I've been debating as well - these files are all +33 so the established function we have will work but maybe it would be worth broadening the scope long term? I haven't decided on this yet so I'm actually super curious on your thoughts!