Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Conversation

@swernli
Copy link
Collaborator

@swernli swernli commented Oct 13, 2020

The e2e build failures were due to the project path changes introduced by the feature work. This reverts the Simulation\Simulators.Core folder to Simulation\Simulators to fix e2e build.

This change is mostly file renames and path fixes, and the only change to functionality was a bug fix in MResetY to add the adjoint functor when calling S.

ricardo-espinoza and others added 5 commits September 22, 2020 19:42
…ionable information (#380)

Improving the logging of WorkspaceClientException to include more informative data in its message from its inner exception. This is a follow up to #378 .

- Instead of providing the full stack of the exception in the output, we extract more relevant fields from the source RestErrorException and provide them to the user.
- Also, this change exposes header "x-ms-request-id"if present in the response, for additional debugging.
@swernli
Copy link
Collaborator Author

swernli commented Oct 14, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@anpaz
Copy link
Member

anpaz commented Oct 14, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

Copy link
Contributor

@ScottCarda-MS ScottCarda-MS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@swernli
Copy link
Collaborator Author

swernli commented Oct 14, 2020

There are some failures in iqsharp python tests that I think will require some work to fix. I'd like to get this PR merged up to the feature branch and tackle those in a separate one to make that work easier to review in isolation.

@swernli swernli merged commit ac3a622 into feature/decomp Oct 14, 2020
@swernli swernli deleted the swernli/decomp-project-rename branch October 14, 2020 21:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants