From 92813b2e02a0a68b941ff198d4da60b77dc6bc6e Mon Sep 17 00:00:00 2001
From: Diane Trout <diane@ghic.org>
Date: Tue, 5 May 2015 16:29:40 -0700
Subject: [PATCH] simplify library report

---
 .../templates/samples/library_index.html      | 32 +++++----
 samples/views.py                              | 66 ++-----------------
 2 files changed, 27 insertions(+), 71 deletions(-)

diff --git a/htsworkflow/templates/samples/library_index.html b/htsworkflow/templates/samples/library_index.html
index 6ee9750..4491a09 100644
--- a/htsworkflow/templates/samples/library_index.html
+++ b/htsworkflow/templates/samples/library_index.html
@@ -71,26 +71,34 @@
     </thead>
     <tbody >
       {% for lib in library_list %}
-      <tr typeof="libns:Library" about="{{lib.library.get_absolute_url}}">
-        <td ><a href="{{lib.library.get_absolute_url}}">{{ lib.amplified_from }}</a></td>
-        <td ><a href="{{lib.library.get_absolute_url}}"><span property="libns:library_id">{{ lib.library_id }}</span></a></td>
+      <tr typeof="libns:Library" about="{{lib.get_absolute_url}}">
+        <td >
+          <a href="{{lib.amplified_from_sample.get_absolute_url }}">
+            {{ lib.amplified_from_sample.id }}
+          </a>
+        </td>
+        <td >
+          <a href="{{lib.get_absolute_url}}">
+            <span property="libns:library_id">{{ lib.id }}</span>
+          </a>
+        </td>
         <td rel="libns:species">
           <a typeof="libns:Species"
-             href="{{lib.library.library_species.get_absolute_url}}">
-            <span property="libns:species_name">{{ lib.species_name }}</span>
+             href="{{ lib.library_species.get_absolute_url }}">
+            <span property="libns:species_name">{{ lib.library_species.scientific_name }}</span>
           </a>
         </td>
         <td >
-          <a href="{{ lib.library.get_absolute_url }}">
+          <a href="{{ lib.get_absolute_url }}">
             <span property="libns:name">{{ lib.library_name }}</span>
           </a>
         </td>
-        <td  bgcolor="#00BFFF">{{ lib.lanes_run.0.0 }}</td>
-        <td  bgcolor="#00BFFF">{{ lib.lanes_run.0.1 }}</td>
-        <td  bgcolor="#00BFFF">{{ lib.lanes_run.0.2 }}</td>
-        <td  bgcolor="#66CDAA">{{ lib.lanes_run.1.0 }}</td>
-        <td  bgcolor="#66CDAA">{{ lib.lanes_run.1.1 }}</td>
-        <td  bgcolor="#66CDAA">{{ lib.lanes_run.1.2 }}</td>
+        {% for ended in lib.lanes_sequenced %}
+          {% cycle '#00BFFF' '#66CDAA' as bincolor silent %}
+          {% for bin in ended %}
+          <td bgcolor="{{ bincolor }}">{% if bin %}{{ bin }}{% endif %}</td>
+          {% endfor %}
+        {% endfor %}
         {% if lib.is_archived %}
           <td ><img src="{% static "img/hdd_unmount.png" %}" alt="Archived" /></td>
         {% else %}
diff --git a/samples/views.py b/samples/views.py
index e173b08..3b5ea35 100644
--- a/samples/views.py
+++ b/samples/views.py
@@ -35,75 +35,23 @@ SAMPLES_CONTEXT_DEFAULTS = {
 LOGGER = logging.getLogger(__name__)
 
 
-def count_lanes(lane_set):
-    single = 0
-    paired = 1
-    short_read = 0
-    medium_read = 1
-    long_read = 2
-    counts = [[0, 0, 0], [0, 0, 0]]
-
-    for lane in lane_set.all():
-        if lane.flowcell.paired_end:
-            lane_type = paired
-        else:
-            lane_type = single
-        if lane.flowcell.read_length < 40:
-            read_type = short_read
-        elif lane.flowcell.read_length < 100:
-            read_type = medium_read
-        else:
-            read_type = long_read
-        counts[lane_type][read_type] += 1
-
-    return counts
-
-
-def create_library_context(cl):
-    """
-     Create a list of libraries that includes how many lanes were run
-    """
-    records = []
-    #for lib in library_items.object_list:
-    for lib in cl.result_list:
-        summary = {}
-        summary['library'] = lib
-        summary['library_id'] = lib.id
-        summary['library_name'] = lib.library_name
-        summary['species_name'] = lib.library_species.scientific_name
-        if lib.amplified_from_sample is not None:
-            summary['amplified_from'] = lib.amplified_from_sample.id
-        else:
-            summary['amplified_from'] = ''
-        lanes_run = count_lanes(lib.lane_set)
-        # suppress zeros
-        for row_index, row in enumerate(lanes_run):
-            for col_index, cell in enumerate(row):
-                if lanes_run[row_index][col_index] == 0:
-                    lanes_run[row_index][col_index] = ''
-        summary['lanes_run'] = lanes_run
-        summary['is_archived'] = lib.is_archived()
-        records.append(summary)
-    cl.result_count = str(cl.paginator._count)
-    return {'library_list': records}
-
-
 def library(request, todo_only=False):
-    queryset = Library.objects.filter(hidden__exact=0)
-    filters = {'hidden__exact': 0}
+    #filters = {'hidden__exact': 0}
     if todo_only:
         filters['lane'] = None
-    # build changelist
+
     fcl = HTSChangeList(request, Library,
                         list_filter=['affiliations', 'library_species'],
                         search_fields=['id', 'library_name', 'amplified_from_sample__id'],
                         list_per_page=200,
                         model_admin=LibraryOptions(Library, None),
-                        extra_filters=filters
                         )
 
-    context = {'cl': fcl, 'title': 'Library Index', 'todo_only': todo_only}
-    context.update(create_library_context(fcl))
+    context = {'cl': fcl,
+               'library_list': fcl.result_list,
+               'title': 'Library Index',
+               'todo_only': todo_only}
+
     t = get_template('samples/library_index.html')
     c = RequestContext(request, context)
     return HttpResponse(t.render(c))
-- 
2.30.2