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

API review March 2022 #535

Merged
merged 7 commits into from
Apr 4, 2022
Merged

API review March 2022 #535

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
6694867
API review meeting notes.
msoeken Mar 2, 2022
93f1f05
Merge branch 'main' into msoeken/api-review-2203
msoeken Mar 3, 2022
03d4d7b
Merge branch 'main' into msoeken/api-review-2203
msoeken Mar 22, 2022
08ca545
Update Design/meetings/2022/api-design-2022-03.md
msoeken Mar 22, 2022
105ac3b
Apply suggestions from code review
msoeken Mar 22, 2022
c589b24
Merge branch 'main' into msoeken/api-review-2203
msoeken Apr 4, 2022
9e10368
Update consensus.
msoeken Apr 4, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
59 changes: 59 additions & 0 deletions Design/meetings/2022/api-design-2022-03.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,59 @@
# Q# API Design Discussions / March 2022

Reviewers (in order by username): .

## Agenda

- https://github.com/microsoft/QuantumLibraries/issues/471
- https://github.com/microsoft/QuantumLibraries/issues/442
- https://github.com/microsoft/QuantumLibraries/issues/423
- https://github.com/microsoft/QuantumLibraries/issues/515

## Discussion

### Multivariate stochastic optimization

**Proposal**: https://github.com/microsoft/QuantumLibraries/issues/471

**Reviews**:
* @msoeken, *approve*
> Please add a bullet point including your alias, your review result (*approve*, *reject*, *comment*), and a comment (optional when result is *approve*). Alternatively, add a line to the PR discussion incl. a reference to this issue.

**Consensus**: Approved (incl. implicit approval from proposal author)

---

### New functions for singly controlled operations

**Proposal**: https://github.com/microsoft/QuantumLibraries/issues/442

**Reviews**:
* @tcNickolas, *approve*, left a couple of small comments in the issue
* @msoeken, *approve*
> Please add a bullet point including your alias, your review result (*approve*, *reject*, *comment*), and a comment (optional when result is *approve*). Alternatively, add a line to the PR discussion incl. a reference to this issue.

**Consensus**: Approved (incl. implicit approval from proposal author)

---

### Simple and consistent arithmetic API

**Proposal**: https://github.com/microsoft/QuantumLibraries/issues/423

**Reviews**:

> Please add a bullet point including your alias, your review result (*approve*, *reject*, *comment*), and a comment (optional when result is *approve*). Alternatively, add a line to the PR discussion incl. a reference to this issue.

**Consensus**: Postponed (no additional review was provided)

---

### New operation: AmplitudeTransduction

**Proposal**: https://github.com/microsoft/QuantumLibraries/issues/515

**Reviews**:
@msoeken, *approve*
> Please add a bullet point including your alias, your review result (*approve*, *reject*, *comment*), and a comment (optional when result is *approve*). Alternatively, add a line to the PR discussion incl. a reference to this issue.

**Consensus**: Approved (incl. implicit approval from proposal author)