Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reference RTCP subsections from tables #80

Closed
mengelbart opened this issue Apr 27, 2023 · 3 comments · Fixed by #120
Closed

Reference RTCP subsections from tables #80

mengelbart opened this issue Apr 27, 2023 · 3 comments · Fixed by #120
Assignees
Labels
NextInterim Targeting PR for next interim meeting

Comments

@mengelbart
Copy link
Owner

The tables in sections 6.5 - 6.8 contain a lot of information which is also explained in more detail in section 6.3. I think we can reference the relevant subsections from the tables instead.

@SpencerDawkins
Copy link
Collaborator

@mengelbart - I agree that putting all of the material relating to a specific feedback mechanism in one place, and referring to it rather than adding text elsewhere makes sense. Here's what I was thinking about while working on PR #71.

  • some material is duplicated (in text and in tables),
  • some material is split (between text in tables),

That's roughly this issue. But, in addition,

  • It makes sense to me that we would have a list of what can be mapped between approved QUIC (RFC 9000 + RFC 9221) and RTCP, and a separate list of what could be mapped if we had one of the timestamps drafts (to help people understand why we need one of those drafts). Does that make sense to you?
  • I made the (not necessarily correct) decision to list everything that we looked at, whether it can be mapped between QUIC and RTCP or not, so we have a lot of table entries that say "no", even if some entries in the same table say "yes", and that's begging for us to also document why we think the answer is "no" (again, should we be adding that?)

It might make sense to move this second group of possible changes into its own, but (especially if we decide that spending a lot of effort on "no" entries isn't worth doing), we might be making to changes to text/tables/rows that we'll end up deleting from the document.

Do you have thoughts about that? Should we have a short Slackathon to discuss?

@mengelbart
Copy link
Owner Author

mengelbart commented May 2, 2023

It makes sense to me that we would have a list of what can be mapped between approved QUIC (RFC 9000 + RFC 9221) and RTCP, and a separate list of what could be mapped if we had one of the timestamps drafts (to help people understand why we need one of those drafts). Does that make sense to you?

I agree that having a list of what could be mapped with or without a timestamp extension would be helpful. I am not sure if that should be a separate table or if it would be enough to have an extra column for that.

I made the (not necessarily correct) decision to list everything that we looked at, whether it can be mapped between QUIC and RTCP or not, so we have a lot of table entries that say "no", even if some entries in the same table say "yes", and that's begging for us to also document why we think the answer is "no" (again, should we be adding that?)

That sounds good, too. I assume that the explanation would be the same for many of the fields so maybe we can do something similar to what you did in the topologies section, so we can reference the explanation/notes/subsection from all rows in the table to which it applies?

@SpencerDawkins
Copy link
Collaborator

We may want to move some of these sections into an appendix ("we looked at these and don't think they're useful, so other people don't have to wonder if we looked at them").

@SpencerDawkins SpencerDawkins self-assigned this Jun 16, 2023
@SpencerDawkins SpencerDawkins added the IETF118 Targeting PR by IETF 118 label Jun 16, 2023
@SpencerDawkins SpencerDawkins added NextInterim Targeting PR for next interim meeting and removed IETF118 Targeting PR by IETF 118 labels Jul 27, 2023
@mengelbart mengelbart linked a pull request Aug 29, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NextInterim Targeting PR for next interim meeting
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants