Merge remote-tracking branch 'refs/remotes/origin/main' into raffi_re…

…cord_adapter # Conflicts: # py/server/deephaven/table.py
deephaven · Aug 23, 2024 · d9a12bc · d9a12bc
2 parents 6745f95 + 187bbb6
commit d9a12bc
Show file tree

Hide file tree

Showing 242 changed files with 7,836 additions and 3,269 deletions.
diff --git a/.github/workflows/build-ci.yml b/.github/workflows/build-ci.yml
@@ -29,6 +29,9 @@ jobs:
           distribution: 'temurin'
           java-version: '17'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -88,11 +91,7 @@ jobs:
           password: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Create Dockerfile and context
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: build-server
-          arguments: --scan outputVersion docker-server-slim:prepareDocker docker-server:prepareDockerAll
-          gradle-version: wrapper
+        run: ./gradlew --scan outputVersion docker-server-slim:prepareDocker docker-server:prepareDockerAll
 
       - name: Get Deephaven Version
         id: deephaven_version
@@ -158,6 +157,9 @@ jobs:
           distribution: 'temurin'
           java-version: '11'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -182,11 +184,7 @@ jobs:
           password: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Create Dockerfile and context
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: build-web
-          arguments: --scan outputVersion docker-web-plugin-packager:prepareDocker
-          gradle-version: wrapper
+        run: ./gradlew --scan outputVersion docker-web-plugin-packager:prepareDocker
 
       - name: Get Deephaven Version
         id: deephaven_version

diff --git a/.github/workflows/check-ci.yml b/.github/workflows/check-ci.yml
@@ -33,6 +33,9 @@ jobs:
           distribution: 'temurin'
           java-version: '17'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -42,11 +45,7 @@ jobs:
           cat gradle.properties
 
       - name: Check
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: checks
-          arguments: --scan --continue check
-          gradle-version: wrapper
+        run: ./gradlew --scan --continue check
 
       - name: Upload Test Results
         uses: actions/upload-artifact@v4

diff --git a/.github/workflows/cla.yml b/.github/workflows/cla.yml
@@ -21,7 +21,7 @@ jobs:
     steps:
       - name: "CLA Assistant"
         if: (github.event.comment.body == 'recheck' || github.event.comment.body == 'I have read the CLA Document and I hereby sign the CLA') || github.event_name == 'pull_request_target'
-        uses: cla-assistant/github-action@v2.4.0
+        uses: cla-assistant/github-action@v2.5.1
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           PERSONAL_ACCESS_TOKEN : ${{ secrets.CLA_PERSONAL_ACCESS_TOKEN }}

diff --git a/.github/workflows/dependency-submission.yml b/.github/workflows/dependency-submission.yml
@@ -30,7 +30,7 @@ jobs:
           cat gradle.properties
 
       - name: Generate and submit dependency graph
-        uses: gradle/actions/dependency-submission@v3
+        uses: gradle/actions/dependency-submission@v4
         env:
           # Dependencies derived from :server-jetty-app runtimeClasspath get a "runtime" scope and everything else gets
           # a "development" scope. Ideally, gradle would be able to pass along the finer dependency details (for

diff --git a/.github/workflows/docs-ci.yml b/.github/workflows/docs-ci.yml
@@ -76,6 +76,9 @@ jobs:
           distribution: 'temurin'
           java-version: '17'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -85,12 +88,8 @@ jobs:
           cat gradle.properties
 
       - name: All Javadoc
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: allJavadoc
-          arguments: --scan outputVersion combined-javadoc:allJavadoc
-          gradle-version: wrapper
-
+        run: ./gradlew --scan outputVersion combined-javadoc:allJavadoc
+
       - name: Get Deephaven Version
         id: dhc-version
         run: echo "version=$(cat build/version)" >> $GITHUB_OUTPUT
@@ -139,6 +138,9 @@ jobs:
           distribution: 'temurin'
           java-version: '17'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -148,12 +150,8 @@ jobs:
           cat gradle.properties
 
       - name: Run typedoc on JS API
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: typedoc
-          arguments: --scan outputVersion :web-client-api:types:typedoc
-          gradle-version: wrapper
-
+        run: ./gradlew --scan outputVersion :web-client-api:types:typedoc
+
       - name: Get Deephaven Version
         id: dhc-version
         run: echo "version=$(cat build/version)" >> $GITHUB_OUTPUT
@@ -202,6 +200,9 @@ jobs:
           distribution: 'temurin'
           java-version: '17'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -211,12 +212,8 @@ jobs:
           cat gradle.properties
 
       - name: Generate Python Docs
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: pythonDocs
-          arguments: --scan outputVersion sphinx:pythonDocs sphinx:pydeephavenDocs
-          gradle-version: wrapper
-
+        run: ./gradlew --scan outputVersion sphinx:pythonDocs sphinx:pydeephavenDocs
+
       - name: Get Deephaven Version
         id: dhc-version
         run: echo "version=$(cat build/version)" >> $GITHUB_OUTPUT
@@ -285,6 +282,9 @@ jobs:
           distribution: 'temurin'
           java-version: '11'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -294,12 +294,8 @@ jobs:
           cat gradle.properties
 
       - name: Generate C++ Docs
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: cppDocs
-          arguments: --scan outputVersion sphinx:cppClientDocs sphinx:cppExamplesDocs
-          gradle-version: wrapper
-
+        run: ./gradlew --scan outputVersion sphinx:cppClientDocs sphinx:cppExamplesDocs
+
       - name: Get Deephaven Version
         id: dhc-version
         run: echo "version=$(cat build/version)" >> $GITHUB_OUTPUT
@@ -360,6 +356,9 @@ jobs:
           distribution: 'temurin'
           java-version: '11'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -369,12 +368,8 @@ jobs:
           cat gradle.properties
 
       - name: Generate R Docs
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: rDocs
-          arguments: --scan outputVersion R:rClientSite
-          gradle-version: wrapper
-
+        run: ./gradlew --scan outputVersion R:rClientSite
+
       - name: Get Deephaven Version
         id: dhc-version
         run: echo "version=$(cat build/version)" >> $GITHUB_OUTPUT

diff --git a/.github/workflows/nightly-check-ci.yml b/.github/workflows/nightly-check-ci.yml
@@ -55,6 +55,9 @@ jobs:
           distribution: 'temurin'
           java-version: '22'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -64,11 +67,7 @@ jobs:
           cat gradle.properties
 
       - name: Run gradle ${{ matrix.gradle-task }} on java ${{ matrix.test-jvm-version }}
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: gradle-run
-          arguments: --scan --continue --rerun-tasks ${{ matrix.gradle-task }} -PtestRuntimeVersion=${{ matrix.test-jvm-version }}
-          gradle-version: wrapper
+        run: ./gradlew --scan --continue --rerun-tasks ${{ matrix.gradle-task }} -PtestRuntimeVersion=${{ matrix.test-jvm-version }}
 
       - name: Upload Test Results
         uses: actions/upload-artifact@v4

diff --git a/.github/workflows/nightly-image-check.yml b/.github/workflows/nightly-image-check.yml
@@ -20,6 +20,9 @@ jobs:
           distribution: 'temurin'
           java-version: '11'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -29,11 +32,7 @@ jobs:
           cat gradle.properties
 
       - name: Run gradle
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: image-compare
-          arguments: --continue pullImage compareImage
-          gradle-version: wrapper
+        run: ./gradlew --continue pullImage compareImage
 
       - name: Notify Slack
         uses: slackapi/[email protected]

diff --git a/.github/workflows/nightly-publish-ci.yml b/.github/workflows/nightly-publish-ci.yml
@@ -12,6 +12,7 @@ concurrency:
 jobs:
   nightly-publish:
     runs-on: ubuntu-24.04
+    if: ${{ github.repository_owner == 'deephaven' }}
 
     steps:
       - name: Checkout
@@ -32,7 +33,7 @@ jobs:
           java-version: '17'
 
       - name: Setup Gradle
-        uses: gradle/actions/setup-gradle@v3
+        uses: gradle/actions/setup-gradle@v4
 
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV

diff --git a/.github/workflows/publish-ci.yml b/.github/workflows/publish-ci.yml
@@ -33,6 +33,9 @@ jobs:
           distribution: 'temurin'
           java-version: '17'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -51,20 +54,12 @@ jobs:
 
       - name: Build all artifacts, publish to Maven Local
         if: ${{ !startsWith(github.ref, 'refs/heads/release/v') }}
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: publish-local
-          arguments: server-netty-app:build server-jetty-app:build py-server:build py-embedded-server:build py-client:build py-client-ticking:build web-client-api:types:build publishToMavenLocal
-          gradle-version: wrapper
+        run: ./gradlew server-netty-app:build server-jetty-app:build py-server:build py-embedded-server:build py-client:build py-client-ticking:build web-client-api:types:build publishToMavenLocal
 
       - name: Build all artifacts, publish to Sonatype for staging to Maven Central
         if: ${{ startsWith(github.ref, 'refs/heads/release/v') }}
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: publish
-          # We need to be explicit here about no parallelism to ensure we don't create disjointed staging repositories.
-          arguments: --no-parallel server-netty-app:build server-jetty-app:build py-server:build py-embedded-server:build py-client:build py-client-ticking:build web-client-api:types:build publish
-          gradle-version: wrapper
+        # We need to be explicit here about no parallelism to ensure we don't create disjointed staging repositories.
+        run: ./gradlew --no-parallel server-netty-app:build server-jetty-app:build py-server:build py-embedded-server:build py-client:build py-client-ticking:build web-client-api:types:build publish
         env:
           ORG_GRADLE_PROJECT_ossrhUsername: ${{ secrets.SONATYPE_USERNAME }}
           ORG_GRADLE_PROJECT_ossrhPassword: ${{ secrets.SONATYPE_PASSWORD }}

diff --git a/.github/workflows/quick-ci.yml b/.github/workflows/quick-ci.yml
@@ -27,6 +27,9 @@ jobs:
           distribution: 'temurin'
           java-version: '11'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Set JAVA_HOME
         run: echo "JAVA_HOME=${{ steps.setup-java-11.outputs.path }}" >> $GITHUB_ENV
 
@@ -36,12 +39,8 @@ jobs:
           cat gradle.properties
 
       - name: Quick Task
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: quick-task
-          # Even though quick includes spotlessCheck, we want to make sure it runs first and fails ASAP for quick feedback
-          arguments: --scan spotlessCheck quick
-          gradle-version: wrapper
+        # Even though quick includes spotlessCheck, we want to make sure it runs first and fails ASAP for quick feedback
+        run: ./gradlew --scan spotlessCheck quick
 
       - name: Upload JVM Error Logs
         uses: actions/upload-artifact@v4
@@ -64,4 +63,4 @@ jobs:
         run: pip install vermin==1.6.0
 
       - name: Verify minimum version support
-        run: vermin -t=3.8 --no-tips --eval-annotations --violations py/server/deephaven py/client py/client-ticking py/embedded-server
+        run: vermin -t=3.8 --no-tips --eval-annotations --violations --feature fstring-self-doc --feature union-types py/server/deephaven py/client py/client-ticking py/embedded-server
diff --git a/.github/workflows/tag-base-images.yml b/.github/workflows/tag-base-images.yml
@@ -29,6 +29,9 @@ jobs:
           distribution: 'temurin'
           java-version: '17'
 
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v4
+
       - name: Setup Crane
         if: ${{ startsWith(github.ref, 'refs/heads/release/v') }}
         uses: imjasonh/[email protected]
@@ -50,11 +53,7 @@ jobs:
           password: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Create Crane scripts
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          job-id: crane-scripts
-          arguments: createCraneTagScript
-          gradle-version: wrapper
+        run: ./gradlew createCraneTagScript
 
       - name: Tag upstream images
         if: ${{ startsWith(github.ref, 'refs/heads/release/v') }}

diff --git a/Generators/src/main/java/io/deephaven/plot/util/GeneratePyV2FigureAPI.java b/Generators/src/main/java/io/deephaven/plot/util/GeneratePyV2FigureAPI.java
@@ -1280,7 +1280,7 @@ private static List<PyFunc> getPyFuncs() {
         rst.add(new PyFunc("y_ticks_minor", SEQUENTIAL, new String[] {"yMinorTicks", "yMinorTicksVisible"}, null,
                 "Updates the configuration for minor ticks of the y-Axis."));
         rst.add(new PyFunc("y_twin", SINGLETON, new String[] {"twinY"}, null,
-                "Creates a new Axes which shares the y-Axis with the current Axes. For example, this is used for creating plots with a common x-axis but two different y-axes."));
+                "Creates a new Axes which shares the y-Axis with the current Axes. For example, this is used for creating plots with a common y-axis but two different x-axes."));
         rst.add(new PyFunc("series", SEQUENTIAL, new String[] {"series", "group", "seriesColor", "toolTipPattern",
                 "xToolTipPattern", "yToolTipPattern", "errorBarColor", "gradientVisible", "seriesNamingFunction"}, null,
                 "Gets a specific data series and updates the data series's configurations."));

diff --git a/Integrations/build.gradle b/Integrations/build.gradle
@@ -15,6 +15,7 @@ dependencies {
     implementation project(':plugin')
     implementation project(':Configuration')
     implementation project(':log-factory')
+    implementation libs.commons.lang3
 
     testImplementation project(':engine-test-utils')
     testImplementation project(path: ':Base', configuration: 'tests')