Skip to content
This repository has been archived by the owner on Dec 13, 2024. It is now read-only.

Commit

Permalink
Merge pull request #4 from HAW-RN/initial
Browse files Browse the repository at this point in the history
Improve the output PDF & Meeting Notes
  • Loading branch information
AnnsAnns authored Jun 1, 2024
2 parents ff247e4 + 08cb6f2 commit e751161
Show file tree
Hide file tree
Showing 8 changed files with 80 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,11 @@ jobs:
input_path: merged_output/output.md
output_dir: pdfs
images_dir: protocol/images
theme: ./pico.min.css
# for example <img src="./images/file-name.png">
image_import: ./images
# Default is true, can set to false to only get PDF files
build_html: false
table_of_contents: true
- uses: actions/upload-artifact@v4
with:
name: protocol
Expand Down
69 changes: 69 additions & 0 deletions meetings/20240531.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,69 @@
# Date: 31.03.2024

# Participants
- Tom Hert
- Daniil Khoma
- Laurin Zacharias
- Tom Westendorf
- Eike Balling
- Jeremy Reimers
- Alexander Kelsch
- Bastian Basler
- Ruben Marin Grez
- Tim Hagedorn
- Michael German
- Andrii Lysyi

# Agenda

- Showcase of the MVP of each team

# Recap

## Routing protocol

- Showcase of routing protocol MVP
- Do we need to specify the port
- Answer: Yes, makes sense to specify the port
- Move packet type to common header


## Routed protocol

- Showcase of routed protocol MVP
- Max size, example, etc.
- Move checksum to common header

## Coding

- We use JSON
- What to do next [See org issues]
- Have a basic common header ala:
```json
{
"type": "TYPE",
"version": 1,
"checksum": "HASH",
"payload": {
"ACTUAL_DATA": "ACTUAL_DATA",
"MORE_DATA": "MORE_DATA",
"EVEN_MORE_DATA": "EVEN_MORE_DATA",
}
}
```

## Use Case

- Showcase of use cases
- Consider implementing a client-side "exit" packet
- Makes sense, only works on graceful shutdown
- Can be used to signal the end of a connection
- Otherwise, the connection would be closed by the server


## Organizational issues

- Use Cases and Coding are both at the end of what they have to do [Maybe consolidate teams?]
- Routed protocol also close to done
- Showcase our work on monday
- Ask Becke for how he would imagine tests for our project
4 changes: 4 additions & 0 deletions pico.min.css

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion protocol/01_introduction.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@

## Purpose

## General requirements
## General requirements
2 changes: 1 addition & 1 deletion protocol/03_encoding.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,4 @@

## Format

## Example
## Example
2 changes: 1 addition & 1 deletion protocol/04_routing_protocol.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,4 @@

## Procedure

## Example
## Example
3 changes: 2 additions & 1 deletion protocol/05_routed_protocol.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ The task of the routed protocol is to transfer chat messages from one client to
## Procedure

## Example

```json
{
"package_type": "data_message"
Expand All @@ -34,4 +35,4 @@ Source Port: Port of the sender
Destination IP: IPv4 address of the destination
Destination Port: Port of the destination
TTL: time to live of the package. maximum of 16 to 64
Data: message value limited to 128 chars in utf-8 for smaller package size
Data: message value limited to 128 chars in utf-8 for smaller package size
2 changes: 1 addition & 1 deletion protocol/06_appendix.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@

## Glossary

## References
## References

0 comments on commit e751161

Please sign in to comment.