-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unintended behaviour in dseq.__init__? #253
Comments
This was by design. It is there so that we can make staggered sequences like so:
Does this create problems in other use cases? Maybe a warning would be appropriate. |
Hi @BjornFJohansson, in my example the returned sequence has mismatches at both ends, that's the problematic bit. If you are manually typing both strands, you may make a mistake when typing one of them, and you may want to get an error in that case. You can create a sequence with mismatches and stagger by passing the overhang, but not sure the auto-find of overhangs should be returning sequences with mismatches. In general, most functions of pydna will give unexpected behavior if there are mistmaches I guess? So I think an error would be good |
Hi @BjornFJohansson I was wondering whether we want to support this kind of behaviour for Dseq, or whether it is unintended.
gives a dseq with mismatches
I wonder if we should constrict the representation to have no mismatches (e.g. use terminal_overlap instead of common_substrings)? Or give an error if one like this comes up?
The text was updated successfully, but these errors were encountered: