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

Commit 3496411

Browse files
msoekentcNickolasCassandra Granade
authored
API review June 2022 (#596)
* API review June 2022 * Apply suggestions from code review Co-authored-by: Mariia Mykhailova <mamykhai@microsoft.com> * Apply suggestions from code review Co-authored-by: Cassandra Granade <chgranad@microsoft.com> * Apply suggestions from code review Co-authored-by: Mariia Mykhailova <mamykhai@microsoft.com> Co-authored-by: Cassandra Granade <chgranad@microsoft.com>
1 parent 2117bf2 commit 3496411

File tree

1 file changed

+47
-0
lines changed

1 file changed

+47
-0
lines changed
Lines changed: 47 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,47 @@
1+
# Q# API Design Discussions / June 2022
2+
3+
Reviewers (in order by username): @tcNickolas, @cgranade
4+
5+
## Agenda
6+
7+
- https://github.com/microsoft/QuantumLibraries/issues/593
8+
- https://github.com/microsoft/QuantumLibraries/issues/594
9+
- https://github.com/microsoft/QuantumLibraries/issues/595
10+
11+
## Discussion
12+
13+
### Comparison operations
14+
15+
**Proposal**: https://github.com/microsoft/QuantumLibraries/issues/593
16+
17+
**Reviews**:
18+
- @cgranade, *comment*: Does the `action` input to `ApplyControlledOnLessThanFxP` need to be adjointable? Should there be a separate `Adj` variant? We may also want to describe quickly how this proposal works with the outstanding proposal for numerics refactoring all-up (https://github.com/microsoft/QuantumLibraries/issues/337).
19+
* @tcNickolas, *approve*, left a comment
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**: *approve* subject to comments to be addressed in the issue, discussion will be deferred to issue.
23+
24+
---
25+
26+
### Functions for smallest and largest representable fixed point
27+
28+
**Proposal**: https://github.com/microsoft/QuantumLibraries/issues/594
29+
30+
**Reviews**: @cgranade, *approve*, same comment as above (https://github.com/microsoft/QuantumLibraries/issues/337).
31+
* @tcNickolas, *approve*, left a comment
32+
> 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.
33+
34+
**Consensus**: *approve*
35+
36+
---
37+
38+
### Conversion functions for signed integers
39+
40+
**Proposal**: https://github.com/microsoft/QuantumLibraries/issues/595
41+
42+
**Reviews**:
43+
* @cgranade: *comment*. We should coordinate this with the Q# language discussion around bit vectors in general. @swernli and @bettinaheim in particular may want to provide input here.
44+
* @tcNickolas, *approve*, left a comment
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**: *approve* subject to comments to be addressed in the issue, discussion will be deferred to issue.

0 commit comments

Comments
 (0)