[ARVADOS] updated: bb2574d7e3fc9b3031c881044b5878c2307b9d40

git at public.curoverse.com git at public.curoverse.com
Wed Feb 12 15:53:51 EST 2014


Summary of changes:
 .../controllers/pipeline_instances_controller.rb   |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

       via  bb2574d7e3fc9b3031c881044b5878c2307b9d40 (commit)
      from  2eb5599c5a74fd84875258a3f0fdcff21a923ce4 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit bb2574d7e3fc9b3031c881044b5878c2307b9d40
Author: Ward Vandewege <ward at curoverse.com>
Date:   Wed Feb 12 15:52:13 2014 -0500

    In the pipeline instance comparison, do not blow up when a row component
    is not defined (for example, when one or more of the pipeline instances
    in the comparison has not yet been executed).

diff --git a/apps/workbench/app/controllers/pipeline_instances_controller.rb b/apps/workbench/app/controllers/pipeline_instances_controller.rb
index 7ca4f6a..42cb2e9 100644
--- a/apps/workbench/app/controllers/pipeline_instances_controller.rb
+++ b/apps/workbench/app/controllers/pipeline_instances_controller.rb
@@ -95,6 +95,7 @@ class PipelineInstancesController < ApplicationController
       highscore = {}           # attr => how common "normal" is
       score = {}               # attr => { value => how common }
       row[:components].each do |pj|
+        next if pj.nil?
         pj.each do |k,v|
           vstr = for_comparison v
           score[k] ||= {}
@@ -113,6 +114,7 @@ class PipelineInstancesController < ApplicationController
 
       # Add a hash in component[:is_normal]: { attr => is_the_value_normal? }
       row[:components].each do |pj|
+        next if pj.nil?
         pj[:is_normal] = {}
         pj.each do |k,v|
           pj[:is_normal][k] = (normal.has_key?(k) && normal[k] == for_comparison(v))

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list