Conversation
a42f7e4 to
d6aa578
Compare
9eaaf3c to
cfad3f1
Compare
| } | ||
|
|
||
| // Describes the content of an audio as text. | ||
| message Transcript { |
There was a problem hiding this comment.
Maybe Transcript is unnecessary, as it only wrapped in OperationResult? Instead the Operation could include repeated OperationResult which could have ResultType instead of TranscriptType. This would make the result tree shallower and thus easier to understand.
There was a problem hiding this comment.
I added the OperationResult as it is expected that there will be result types that are sufficiently different from Transcript in the near future, so they can be added with only additive changes.
There was a problem hiding this comment.
that would be possible also by extending the OperationResult itself, it could be just
message OperationResult {
enum ResultType {
...
RESULT_TYPE_TRANSCRIPT_LEXICAL = 1;
..
}
ResultType type = 1;
string text = 2;
repeated Token tokens = 3;
// future additions here
// repeated Event events = 4;
}
This would collect the result alternatives in the same type, instead of an additional intermediate container type.
52c415e to
9b3e657
Compare
No description provided.