Skip to content
Snippets Groups Projects

Add GPU tests and conda env build tests

Merged Add GPU tests and conda env build tests
tp_test_create_conda into dev
All threads resolved!
Merged Minh Tuan Pham requested to merge tp_test_create_conda into dev
All threads resolved!

This MR attempts to add (1) similar tests as well have always had but run on GPU and (2) a test which ensure that the “create_conda_environment.sh” script can run and create an environment which work

  • add these new tests in test gitlab-ci yaml file
  • modify check_acorn.py to only print out the number of GPU if GPU is available. Otherwise it will report an error. Test (2) would fail if the machine doesn’t have GPU cards.

must be merged after !152 (merged)

Edited by Minh Tuan Pham

Merge request reports

Merge request pipeline #10873626 passed with warnings

Stage: linting
    Stage: build
      Stage: test

        Merge request pipeline passed with warnings for 9c5ee8ac

        Test coverage 32.00% (0.00%) from 2 jobs
        Approved by
        Jay Chan Xiangyang Ju

        Merged by Xiangyang JuXiangyang Ju 1 month ago (Feb 25, 2025 4:21am UTC)

        Merge details

        • Changes merged into dev with 96781690 (commits were squashed).
        • Deleted the source branch.
        • Auto-merge enabled

        Pipeline #10873725 passed with warnings

        Stage: linting
          Stage: build
            Stage: test
              Stage: deploy

                Pipeline passed with warnings for 96781690 on dev

                Test coverage 32.00% (0.00%) from 2 jobs

                Activity

                Filter activity
                • Approvals
                • Assignees & reviewers
                • Comments (from bots)
                • Comments (from users)
                • Commits & branches
                • Edits
                • Labels
                • Lock status
                • Mentions
                • Merge request status
                • Tracking
              • Sylvain Caillou
              • Minh Tuan Pham added 8 commits

                added 8 commits

                Compare with previous version

              • Minh Tuan Pham requested review from @scaillou, @jburzyns, @dmurnane, and @xju

                requested review from @scaillou, @jburzyns, @dmurnane, and @xju

              • Minh Tuan Pham resolved all threads

                resolved all threads

              • Minh Tuan Pham marked this merge request as ready

                marked this merge request as ready

              • Minh Tuan Pham resolved all threads

                resolved all threads

              • Minh Tuan Pham resolved all threads

                resolved all threads

              • Minh Tuan Pham
                Minh Tuan Pham @tupham ·
                Author Owner

                This MR is ready to be merged. Please take a look and approve when you are happy with it @scaillou @xju @dmurnane @jburzyns

              • Xiangyang Ju approved this merge request

                approved this merge request

              • Xiangyang Ju requested review from @cchan

                requested review from @cchan

              • Xiangyang Ju
                Xiangyang Ju @xju ·
                Owner

                It looks good to me. Approved.

              • Jay Chan approved this merge request

                approved this merge request

              • Jay Chan
                Jay Chan @cchan ·
                Maintainer

                @tupham please rebase the branch

              • Xiangyang Ju added 23 commits

                added 23 commits

                Compare with previous version

              • Xiangyang Ju enabled an automatic merge when all merge checks for 9c5ee8ac pass

                enabled an automatic merge when all merge checks for 9c5ee8ac pass

              • Xiangyang Ju merged

                merged

              • Xiangyang Ju mentioned in commit 96781690

                mentioned in commit 96781690

                • Please register or sign in to reply