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.

Sign up for GitHub

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

CCSMB-5: PCM Audio Transmissions over modem #18

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

CCSMB-5: PCM Audio Transmissions over modem #18

Changes from 8 commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
a170d9f
First iteration
EmmaKnijn Jan 7, 2023
01afc24
Change songName to be more generic
EmmaKnijn Jan 7, 2023
08e8161
Drop the id field
EmmaKnijn Jan 7, 2023
b8fab17
Add protocol field and clarify the types in the fields
EmmaKnijn Jan 7, 2023
a7a79e5
Enforce protocol to be CCSMB-5
EmmaKnijn Jan 7, 2023
3445da9
Fix markdown formatting
EmmaKnijn Jan 7, 2023
8ca7b64
Restructure compliancy and expand on crosstalk
EmmaKnijn Jan 7, 2023
bccaf85
Improve based on some reviews
EmmaKnijn Jan 8, 2023
aa863e1
Add port allocation
EmmaKnijn Jan 8, 2023
c5a8320
Add a base for the discovery packet
EmmaKnijn Feb 28, 2023
2f87bec
Add a couple things that people requested
EmmaKnijn Mar 13, 2023
1b026dd
Add reference to RFC 2119
EmmaKnijn Oct 26, 2023
eb04510
Add clarification for the packet timing
EmmaKnijn Oct 26, 2023
5122823
Change title
EmmaKnijn Oct 26, 2023
70dba1f
Change the protocol name to PASC
EmmaKnijn Oct 26, 2023
779b4e9
Grammatical changes
EmmaKnijn Oct 26, 2023
2d2b02f
Grammatical fix
EmmaKnijn Oct 26, 2023
f6179e0
Recommendation on packet length
EmmaKnijn Oct 26, 2023
05e4960
Add multiple channels, clean up some stuff, add example packet
EmmaKnijn Mar 29, 2024
953963f
Enforce 759 for replyChannel
EmmaKnijn Mar 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 23 additions & 0 deletions Standards/CCSMB-5.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
# CCSMB-5: Radio Transmissions
EmmaKnijn marked this conversation as resolved.
Show resolved Hide resolved
*Author: Emma Beurskens <@emmaknijn>*
*Version: v1.0.0*
*Last updated: YYYY-MM-DD*

## Design
A client will only listen to packets while the audio is being transmitted from a server
The server will communicate to the client over modem
EmmaKnijn marked this conversation as resolved.
Show resolved Hide resolved


## Packet
A packet consists of the following fields in a table:
- `buffer`: A table that contains Signed 8-bit PCM @ 48kHz audio
- `id`: The ID of the server sending the message
- `station`: A string that contains the name of the radio station
- `title`: A string that contains the song that is currently being played, or other relevant metadata
Erb3 marked this conversation as resolved.
Show resolved Hide resolved
- `protocol`: A string that contains the protocol used

In order to be compliant the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is worded a bit informally; I'd prefer it to use full sentences in bullet points or an ordered list.

We should also mention RFC 2119 if using keywords like "MUST", "SHOULD", etc.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you write an example, I feel like writing the same In order to be compliant the in 5 bullet points is a bit redundant

- protocol field must be "CCSMB-5"
EmmaKnijn marked this conversation as resolved.
Show resolved Hide resolved
- client must be able to deal with crosstalk by allowing the user to select a station if there are multiple stations on a single channel
EmmaKnijn marked this conversation as resolved.
Show resolved Hide resolved
- client must reject a packet with an invalid `protocol` field
- server must time the packets appropriately in a way that the client is just able to play back the buffer
EmmaKnijn marked this conversation as resolved.
Show resolved Hide resolved