Conversation
|
Once the build has completed, you can preview your PR at this URL: https://biojulia.dev/BiojuliaDocs/previews/PR15/ |
|
@kescobo Please let me know if the changes I implemented look good and if I can merge in! |
kescobo
left a comment
kescobo
left a comment
There was a problem hiding this comment.
Also, this still needs semantic line breaks (sembr.org). You can ask an LLM to suggest if you're not sure, but it's not a hard-and fast rule, just shooting for shorter lines that are more-or-less complete clauses. It makes the diffs much nicer if future edits are made
|
|
||
| ### For Loop | ||
|
|
||
| Another way we can approach this would be to use the for-loop. This method will be a bit slower. |
There was a problem hiding this comment.
Not necessarily slower - actually this one might be a good candidate for a benchmarking run at the end.
Julia can often optimize for-loops like this, which is one of the things that makes it so powerful. You get SIMD from the count version, but you can also potentially make the for-loop parallel.
|
|
||
|
|
There was a problem hiding this comment.
Some excessive line breaks here
| hamming(ex_seq_a, ex_seq_b) | ||
|
|
||
| ``` | ||
|
|
2 participants
Tutorial for Hamming Distance problem
I added two different ways to solve the problem: with a function I wrote, as well as a function in the BioAlignments package. Please let me know if there's anything else that would be good to add to this tutorial!