Merge branch 'main' into t-abalaji/update-resource-estimator

Author: msoeken

bootstrap.ps1

@@ -19,7 +19,7 @@ Push-Location (Join-Path $PSScriptRoot "./src/Simulation/qdk_sim_rs")
         --version $Env:NUGET_VERSION;
 Pop-Location
 
-if (-not (Test-Path Env:AGENT_OS)) {                                    # If not CI build, i.e. local build (if AGENT_OS envvar is not defined)
+if (-not (Test-Path Env:/AGENT_OS)) {                                    # If not CI build, i.e. local build (if AGENT_OS envvar is not defined)
     if ($Env:ENABLE_NATIVE -ne "false") {
         Write-Host "Build release flavor of the native simulator"
         $Env:BUILD_CONFIGURATION = "Release"

build/ci-codecheck.yml

@@ -0,0 +1,37 @@
+name: $(date:yyMM).$(DayOfMonth)$(rev:rr)
+
+trigger: none
+
+pr:
+- main
+- feature/*
+- features/*
+- release/*
+
+schedules:
+- cron: "0 9 * * Sat"
+  displayName: 'Build for Component Governance'
+  branches:
+    include:
+    - main
+  always: true
+
+variables:
+  CI: "true"
+
+jobs:
+- job: build
+  displayName: Build
+  strategy:
+    matrix:
+      linux:
+        imageName: 'ubuntu-latest'
+      mac:
+        imageName: 'macOS-latest'
+      #windows:                         # No sanitizers supported on Win at the moment.
+      #  imageName: 'windows-latest'
+  pool: 
+    vmImage: $(imageName)
+
+  steps:
+  - template: steps-codecheck.yml

build/steps-codecheck.yml

@@ -0,0 +1,50 @@
+##
+# Run all build steps.
+##
+steps:
+
+- task: NuGetToolInstaller@0
+  displayName: 'Use NuGet 5.6.0'
+  inputs:
+    versionSpec: '5.6.0'
+ 
+  # QIR Runtime:
+- pwsh: src/Qir/Runtime/prerequisites.ps1
+  displayName: "Install QIR Runtime Prerequisistes"
+  workingDirectory: $(System.DefaultWorkingDirectory)
+
+- pwsh: src/Qir/Runtime/build-qir-runtime.ps1
+  displayName: "Build QIR Runtime"
+  workingDirectory: $(System.DefaultWorkingDirectory)
+
+- pwsh: src/Qir/Runtime/test-qir-runtime.ps1
+  displayName: "Test QIR Runtime"
+  workingDirectory: $(System.DefaultWorkingDirectory)
+
+  # Native Simulator (needed to build and run the QIR tests):
+- pwsh: src/Simulation/Native/prerequisites.ps1
+  displayName: "Install Native Simulator Prerequisites"
+  workingDirectory: $(System.DefaultWorkingDirectory)
+
+- powershell: |
+    .\build-native-simulator.ps1
+  displayName: "Build Native Simulator"
+  workingDirectory: $(System.DefaultWorkingDirectory)/src/Simulation/Native
+
+  # QIR Tests:
+- pwsh: src/Qir/Tests/build-qir-tests.ps1
+  displayName: "Build QIR Tests"
+  workingDirectory: $(System.DefaultWorkingDirectory)
+
+- pwsh: src/Qir/Tests/test-qir-tests.ps1
+  displayName: "Run QIR Tests"
+  workingDirectory: $(System.DefaultWorkingDirectory)
+
+  # QIR Samples:
+- pwsh: src/Qir/Samples/build-qir-samples.ps1
+  displayName: "Build QIR Samples"
+  workingDirectory: $(System.DefaultWorkingDirectory)
+
+- pwsh: src/Qir/Samples/test-qir-samples.ps1
+  displayName: "Run QIR Samples"
+  workingDirectory: $(System.DefaultWorkingDirectory)

src/Azure/Azure.Quantum.Client.Test/Authentication/TokenFileCredentialTests.cs

@@ -49,19 +49,19 @@ public class TokenFileCredentialTests
         [TestInitialize]
         public void TestInitialize()
         {
-            Environment.SetEnvironmentVariable("AZUREQUANTUM_TOKEN_FILE", _tokenFilePath);
+            Environment.SetEnvironmentVariable("AZURE_QUANTUM_TOKEN_FILE", _tokenFilePath);
         }
 
         [TestCleanup]
         public void TestCleanup()
         {
-            Environment.SetEnvironmentVariable("AZUREQUANTUM_TOKEN_FILE", null);
+            Environment.SetEnvironmentVariable("AZURE_QUANTUM_TOKEN_FILE", null);
         }
 
         [TestMethod]
         public void Test_WhenFileNotSet_Throws_CredentialUnavailableException()
         {
-            Environment.SetEnvironmentVariable("AZUREQUANTUM_TOKEN_FILE", null);
+            Environment.SetEnvironmentVariable("AZURE_QUANTUM_TOKEN_FILE", null);
             var credential = new TokenFileCredential();
 
             var exception = Assert.ThrowsException<CredentialUnavailableException>(() =>

src/Azure/Azure.Quantum.Client.Test/WorkspaceTest.cs

@@ -24,10 +24,10 @@ public class WorkspaceTest
     {
         private const string SETUP = @"
 Live tests require you to configure your environment with these variables:
-  * AZUREQUANTUM_WORKSPACE_NAME: the name of an Azure Quantum workspace to use for live testing.
-  * AZUREQUANTUM_SUBSCRIPTION_ID: the Azure Quantum workspace's Subscription Id.
-  * AZUREQUANTUM_WORKSPACE_RG: the Azure Quantum workspace's resource group.
-  * AZUREQUANTUM_WORKSPACE_LOCATION: the Azure Quantum workspace's location (region).
+  * AZURE_QUANTUM_WORKSPACE_NAME: the name of an Azure Quantum workspace to use for live testing.
+  * AZURE_QUANTUM_SUBSCRIPTION_ID: the Azure Quantum workspace's Subscription Id.
+  * AZURE_QUANTUM_WORKSPACE_RG: the Azure Quantum workspace's resource group.
+  * AZURE_QUANTUM_WORKSPACE_LOCATION: the Azure Quantum workspace's location (region).
 
 We'll also try to authenticate with Azure using an instance of DefaultCredential. See
 https://docs.microsoft.com/en-us/dotnet/api/overview/azure/identity-readme#authenticate-the-client
@@ -187,24 +187,24 @@ private static void AssertJob(CloudJob job)
 
         private IWorkspace GetLiveWorkspace()
         {
-            if (string.IsNullOrWhiteSpace(System.Environment.GetEnvironmentVariable("AZUREQUANTUM_SUBSCRIPTION_ID")) ||
-                string.IsNullOrWhiteSpace(System.Environment.GetEnvironmentVariable("AZUREQUANTUM_WORKSPACE_RG")) ||
-                string.IsNullOrWhiteSpace(System.Environment.GetEnvironmentVariable("AZUREQUANTUM_WORKSPACE_NAME")) ||
-                string.IsNullOrWhiteSpace(System.Environment.GetEnvironmentVariable("AZUREQUANTUM_SUBSCRIPTION_ID")))
+            if (string.IsNullOrWhiteSpace(System.Environment.GetEnvironmentVariable("AZURE_QUANTUM_SUBSCRIPTION_ID")) ||
+                string.IsNullOrWhiteSpace(System.Environment.GetEnvironmentVariable("AZURE_QUANTUM_WORKSPACE_RG")) ||
+                string.IsNullOrWhiteSpace(System.Environment.GetEnvironmentVariable("AZURE_QUANTUM_WORKSPACE_NAME")) ||
+                string.IsNullOrWhiteSpace(System.Environment.GetEnvironmentVariable("AZURE_QUANTUM_SUBSCRIPTION_ID")))
             {
                 Assert.Inconclusive(SETUP);
             }
 
             var options = new QuantumJobClientOptions();
-            options.Diagnostics.ApplicationId = "ClientTests";
+            options.Diagnostics.ApplicationId = Environment.GetEnvironmentVariable("AZURE_QUANTUM_NET_APPID") ?? "ClientTests";
 
             var credential = Authentication.CredentialFactory.CreateCredential(Authentication.CredentialType.Default);
 
             return new Workspace(
-                subscriptionId: System.Environment.GetEnvironmentVariable("AZUREQUANTUM_SUBSCRIPTION_ID"),
-                resourceGroupName: System.Environment.GetEnvironmentVariable("AZUREQUANTUM_WORKSPACE_RG"),
-                workspaceName: System.Environment.GetEnvironmentVariable("AZUREQUANTUM_WORKSPACE_NAME"),
-                location: System.Environment.GetEnvironmentVariable("AZUREQUANTUM_WORKSPACE_LOCATION"),
+                subscriptionId: System.Environment.GetEnvironmentVariable("AZURE_QUANTUM_SUBSCRIPTION_ID"),
+                resourceGroupName: System.Environment.GetEnvironmentVariable("AZURE_QUANTUM_WORKSPACE_RG"),
+                workspaceName: System.Environment.GetEnvironmentVariable("AZURE_QUANTUM_WORKSPACE_NAME"),
+                location: System.Environment.GetEnvironmentVariable("AZURE_QUANTUM_WORKSPACE_LOCATION"),
                 options: options,
                 credential: credential);
         }

src/Azure/Azure.Quantum.Client/Authentication/TokenFileCredential.cs

@@ -17,7 +17,7 @@ namespace Microsoft.Azure.Quantum.Authentication
     /// <summary>
     /// Implements a custom TokenCredential to use a local file as the source for an AzureQuantum token.
     ///
-    /// It will only use the local file if the <c>AZUREQUANTUM_TOKEN_FILE</c> environment variable is set, and references
+    /// It will only use the local file if the <c>AZURE_QUANTUM_TOKEN_FILE</c> environment variable is set, and references
     /// an existing json file that contains the access_token and expires_on timestamp in milliseconds.
     ///
     /// If the environment variable is not set, the file does not exist, or the token is invalid in any way(expired, for example),
@@ -28,7 +28,7 @@ public class TokenFileCredential : TokenCredential
         /// <summary>
         /// Environment variable name for the token file path.
         /// </summary>
-        private const string TokenFileEnvironmentVariable = "AZUREQUANTUM_TOKEN_FILE";
+        private const string TokenFileEnvironmentVariable = "AZURE_QUANTUM_TOKEN_FILE";
 
         /// <summary>
         /// File system dependency injected so that unit testing is possible.

src/Azure/Azure.Quantum.Client/JobManagement/Workspace.cs

@@ -51,6 +51,8 @@ public Workspace(
             // Optional parameters:
             credential ??= CredentialFactory.CreateCredential(CredentialType.Default, subscriptionId);
             options ??= new QuantumJobClientOptions();
+            options.Diagnostics.ApplicationId = options.Diagnostics.ApplicationId
+                                                ?? Environment.GetEnvironmentVariable("AZURE_QUANTUM_NET_APPID");
 
             this.ResourceGroupName = resourceGroupName;
             this.WorkspaceName = workspaceName;

src/Qir/.clang-tidy

@@ -1,5 +1,11 @@
 Checks: 
-  '-*,bugprone-*,-readability-*,readability-identifier-*,readability-braces-around-statements'
+  'bugprone-*,readability-identifier-*,readability-braces-around-statements,cert*,\
+  -llvmlibc-callee-namespace,-llvmlibc-implementation-in-namespace,\
+  -llvmlibc-restrict-system-libc-headers,-modernize-use-trailing-return-type,\
+  -fuchsia-default-arguments-calls,-fuchsia-default-arguments-declarations,
+  -google-readability-casting'
+# TODO(rokuzmin): '*, . . .'
+
 WarningsAsErrors: '*'
 HeaderFilterRegex: '.*'
 

src/Qir/CommandLineTool/Program.cs

@@ -42,10 +42,14 @@ private static Command CreateBuildCommand()
         {
             var buildCommand = new Command("build", "(default) Build the executables from a QIR DLL.")
             {
-                Handler = CommandHandler.Create((BuildOptions settings) =>
-                    QirTools.BuildFromQSharpDll(settings.QSharpDll, settings.LibraryDirectories, settings.IncludeDirectories, settings.ExecutablesDirectory))
+                Handler = CommandHandler.Create((BuildOptions settings) => QirTools.BuildFromQSharpDll(
+                    settings.QSharpDll,
+                    settings.LibraryDirectories,
+                    settings.IncludeDirectories,
+                    settings.ExecutablesDirectory,
+                    settings.Debug)),
+                TreatUnmatchedTokensAsErrors = true
             };
-            buildCommand.TreatUnmatchedTokensAsErrors = true;
 
             buildCommand.AddOption(new Option<FileInfo>(
                 aliases: new string[] { "--qsharp-dll", "--dll" },
@@ -83,6 +87,10 @@ private static Command CreateBuildCommand()
                 Required = true
             });
 
+            buildCommand.AddOption(new Option<bool>(
+                alias: "--debug",
+                description: "Enable additional debugging checks at runtime."));
+
             return buildCommand;
         }
 
@@ -145,6 +153,11 @@ public sealed class BuildOptions
             /// The path to the output directory where the created executables will be placed.
             /// </summary>
             public DirectoryInfo ExecutablesDirectory { get; set; }
+
+            /// <summary>
+            /// Enable additional debugging checks at runtime.
+            /// </summary>
+            public bool Debug { get; set; }
         }
     }
 }