fix: exclude factors from unique gene name check in MPs (#499)

dgidb · Jun 4, 2024 · 34c4f66 · 34c4f66
1 parent 1c0c3e2
commit 34c4f66
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/server/lib/genome/importers/api_importers/civic/api_client.rb b/server/lib/genome/importers/api_importers/civic/api_client.rb
@@ -109,6 +109,7 @@ def headers(_context)
                 variants {
                   feature {
                     featureInstance {
+                      __typename
                       ... on Gene {
                         name
                       }

diff --git a/server/lib/genome/importers/api_importers/civic/importer.rb b/server/lib/genome/importers/api_importers/civic/importer.rb
@@ -71,7 +71,14 @@ def create_interaction_claims
 
               # retain molecular profiles consisting of multiple variations on the same gene,
               # but skip multi-gene profiles (eg fusions)
-              gene_names = ei.molecular_profile.variants.map { |v| v.feature.feature_instance.name.upcase }.uniq
+              gene_names = ei.molecular_profile.variants.map do |variant|
+                feature_instance = variant.feature.feature_instance
+                if feature_instance.__typename == "Gene"
+                  feature_instance.name.upcase
+                else  # skip Factors for now
+                  nil
+                end
+              end.compact.uniq
               next if gene_names.length != 1
               gc = GeneClaim.joins(:source).where(sources: {source_db_name: "CIViC"}, gene_claims: {name: gene_names[0]}).first