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

map-style deserialiser output #29

Open
RalphSteinhagen opened this issue May 28, 2021 · 0 comments
Open

map-style deserialiser output #29

RalphSteinhagen opened this issue May 28, 2021 · 0 comments
Labels
question Further information is requested

Comments

@RalphSteinhagen
Copy link
Member

RalphSteinhagen commented May 28, 2021

map-style deserialiser output

presently: class -> wire-format -> class

do we have a strong use-case for this?
pros:

  • support of more flexible run-time declared use-cases?
  • AK: -> separate deserialiser anyway, added later on

cons:

  • more coding (nested unordered_map<std::string, unordered_map<...>> (probably need to use std::any a lot) -> least constexpr-ness
  • AK: if we allow services to use this dynamic interface, the API evolution issue becomes a lot more complicated -> deserialisation only?
  • how many strong (<-> sort of mandatory) or frequent (80/20 rule) use-cases do we have?
    1. generic top-level UIs that can subscribe/get/set to any service (opencmw-explorer)
    2. AK: needed for potential python integration (else we have to dynamically compile opencmw c++ code from python, can be done but involves a lot of complexity)

Application: interface to Python to be evaluated

@RalphSteinhagen RalphSteinhagen added the question Further information is requested label May 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant