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

Support mixed component exchanges #111

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Support mixed component exchanges #111

wants to merge 2 commits into from

Conversation

huyenngn
Copy link
Member

@huyenngn huyenngn commented Jun 19, 2024

Closes #83

  • Scenario 1:
    Screenshot 2024-06-25 093317

Interface Context of Speed Data Transmission

  • Scenario 2:
    Screenshot 2024-06-25 093016

Interface Context of Data Transmission

@huyenngn huyenngn marked this pull request as ready for review July 2, 2024 11:17
@huyenngn huyenngn requested review from ewuerger and vik378 as code owners July 2, 2024 11:17
@huyenngn
Copy link
Member Author

huyenngn commented Jul 9, 2024

@ewuerger Is this planned to get merged eventually? It won't make much sense to merge this after rewriting the Interface Context Collector. Should this PR be discarded and be redone in the new Interface context collector?

@ewuerger
Copy link
Member

ewuerger commented Jul 9, 2024

I think we should merge this PR first. Just need to find some time to review it. To resolve the conflicts we should group up. :)

@ewuerger
Copy link
Member

ewuerger commented Jul 9, 2024

There are still some issues in the interface collector that we need to resolve. Can't approve yet. :(

@ewuerger
Copy link
Member

ewuerger commented Oct 2, 2024

It should be checked if this isn't yet covered by the main/latest release version. Only if not, we should keep this open and work on it further.
@huyenngn please check for the cases from the linked issues.

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

Successfully merging this pull request may close these issues.

Add support for mixed or messed up component exchange contexts
2 participants