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

Commit 9ea44b5

Browse files
msoekentcNickolas
andauthored
API review March 2022 (#535)
* API review meeting notes. * Update Design/meetings/2022/api-design-2022-03.md Co-authored-by: Mariia Mykhailova <[email protected]> * Apply suggestions from code review * Update consensus. Co-authored-by: Mariia Mykhailova <[email protected]>
1 parent 2a892a9 commit 9ea44b5

File tree

1 file changed

+59
-0
lines changed

1 file changed

+59
-0
lines changed
Lines changed: 59 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,59 @@
1+
# Q# API Design Discussions / March 2022
2+
3+
Reviewers (in order by username): .
4+
5+
## Agenda
6+
7+
- https://github.com/microsoft/QuantumLibraries/issues/471
8+
- https://github.com/microsoft/QuantumLibraries/issues/442
9+
- https://github.com/microsoft/QuantumLibraries/issues/423
10+
- https://github.com/microsoft/QuantumLibraries/issues/515
11+
12+
## Discussion
13+
14+
### Multivariate stochastic optimization
15+
16+
**Proposal**: https://github.com/microsoft/QuantumLibraries/issues/471
17+
18+
**Reviews**:
19+
* @msoeken, *approve*
20+
> 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.
21+
22+
**Consensus**: Approved (incl. implicit approval from proposal author)
23+
24+
---
25+
26+
### New functions for singly controlled operations
27+
28+
**Proposal**: https://github.com/microsoft/QuantumLibraries/issues/442
29+
30+
**Reviews**:
31+
* @tcNickolas, *approve*, left a couple of small comments in the issue
32+
* @msoeken, *approve*
33+
> 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.
34+
35+
**Consensus**: Approved (incl. implicit approval from proposal author)
36+
37+
---
38+
39+
### Simple and consistent arithmetic API
40+
41+
**Proposal**: https://github.com/microsoft/QuantumLibraries/issues/423
42+
43+
**Reviews**:
44+
45+
> 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.
46+
47+
**Consensus**: Postponed (no additional review was provided)
48+
49+
---
50+
51+
### New operation: AmplitudeTransduction
52+
53+
**Proposal**: https://github.com/microsoft/QuantumLibraries/issues/515
54+
55+
**Reviews**:
56+
@msoeken, *approve*
57+
> 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.
58+
59+
**Consensus**: Approved (incl. implicit approval from proposal author)

0 commit comments

Comments
 (0)