Merge pull request #2 from JLDC/fix-results

Fix double call issue to results

Author: JLDC

.gitignore

@@ -158,4 +158,6 @@ cython_debug/
 #  and can be added to the global gitignore or merged into this file.  For a more nuclear
 #  option (not recommended) you can uncomment the following to ignore the entire idea folder.
 #.idea/
-.vscode
+.vscode
+
+pipeline.md

model_confidence_set/core.py

@@ -251,14 +251,14 @@ def results(self, as_dataframe: bool = True) -> Union[dict, pd.DataFrame]:
 
         idx = np.concatenate([self.excluded, self.included]).astype(int) - 1
 
-        self.results = {
+        self.results_dict = {
             "pvalues": self.pvalues,
             "status": np.where(self.pvalues >= self.alpha, "included", "excluded"),
             "models": self.model_names[idx],
         }
         if as_dataframe:
-            df = pd.DataFrame(self.results)
+            df = pd.DataFrame(self.results_dict)
             df.index = df.pop("models")
             return df
         else:
-            return self.results
+            return self.results_dict

setup.py

@@ -9,7 +9,7 @@
 
 setup(
     name="model-confidence-set",
-    version="0.1.2",
+    version="0.1.3",
     license="MIT",
     description="model-confidence-set provides a Python implementation of the Model Confidence Set (MCS) procedure (Hansen, Lunde, and Nason, 2011), a statistical method for comparing and selecting models based on their performance.",
     long_description=long_description,