irtf-nwcrg / draft-irtf-nwcrg-network-coding-satellites

Network coding and satellites
0 stars 1 forks source link

Stuart Card comment #42

Closed NicoKos closed 5 years ago

NicoKos commented 5 years ago

Abstract p 1 "cope from" -> "cope with"

S 1 Introduction p 2 "cope from" -> "cope with"

S 1 Introduction p 2 "link layers" -> "link layer"

S 1 Introduction p 3 "an non-negligible" -> "a non-negligible"

S 1 Introduction p 3 "FEC... operate" -> "FEC... operates"

S 1 Introduction p 3 Figure 1 E2E, NC, IntraF, InterF, SPC, MPC abbrevations/acronyms neither spelled out on first use (as they were in earlier versions of this draft), included in this draft's glossary, nor defined in RFC 8406

S 1 Introduction p 4 "PEP... include" -> "PEP... including"

S 2 A note on satellite topology p 5 acronyms DVB etc. not spelled out on first use etc.

S 2 A note... p 6 Figure 2 seems to cover only the case where a single [simple] user device is connected by the satellite terminal (leaves out e.g. firewall/router on user end of satellite link); if this is intentional just to avoid unnecessary complication of the figure, a note explaining this would be in order

S 3.1 Two-way channel mode p 7 "computationnal" -> "computational"

S 3.3 Hybrid access p 9 "transport level" -> "transport layer", "using multiple path does not" -> "using multiple paths does not", "appart" -> "apart", "envisionned" -> "envisioned"

S 3.4 Dealing with LAN losses p 9 "cope from" -> "cope with"

S 3.4 Dealing with LAN losses p 10 Figure 6 is confusing: it omits explicitly showing the WiLAN; it uses an undefined label "C" (presumably indicating something relating to encryption, but why would that end at the Access Gateway?)

S 3.5 Dealing with varying channel conditions p 10 "cope from" -> "cope with"

S 3.5 Dealing with varying... p 10 "mechanisms that is" -> "mechanisms that are", "higher layers redundancy" -> "higher layer redundancy", "chosen band induce a required" -> "chosen band induces a required" or "chosen band requires a"

S 3.5 Dealing with varying... p 10 discussion of use cases and deployment use cases is confusing

S 3.5 Dealing with varying... p 10 Figure 7 again uses undefined label "C"

S 4.2 On the efficient usage of satellite ressource p 12 HARQ / incremental redundancy ARQ / adaptive FEC might deserve a mention here; also "ressource" -> "resource"

S 4.3 Interaction with virtualized satellite gateways and terminals p 12 "VNF functions deployment" -> "VNF deployment", "an efficient radio usage" -> "efficient radio usage", "a virtualized SATCOM terminals" -> "virtualized SATCOM equipment", "limited buffered equipments" -> "limited buffer capacities"

S 4.4 Delay/Disruption Tolerant Networks p 12 "In the context of deep-space communications, establishing communications from terrestrial gateways to satellite platforms" -> "Communications among deep-space platforms and terrestrial gateways"

5 Conclusion p 13 "'network function' block gather" -> "'network function' block gathers"

8 Security Considerations p 14 "togething" -> "together"

NicoKos commented 5 years ago

Discussed by PR #45