search

seqan / seqan3

The modern C++ library for sequence analysis. Contains version 3 of the library and API docs.
https://www.seqan.de
Other
411 stars 82 forks source link

Fix/issue_3264 #3268

Closed rrahn closed 4 months ago

rrahn commented 4 months ago

Further constraints the conversion constructors of the nucleotide and aminoacid base alphabet types to only consider conversion if all requirements are fulfilled. This is now represented by the specific concept convertable_to_through_char_representation. Through this proxy types will not be converted through their char representation but will be implicitly converted to their underling base alphabet type, e.g. dna4, and then copied/moved using the copy/move constructor of the underlying base alphabet type.

Fixes #3264 Fixes #3267

vercel[bot] commented 4 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
seqan3 ✅ Ready (Inspect) Visit Preview Jul 16, 2024 5:39pm
rrahn commented 4 months ago

Do you have an opinion regarding https://github.com/seqan/seqan3/issues/3267#issuecomment-2230757229 ?

I answered on the issue page.