Cole Lyman

Glad to hear that you got it working! Good luck analyzing your experiments. > On Sep 12, 2022, at 4:18 AM, clarazourray ***@***.***> wrote: > > > Hi Cole, >...

@GuillaumeHolley is there anything keeping this PR from being merged in and released? I am happy to fix it if that is the case. Thanks, Cole

From what I can tell, the `combine_A_and_B.py` script was lifted from this repository, https://github.com/EliasVansteenkiste/CycleGANwithPerceptionLoss At the end of the README for that repo, they offer this documentation for the script:...

I, Cole Lyman, give permission to license my contributions to Rash under MIT/Apache 2. I've used the email address cole at colelyman dot com.

This was implemented in https://github.com/pinellolab/CRISPResso2/commit/1d4679c72d0c8b4154317c9aff5179217198e2d7 thanks for the suggestion @jykr!

I agree with @martincpt, for anyone else that stumbles here, I was able to get it running on M1 Mac 12.7.5 after checking out this branch and running `go build`....

I have split up the genome and discovered that the failed assertion comes from somewhere in the X chromosome. I will split up the file further in order to find...

So I deleted three bases (an arbitrary number) off of the end of the `X` chromosome (which are `TAC`), and the assert no longer fails. Could this behavior possibly be...

I have updated the function so that the lexicographically smallest possible nucleotide for each ambiguity code. Thanks for the suggestion @sjackman

My use case is using assembled genomes. Ideally, I would like to be able to keep `N`s, but then I thought it would be helpful to accept all IUPAC codes....