Update PercolatorInfile.cpp to handle a corner case in sage (OpenMS#7977)

timosachsenberg · coderabbitai[bot] · web-flow · commit f29d4fe4a74d · 2025-04-10T16:48:41.000+02:00
* Update PercolatorInfile.cpp

* Apply suggestions from code review

Co-authored-by: coderabbitai[bot] &lt;136622811+coderabbitai[bot]@users.noreply.github.com&gt;

---------

Co-authored-by: coderabbitai[bot] &lt;136622811+coderabbitai[bot]@users.noreply.github.com&gt;
diff --git a/src/openms/source/FORMAT/PercolatorInfile.cpp b/src/openms/source/FORMAT/PercolatorInfile.cpp
@@ -337,7 +337,9 @@ namespace OpenMS
 
       for (const auto& name : found_extra_scores)
       {
-        ph.setMetaValue(name, row[to_idx.at(name)]);
+        String value = row[to_idx.at(name)];
+        if (name == "ln(-posson)" && value == "inf") value = "3.5"; // workaround for Sage
+        ph.setMetaValue(name, value);
       }
       ph.setRank(rank);
 
@@ -645,4 +647,4 @@ namespace OpenMS
     return count;
   }
 
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -337,7 +337,9 @@ namespace OpenMS`
`337`	`337`
`338`	`338`	`for (const auto& name : found_extra_scores)`
`339`	`339`	`{`
`340`		`- ph.setMetaValue(name, row[to_idx.at(name)]);`
	`340`	`+ String value = row[to_idx.at(name)];`
	`341`	`+ if (name == "ln(-posson)" && value == "inf") value = "3.5"; // workaround for Sage`
	`342`	`+ ph.setMetaValue(name, value);`
`341`	`343`	`}`
`342`	`344`	`ph.setRank(rank);`
`343`	`345`
`@@ -645,4 +647,4 @@ namespace OpenMS`
`645`	`647`	`return count;`
`646`	`648`	`}`
`647`	`649`
`648`		`-}`
	`650`	`+}`