Changeset dd3a80 for molecuilder/src/analysis_correlation.cpp

Timestamp:

Mar 3, 2010, 7:10:36 PM (16 years ago)

Author:

Frederik Heber <heber@…>

Children:

bc46c1

Parents:

f78203

git-author:

Frederik Heber <heber@…> (03/03/10 18:41:03)

git-committer:

Frederik Heber <heber@…> (03/03/10 19:10:36)

Message:

BUGFIX: rounding imprecision let values appear twice in BinMap.

• we fix this by always converting bin to integer and then back to double when putting into map in BinData()
• GetBin returns integer

Signed-off-by: Frederik Heber <heber@…>

File:

• molecuilder/src/analysis_correlation.cpp (modified) (1 diff)

molecuilder/src/analysis_correlation.cpp

@@ -342 +342 @@
 };
 
-/** Returns the start of the bin for a given value.
+/** Returns the index of the bin for a given value.
  * \param value value whose bin to look for
  * \param BinWidth width of bin
  * \param BinStart first bin
  */
-double GetBin ( const double value, const double BinWidth, const double BinStart )
-{
-  Info FunctionInfo(__func__);
-  double bin =(double) (floor((value - BinStart)/BinWidth));
-  return (bin*BinWidth+BinStart);
+int GetBin ( const double value, const double BinWidth, const double BinStart )
+{
+  Info FunctionInfo(__func__);
+  int bin =(int) (floor((value - BinStart)/BinWidth));
+  return (bin);
 };