Merge pull request #1009 from StanfordVL/fix-profiling

Fix profiling, remove unnecessary CI steps
StanfordVL · Dec 11, 2024 · c83956f · c83956f
2 parents 4a3ee62 + ae931b9
commit c83956f
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 23 deletions.
diff --git a/.github/workflows/examples-as-test.yml b/.github/workflows/examples-as-test.yml
@@ -21,22 +21,18 @@ jobs:
         shell: micromamba run -n omnigibson /bin/bash -leo pipefail {0}
     steps:
       - name: Checkout source
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
         with:
           submodules: true
-          path: omnigibson-src
 
       - name: Install
-        working-directory: omnigibson-src
         run: pip install -e .[dev,primitives]
 
       - name: Generate example tests
-        working-directory: omnigibson-src
         run: python tests/create_tests_of_examples.py
 
       - name: Get list of generated tests
         id: get-test-list
-        working-directory: omnigibson-src
         run: |
           echo "example_tests=$(cat tests/example_tests.json)" >> $GITHUB_OUTPUT
 
@@ -69,17 +65,14 @@ jobs:
         run: echo "HOME=/root" >> $GITHUB_ENV
 
       - name: Checkout source
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
         with:
           submodules: true
-          path: omnigibson-src
 
       - name: Install
-        working-directory: omnigibson-src
         run: pip install -e .[dev,primitives]
 
       - name: Run tests
-        working-directory: omnigibson-src
         run: pytest -s tests/tests_of_examples/${{ matrix.test_file }}.py --junitxml=${{ matrix.test_file }}.xml && cp ${{ matrix.test_file }}.xml ${GITHUB_WORKSPACE}/
 
       - name: Deploy artifact
@@ -99,20 +92,12 @@ jobs:
         shell: micromamba run -n omnigibson /bin/bash -leo pipefail {0}
     needs: [run_test]
     steps:
-      - name: Checkout source
-        uses: actions/checkout@v2
-        with:
-          submodules: true
-          path: omnigibson-src
       - name: Pull reports
         uses: actions/download-artifact@v3
-        with:
-          path: omnigibson-src
       - name: Example Test Report0
         uses: dorny/test-reporter@v1
         with:
           name: Example Test Results
-          working-directory: omnigibson-src
           path: ${{ github.run_id }}-tests-*/*_test.xml
           reporter: java-junit
           fail-on-error: 'true'

diff --git a/.github/workflows/profiling.yml b/.github/workflows/profiling.yml
@@ -30,10 +30,9 @@ jobs:
         run: echo "HOME=/root" >> $GITHUB_ENV
 
       - name: Checkout source
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: Install
-        working-directory: omnigibson-src
         run: pip install -e .[dev,primitives]
 
       - name: Run performance benchmark

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -45,20 +45,17 @@ jobs:
 
     steps:
       - name: Checkout source
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
         with:
           submodules: true
-          path: omnigibson-src
 
       - name: Install
-        working-directory: omnigibson-src
         run: pip install -e .[dev,primitives] --no-build-isolation
 
       - name: Print env
         run: printenv
 
       - name: Run tests
-        working-directory: omnigibson-src
         run: pytest -s tests/${{ matrix.test_file }}.py --junitxml=${{ matrix.test_file }}.xml && cp ${{ matrix.test_file }}.xml ${GITHUB_WORKSPACE}/
         continue-on-error: true