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

Ideas for RCP 2.0 #27

Open
i-n-g-o opened this issue Apr 12, 2023 · 2 comments
Open

Ideas for RCP 2.0 #27

i-n-g-o opened this issue Apr 12, 2023 · 2 comments

Comments

@i-n-g-o
Copy link
Member

i-n-g-o commented Apr 12, 2023

  • rework of id to allow a bigger range and save bits (encode the format of the id within the id-bytes)
  • try to avoid terminators
  • remove language strings (decided to leave them as they are)
    • until now: each client can choose the language individually without the need of a special server
    • this change would imply that each client needs to request a language on initialize
    • reason: it seems not a good idea to transmit more than one language
  • remove timestamp from packet (we leave timestamps because it seems handy to have)
  • type enum: send index instead of string (note: deal with reordered lists outside if protocol)
  • type string: automatically use tiny,short,long string for string values
    • consider to remove long-strings as this type does not seem to be useful (transmitting up to 4.2GB of data)
  • specify Layout
  • specify Style (?)
@i-n-g-o i-n-g-o added this to the v2.0 milestone Apr 12, 2023
@joreg
Copy link
Contributor

joreg commented Oct 27, 2023

from the above consider to bring to 1.0

  • remove language strings -> decided not too because the two arguments (more complex implementation, and saving bandwidth) seem neglectible
    • related, we thought of adding a translation-key to parameters and an idea of translation tables...
  • type enum: send index instead of string -> agreed
  • remove idea of long-string and then only have short and long strings -> agreed
  • make string parameter a long string -> agreed
  • remove "scale" from number definition -> agreed

@i-n-g-o
Copy link
Member Author

i-n-g-o commented Nov 27, 2023

I added #30 for all open issues related to 1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants