BUG: updated dataframe sorting for identical values

.github/workflows/ci-dev-NEW.yaml → .github/workflows/ci-dev.yaml

@@ -1,13 +1,13 @@
 # Example of workflow trigger for calling workflow (the client).
-name: ci-dev-NEW
+name: ci-dev
 on:
   pull_request:
     branches: ["dev"]
   push:
     branches: ["dev"]
 jobs:
   ci:
-    uses: qiime2/distributions/.github/workflows/lib-ci-dev-NEW.yaml@dev
+    uses: qiime2/distributions/.github/workflows/lib-ci-dev.yaml@dev
     with:
       distro: amplicon
       recipe-path: 'conda-recipe'

q2_sample_classifier/classify.py

@@ -291,13 +291,6 @@ def fit_regressor(table: biom.Table,
         n_jobs, optimize_feature_selection, parameter_tuning,
         missing_samples=missing_samples, classification=False)
 
-    # this is sorted by the first column rather than 'importance' because
-    # the column name isn't consistent across methods - so this is the least
-    # invasive way to preserve order with the first column (which does
-    # contain the importance values)
-    importance = importance.sort_values(by=importance.columns[0],
-                                        ascending=False, kind='mergesort')
-
     return estimator, importance
 
 
@@ -369,13 +362,6 @@ def regress_samples_ncv(
         stratify, parameter_tuning, classification=False,
         scoring=mean_squared_error, missing_samples=missing_samples)
 
-    # this is sorted by the first column rather than 'importance' because
-    # the column name isn't consistent across methods - so this is the least
-    # invasive way to preserve order with the first column (which does
-    # contain the importance values)
-    importances = importances.sort_values(by=importances.columns[0],
-                                          ascending=False, kind='mergesort')
-
     return y_pred, importances