Merge branch 'main' into pivot_sort

Author: ShawnZhong

.github/workflows/windows.yml renamed to .github/workflows/macos-windows.yml

@@ -1,4 +1,4 @@
-name: Windows
+name: Windows-MacOS
 
 on:
   push:
@@ -21,18 +21,20 @@ env:
 
 jobs:
   pytest:
-    runs-on: windows-latest
     defaults:
       run:
         shell: bash -el {0}
     timeout-minutes: 90
     strategy:
       matrix:
+        os: [macos-latest, windows-latest]
         env_file: [actions-38.yaml, actions-39.yaml, actions-310.yaml]
       fail-fast: false
+    runs-on: ${{ matrix.os }}
+    name: ${{ format('{0} {1}', matrix.os, matrix.env_file) }}
     concurrency:
       # https://github.community/t/concurrecy-not-work-for-push/183068/7
-      group: ${{ github.event_name == 'push' && github.run_number || github.ref }}-${{ matrix.env_file }}-windows
+      group: ${{ github.event_name == 'push' && github.run_number || github.ref }}-${{ matrix.env_file }}-${{ matrix.os }}
       cancel-in-progress: true
 
     steps:
@@ -47,10 +49,17 @@ jobs:
         mamba-version: "*"
         channels: conda-forge
         activate-environment: pandas-dev
-        channel-priority: strict
+        channel-priority: ${{ matrix.os == 'macos-latest' && 'flexible' || 'strict' }}
         environment-file: ci/deps/${{ matrix.env_file }}
         use-only-tar-bz2: true
 
+    # ImportError: 2): Library not loaded: @rpath/libssl.1.1.dylib
+    # Referenced from: /Users/runner/miniconda3/envs/pandas-dev/lib/libthrift.0.13.0.dylib
+    # Reason: image not found
+    - name: Upgrade pyarrow on MacOS
+      run: conda install -n pandas-dev -c conda-forge --no-update-deps pyarrow=6
+      if: ${{ matrix.os == 'macos-latest' }}
+
     - name: Build Pandas
       uses: ./.github/actions/build_pandas
 

azure-pipelines.yml

@@ -22,11 +22,6 @@ variables:
   PANDAS_CI: 1
 
 jobs:
-- template: ci/azure/posix.yml
-  parameters:
-    name: macOS
-    vmImage: macOS-10.15
-
 - job: py38_32bit
   pool:
     vmImage: ubuntu-18.04

ci/azure/posix.yml

@@ -73,15 +73,6 @@ mamba install -n pandas-dev 'setuptools<60'
 echo "conda list -n pandas-dev"
 conda list -n pandas-dev
 
-# From pyarrow on MacOS
-# ImportError: 2): Library not loaded: @rpath/libssl.1.1.dylib
-# Referenced from: /Users/runner/miniconda3/envs/pandas-dev/lib/libthrift.0.13.0.dylib
-# Reason: image not found
-if [[ "$(uname)" == 'Darwin' ]]; then
-    echo "Update pyarrow for pyarrow on MacOS"
-    conda install -n pandas-dev -c conda-forge --no-update-deps pyarrow=6
-fi
-
 if [[ "$BITS32" == "yes" ]]; then
     # activate 32-bit compiler
     export CONDA_BUILD=1

ci/setup_env.sh

@@ -26,7 +26,10 @@
 
 import numpy as np
 
-from pandas._libs import reduction as libreduction
+from pandas._libs import (
+    Interval,
+    reduction as libreduction,
+)
 from pandas._typing import (
     ArrayLike,
     Manager,
@@ -652,12 +655,9 @@ def value_counts(
 
         if is_interval_dtype(lab.dtype):
             # TODO: should we do this inside II?
+            lab_interval = cast(Interval, lab)
 
-            # error: "ndarray" has no attribute "left"
-            # error: "ndarray" has no attribute "right"
-            sorter = np.lexsort(
-                (lab.left, lab.right, ids)  # type: ignore[attr-defined]
-            )
+            sorter = np.lexsort((lab_interval.left, lab_interval.right, ids))
         else:
             sorter = np.lexsort((lab, ids))