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

A1 Mediator Implementation #13

Conversation

kamilkociszewski
Copy link
Contributor

A1 mediator which allows to communicate between near-RT RIC and non-RT RIC, can be used with FlexRIC.
Based on OSC A1 mediator.

@MinaYonan123 MinaYonan123 self-requested a review November 28, 2024 15:53
@mostafa-ashraf1709 mostafa-ashraf1709 added the enhancement New feature or request label Nov 30, 2024
@mostafa-ashraf1709
Copy link
Collaborator

mostafa-ashraf1709 commented Nov 30, 2024

Hi @kamilkociszewski,
Thanks for your contribution!
I suggest you to open an issue explain this feature/enhancement and will link this PR to the issue.

What do you think @MinaYonan123?

@MinaYonan123
Copy link
Collaborator

MinaYonan123 commented Dec 1, 2024

Hello @kamilkociszewski
Thanks for your contributions :)
My suggestions for this PR:

1- Add Paragraph in sub-module README about the original A1 Mediator and OIP contribution on top of it.
2- Add tutorial/ steps to run with FlexRIC instead of replicate the flexric repo in the submodule.

BR,
Mina Yonan

@mostafa-ashraf1709 mostafa-ashraf1709 changed the title A1 mediator implementation A1 Mediator Implementation Dec 1, 2024
@kamilkociszewski
Copy link
Contributor Author

kamilkociszewski commented Dec 2, 2024

Hi,
i just updated ns3 submodule for the one with RIC TaaP A1 extension.
I will update README soon.

@kamilkociszewski
Copy link
Contributor Author

Hello @kamilkociszewski Thanks for your contributions :) My suggestions for this PR:

1- Add Paragraph in sub-module README about the original A1 Mediator and OIP contribution on top of it. 2- Add tutorial/ steps to run with FlexRIC instead of replicate the flexric repo in the submodule.

BR, Mina Yonan

@MinaYonan123 ,
i implemented your suggestion, let me know your opinion.

I update README in subrepo and then update also README in main repo to match instructions in subrepo.

@kamilkociszewski
Copy link
Contributor Author

I will add new, clean request soon.

@mostafa-ashraf1709
Copy link
Collaborator

mostafa-ashraf1709 commented Dec 10, 2024

I will add new, clean request soon.

Hi @kamilkociszewski, thanks for contribution!
Please mention this issue 14 when you open your new PR.

FYI @MinaYonan123

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

Successfully merging this pull request may close these issues.

3 participants