LJI-Bioinformatics / Excel-Reverse-Complement

A simple add-in for Excel supplying functions to calculate the reverse, complement, and reverse-complement of a nucleotide sequence
42 stars 14 forks source link

complement/revcom not working for lower case nucleotide #6

Open xiaohan2011 opened 2 years ago

xiaohan2011 commented 2 years ago

Good morning, Thanks for sharing this code! Just wanted to point out that the complement function does not work for lower case as its current state. Suggest adding one line input_str = UCase(input_str).

ben-m-scott commented 1 year ago

I've fixed it for you, and added support for n/N degenerate nucleotides. Download revcom.xlam from here https://github.com/Tr0x315/Excel-Reverse-Complement

jgbaum commented 1 year ago

Thanks @Tr0x315! Would you mind copying the source code into the '.vba' file as well?

jgbaum commented 1 year ago

Good morning, Thanks for sharing this code! Just wanted to point out that the complement function does not work for lower case as its current state. Suggest adding one line input_str = UCase(input_str).

Thanks, @xiaohan2011. Which OS and version are you using? Also, which version of Office?

ben-m-scott commented 1 year ago

Thanks @Tr0x315! Would you mind copying the source code into the '.vba' file as well?

Sure, I updated the '.vba' file here: https://github.com/Tr0x315/Excel-Reverse-Complement

ben-m-scott commented 1 year ago

@jgbaum just making sure you see the updated .vba file above, as requested.

I'm a github noob, so hopefully it worked

jvdesomp commented 1 month ago

Using Excel version 16.88 (24081116) on Mac, lower case nucleotides are not complemented in revcom().