Skip to content
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

lower case seqan3::detail::translation_table::value #318

Closed
2 tasks
marehr opened this issue Apr 18, 2021 · 0 comments · Fixed by seqan/seqan3#2597
Closed
2 tasks

lower case seqan3::detail::translation_table::value #318

marehr opened this issue Apr 18, 2021 · 0 comments · Fixed by seqan/seqan3#2597
Assignees

Comments

@marehr
Copy link
Member

marehr commented Apr 18, 2021

Tasks

  • apply patch and use that as basis
  • lower case seqan3::detail::translation_table::value
From 9b1828f9866b11936cd13bb0eeca7fad48217fd5 Mon Sep 17 00:00:00 2001
From: marehr <marehr-github@marehr.dialup.fu-berlin.de>
Date: Tue, 23 Mar 2021 00:27:41 +0100
Subject: [PATCH 08/12] TODO: [MISC] lower case translation_table::value

---
 include/seqan3/alphabet/aminoacid/translation_details.hpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/seqan3/alphabet/aminoacid/translation_details.hpp b/include/seqan3/alphabet/aminoacid/translation_details.hpp
index b32fcaf10..24cde74a0 100644
--- a/include/seqan3/alphabet/aminoacid/translation_details.hpp
+++ b/include/seqan3/alphabet/aminoacid/translation_details.hpp
@@ -60,7 +60,7 @@ template <typename void_type>
 struct translation_table<dna15, seqan3::genetic_code::canonical, void_type>
 {
     //!\brief Holds the translation table for canonical genetic code and nucl16 alphabet.
-    static constexpr aa27 VALUE[dna15::alphabet_size][dna15::alphabet_size][dna15::alphabet_size]
+    static constexpr aa27 value[dna15::alphabet_size][dna15::alphabet_size][dna15::alphabet_size]
     {
         { // a??
             // a,        b,        c,        d,        g,        h,        k,        m,        n,        r,        s,        t,        v,        w,        y
-- 
2.31.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants