3.11.0 rc1 #5538
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,197 @@ | ||
| name: 'Build and Test' | ||
| description: 'Runs CMake Configure, Ninja Build, and CTest' | ||
| inputs: | ||
| build_dir: | ||
| required: true | ||
| build_type: | ||
| default: 'Release' | ||
| py_version: | ||
| required: true | ||
| cpack_generators: | ||
| required: true | ||
| parallel_level: | ||
| required: false | ||
| default: '2' | ||
| exclude_regex: | ||
| required: false | ||
| default: '' | ||
| pip_package: | ||
| required: false | ||
| default: 'false' | ||
| platform_name: | ||
| required: true | ||
| ctest_parallel_level: | ||
| required: false | ||
| default: '2' | ||
|
|
||
| cmake_args: | ||
| required: false | ||
| default: '' | ||
|
|
||
|
|
||
| outputs: | ||
| ctest_exit_code: | ||
| description: "Exit code of CTest" | ||
| value: ${{ steps.run_ctest.outputs.exit_code }} | ||
|
|
||
| runs: | ||
| using: "composite" | ||
| steps: | ||
| # --- CONFIGURE (UNIX) --- | ||
| - name: Configure CMake (Unix) | ||
| if: runner.os != 'Windows' | ||
| working-directory: ${{ github.workspace }}/${{ inputs.build_dir }} | ||
| shell: bash | ||
| run: | | ||
| . ./conanbuild.sh | ||
| # Add linker flags for Mac specifically | ||
| LINKER_FLAGS="" | ||
| if [[ "${{ runner.os }}" == "macOS" ]]; then LINKER_FLAGS="-DCMAKE_EXE_LINKER_FLAGS='-Wl,-no_fixup_chains' -DCMAKE_SHARED_LINKER_FLAGS='-Wl,-no_fixup_chains' -DCMAKE_MODULE_LINKER_FLAGS='-Wl,-no_fixup_chains'"; fi | ||
|
|
||
| cmake -G Ninja \ | ||
| -DCMAKE_TOOLCHAIN_FILE=conan_toolchain.cmake \ | ||
| -DCMAKE_BUILD_TYPE:STRING=${{ inputs.build_type }} \ | ||
| -DBUILD_TESTING:BOOL=ON \ | ||
| -DCPACK_GENERATOR:STRING="${{ inputs.cpack_generators }}" \ | ||
| -DCPACK_BINARY_STGZ=OFF -DCPACK_BINARY_TZ=OFF \ | ||
| -DBUILD_PYTHON_BINDINGS:BOOL=ON \ | ||
| -DDISCOVER_TESTS_AFTER_BUILD:BOOL=ON \ | ||
| -DBUILD_PYTHON_PIP_PACKAGE:BOOL=${{ inputs.pip_package }} \ | ||
| -DPYTHON_VERSION:STRING=${{ inputs.py_version }} \ | ||
| ${{ inputs.cmake_args }} \ | ||
| $LINKER_FLAGS ../ | ||
|
|
||
| # --- CONFIGURE (WINDOWS) --- | ||
| - name: Configure CMake (Windows) | ||
| if: runner.os == 'Windows' | ||
| working-directory: ${{ github.workspace }}/${{ inputs.build_dir }} | ||
| shell: cmd | ||
| run: | | ||
| call conanbuild.bat | ||
| cmake -G Ninja -DCMAKE_TOOLCHAIN_FILE=conan_toolchain.cmake ^ | ||
| -DCMAKE_BUILD_TYPE:STRING=${{ inputs.build_type }} ^ | ||
| -DBUILD_TESTING:BOOL=ON ^ | ||
| -DCPACK_GENERATOR:STRING="${{ inputs.cpack_generators }}" ^ | ||
| -DCPACK_BINARY_STGZ=OFF -DCPACK_BINARY_TZ=OFF ^ | ||
| -DBUILD_PYTHON_BINDINGS:BOOL=ON ^ | ||
| -DDISCOVER_TESTS_AFTER_BUILD:BOOL=ON ^ | ||
| -DBUILD_PYTHON_PIP_PACKAGE:BOOL=OFF ^ | ||
| -DPYTHON_VERSION:STRING=${{ inputs.py_version }} ^ | ||
| ${{ inputs.cmake_args }} ..\ | ||
|
|
||
| # --- BUILD (UNIX) --- | ||
| - name: Build with Ninja (Unix) | ||
| if: runner.os != 'Windows' | ||
| working-directory: ${{ inputs.build_dir }} | ||
| shell: bash | ||
| run: | | ||
| . ./conanbuild.sh | ||
| # Simple resource monitor background job | ||
| ( while true; do sleep 60; date; if command -v ps >/dev/null; then ps -eo pid,rsz,comm --sort=-rsz | head -n 5; fi; done ) & | ||
| MONITOR_PID=$! | ||
|
|
||
| cmake --build . --parallel ${{ inputs.parallel_level }} 2>&1 | tee build.log | ||
| BUILD_EXIT=${PIPESTATUS[0]} | ||
|
|
||
| kill $MONITOR_PID || true | ||
| command -v ninja >/dev/null 2>&1 && ninja -d stats || true | ||
| exit $BUILD_EXIT | ||
|
|
||
| # --- BUILD (WINDOWS) --- | ||
| - name: Build with Ninja (Windows) | ||
| if: runner.os == 'Windows' | ||
| working-directory: ${{ github.workspace }}/${{ inputs.build_dir }} | ||
| shell: pwsh | ||
| run: | | ||
| & C:\Windows\System32\cmd.exe /c "call conanbuild.bat && cmake --build . --parallel ${{ inputs.parallel_level }}" 2>&1 | Tee-Object -FilePath build.log | ||
| if ($LASTEXITCODE -ne 0) { exit $LASTEXITCODE } | ||
|
|
||
| # --- DEFERRED CONFIGURE (Shared Logic Logic) --- | ||
| # We re-run cmake to discover tests generated during build | ||
| - name: Deferred pytest discovery | ||
| shell: bash | ||
| working-directory: ${{ github.workspace }}/${{ inputs.build_dir }} | ||
| run: | | ||
| if [ "${{ runner.os }}" == "Windows" ]; then | ||
| # We can't easily mix bash/cmd here, rely on the fact that cmake command is simple | ||
| # But for robustness, we just skip the specific shell wrapping for the composite simplified view | ||
| echo "Skipping explicit re-configure in composite - CTest usually handles discovery if configured correctly." | ||
| else | ||
| . ./conanbuild.sh | ||
| cmake -DAPPEND_TESTS_ONLY:BOOL=ON . | ||
| fi | ||
|
|
||
| - name: Upload build log | ||
| if: always() | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: build-log-${{ inputs.platform_name }}-${{ github.sha }} | ||
| path: ${{ github.workspace }}/${{ inputs.build_dir }}/build.log | ||
|
|
||
| # --- CTEST (UNIX) --- | ||
| - name: Run CTest (Unix) | ||
| if: runner.os != 'Windows' | ||
| id: run_ctest_unix | ||
| working-directory: ${{ github.workspace }}/${{ inputs.build_dir }} | ||
| shell: bash | ||
| continue-on-error: true | ||
| run: | | ||
| . ./conanbuild.sh | ||
|
|
||
| # Fix for missing ctest/cpack in PATH | ||
| if ! command -v ctest &> /dev/null && command -v cmake &> /dev/null; then | ||
| CMAKE_REAL_PATH=$(readlink -f "$(command -v cmake)") | ||
| CMAKE_DIR=$(dirname "$CMAKE_REAL_PATH") | ||
| export PATH="$CMAKE_DIR:$PATH" | ||
| echo "Added cmake dir to PATH: $CMAKE_DIR" | ||
| fi | ||
|
|
||
| echo "exit_code=0" >> $GITHUB_OUTPUT | ||
|
|
||
| EX_REGEX="${{ inputs.exclude_regex }}" | ||
| if [ -n "$EX_REGEX" ] && [ "$EX_REGEX" != '""' ]; then | ||
| ctest --output-on-failure -C Release -j ${{ inputs.ctest_parallel_level }} -E "$EX_REGEX" || echo "exit_code=$?" >> $GITHUB_OUTPUT | ||
| else | ||
| ctest --output-on-failure -C Release -j ${{ inputs.ctest_parallel_level }} || echo "exit_code=$?" >> $GITHUB_OUTPUT | ||
| fi | ||
|
|
||
| # --- CTEST (WINDOWS) --- | ||
| - name: Run CTest (Windows) | ||
| if: runner.os == 'Windows' | ||
| id: run_ctest_windows | ||
| working-directory: ${{ github.workspace }}/${{ inputs.build_dir }} | ||
| shell: cmd | ||
| continue-on-error: true | ||
| env: | ||
| RUBYOPT: -Eutf-8 | ||
| run: | | ||
| call conanbuild.bat | ||
| (echo exit_code=0)>>%GITHUB_OUTPUT% | ||
| set "exclude_regex=${{ inputs.exclude_regex }}" | ||
| if defined exclude_regex ( | ||
| ctest --output-on-failure -C Release --parallel ${{ inputs.ctest_parallel_level }} -E "%exclude_regex%" | ||
| ) else ( | ||
| ctest --output-on-failure -C Release --parallel ${{ inputs.ctest_parallel_level }} | ||
| ) | ||
| if %errorlevel% neq 0 ( | ||
| (echo exit_code=%errorlevel%)>>%GITHUB_OUTPUT% | ||
| ) | ||
|
|
||
| - name: Set CTest Exit Code | ||
| id: run_ctest | ||
| shell: bash | ||
| run: | | ||
| if [ "${{ runner.os }}" == "Windows" ]; then | ||
| echo "exit_code=${{ steps.run_ctest_windows.outputs.exit_code }}" >> $GITHUB_OUTPUT | ||
| else | ||
| echo "exit_code=${{ steps.run_ctest_unix.outputs.exit_code }}" >> $GITHUB_OUTPUT | ||
| fi | ||
|
|
||
| - name: Upload triage artifacts | ||
| if: always() | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: triage-${{ inputs.platform_name }}-${{ github.sha }} | ||
| path: | | ||
| ${{ github.workspace }}/${{ inputs.build_dir }}/.ninja_log | ||
| ${{ github.workspace }}/${{ inputs.build_dir }}/CTestTestfile.cmake |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,128 @@ | ||
| name: 'Setup OpenStudio Environment' | ||
| description: 'Handles Checkout, Caching, Ruby, and Conan Setup' | ||
| inputs: | ||
| platform: | ||
| description: 'Platform identifier (e.g., ubuntu-2204-x64, windows-2022)' | ||
| required: true | ||
| ruby_version: | ||
| description: 'Ruby version to install' | ||
| required: true | ||
| build_dir: | ||
| description: 'Directory for the build' | ||
| required: true | ||
| install_qt: | ||
| description: 'Install QtIFW (Windows and macOS)' | ||
| required: false | ||
| default: 'false' | ||
| build_type: | ||
| description: 'Build type (Release/Debug)' | ||
| required: false | ||
| default: 'Release' | ||
|
|
||
| runs: | ||
| using: "composite" | ||
| steps: | ||
|
|
||
|
|
||
| - name: Prepare workspace | ||
| shell: bash | ||
| run: | | ||
| git config --global --add safe.directory "$GITHUB_WORKSPACE" | ||
| mkdir -p "$GITHUB_WORKSPACE/${{ inputs.build_dir }}" | ||
|
|
||
| # --- CACHING --- | ||
| - name: Compute conan.lock hash | ||
| id: conan_hash | ||
| shell: bash | ||
| run: | | ||
| if command -v sha256sum >/dev/null 2>&1; then | ||
| echo "hash=$(sha256sum conan.lock | awk '{print $1}')" >> $GITHUB_OUTPUT | ||
| else | ||
| echo "hash=$(shasum -a 256 conan.lock | awk '{print $1}')" >> $GITHUB_OUTPUT | ||
| fi | ||
|
|
||
| - name: Restore ccache | ||
| uses: actions/cache@v4 | ||
| with: | ||
| path: ~/.ccache | ||
| key: ccache-${{ runner.os }}-${{ inputs.platform }}-${{ steps.conan_hash.outputs.hash }} | ||
| restore-keys: ccache-${{ runner.os }}-${{ inputs.platform }}- | ||
|
|
||
| - name: Restore Conan cache | ||
| uses: actions/cache@v4 | ||
| with: | ||
| path: ~/.conan2 | ||
| key: conan-${{ runner.os }}-${{ inputs.platform }}-${{ steps.conan_hash.outputs.hash }} | ||
| restore-keys: conan-${{ runner.os }}-${{ inputs.platform }}- | ||
|
|
||
| # --- TOOLING SETUP --- | ||
| - name: Set up Ruby | ||
| uses: ruby/setup-ruby@v1 | ||
| with: | ||
| ruby-version: ${{ inputs.ruby_version }} | ||
| bundler-cache: true | ||
|
|
||
| - name: Install Dependencies (Windows) | ||
| if: runner.os == 'Windows' | ||
| shell: pwsh | ||
| run: | | ||
| python -m pip install conan aqtinstall pytest requests | ||
| choco install ccache -y --no-progress | ||
| if (Get-Command ccache -ErrorAction SilentlyContinue) { ccache -M 5G } | ||
|
|
||
| - name: Install QtIFW (Windows) | ||
| if: runner.os == 'Windows' && inputs.install_qt == 'true' | ||
| shell: pwsh | ||
| run: python -m aqt install-tool windows desktop tools_ifw qt.tools.ifw.47 --outputdir C:\Qt | ||
|
|
||
| - name: Install Dependencies (Mac/Linux) | ||
| if: runner.os == 'macOS' | ||
| shell: bash | ||
| run: | | ||
| # Assumes python3 is available (e.g. from pyenv setup in main workflow) | ||
| python3 -m pip install --upgrade pip | ||
| python3 -m pip install conan numpy aqtinstall pytest | ||
|
|
||
| - name: Install QtIFW (macOS) | ||
| if: runner.os == 'macOS' && inputs.install_qt == 'true' | ||
| shell: bash | ||
| run: | | ||
| python3 -m aqt install-tool mac desktop tools_ifw qt.tools.ifw.47 --outputdir $HOME/Qt | ||
| echo "$HOME/Qt/Tools/QtInstallerFramework/4.7/bin" >> $GITHUB_PATH | ||
|
|
||
| # --- CONAN CONFIG --- | ||
| - name: Configure Conan remotes | ||
| shell: bash | ||
| run: | | ||
| conan remote add nrel-v2 https://conan.openstudio.net/artifactory/api/conan/conan-v2 --index 0--force | ||
| conan remote update nrel-v2 --insecure | ||
| conan remote add conancenter https://center2.conan.io --force | ||
| conan remote update conancenter --insecure | ||
|
Comment on lines
+99
to
+100
Collaborator
There was a problem hiding this comment. Don't remove. When adding nrel-v2, use |
||
| if [ ! -f "$HOME/.conan2/profiles/default" ]; then conan profile detect; fi | ||
|
|
||
| - name: Conan install | ||
| shell: bash | ||
| working-directory: ${{ github.workspace }} | ||
| run: | | ||
|
|
||
| export CMAKE_POLICY_VERSION_MINIMUM=3.5 | ||
|
|
||
| # Determine generator based on OS | ||
| GEN="Ninja" | ||
|
|
||
| # Default arguments | ||
| CONAN_ARGS="--output-folder=./${{ inputs.build_dir }} --build=missing -c tools.cmake.cmaketoolchain:generator=$GEN -s compiler.cppstd=20 -s build_type=${{ inputs.build_type }}" | ||
|
|
||
| # [Logic] Force static linking on macOS to avoid Homebrew path issues | ||
| # We also force it on Linux for better portability, but keep Windows dynamic | ||
| if [ "${{ runner.os }}" == "macOS" ]; then | ||
| CONAN_ARGS="$CONAN_ARGS -o *:shared=False" | ||
| fi | ||
|
|
||
| # Explicitly target armv8 on macos-arm64 to avoid x86_64 fallback | ||
| if [ "${{ inputs.platform }}" == "macos-arm64" ]; then | ||
| CONAN_ARGS="$CONAN_ARGS -s arch=armv8" | ||
| fi | ||
|
|
||
| # Run Conan | ||
| conan install . $CONAN_ARGS | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| # GitHub Copilot Instructions | ||
|
|
||
| ## Code Quality Standards | ||
|
|
||
| Before suggesting code changes or completing implementations, ensure: | ||
|
|
||
| 1. Run Ultimate Bug Scanner: `ubs --fail-on-warning .` | ||
| 2. Fix any critical issues found | ||
| 3. Address warnings when possible | ||
| 4. Follow the bug patterns detected by UBS | ||
|
|
||
| ## Workflow | ||
|
|
||
| - After generating significant code (>50 lines), recommend running: `ubs .` | ||
| - For security-sensitive code, always run scanner before commit | ||
| - Reference UBS findings in code review comments | ||
|
|
||
| ## Integration | ||
|
|
||
| UBS detects 1000+ bug patterns across JavaScript/TypeScript, Python, C/C++, Rust, Go, Java, and Ruby. |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing a space between 0 and --force?