Skip to content

[CRL] Add unified MR registry with P2P engine integration#513

Open
MC952-arch wants to merge 2 commits into
flagos-ai:mainfrom
MC952-arch:mr-registry-core
Open

[CRL] Add unified MR registry with P2P engine integration#513
MC952-arch wants to merge 2 commits into
flagos-ai:mainfrom
MC952-arch:mr-registry-core

Conversation

@MC952-arch

Copy link
Copy Markdown
Collaborator

PR Category

CRL

PR Types

Optimizations

PR Description

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

Comment thread test/unittest/core/test_mr_registry.cpp
Comment thread flagcx/core/flagcx_mr_registry.cc
Comment thread flagcx/core/flagcx_p2p.cc
Comment thread flagcx/core/flagcx_p2p.cc
Comment thread flagcx/core/flagcx_p2p.cc Outdated
Comment thread flagcx/core/flagcx_mr_registry.cc

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 10 comments.

Comment thread test/unittest/core/test_mr_registry.cpp
Comment thread test/unittest/core/test_mr_registry.cpp
Comment thread flagcx/core/flagcx_mr_registry.cc Outdated
Comment thread flagcx/core/flagcx_mr_registry.cc
Comment thread flagcx/core/flagcx_mr_registry.cc Outdated
Comment thread flagcx/core/flagcx_mr_registry.cc Outdated
Comment thread flagcx/core/flagcx_mr_registry.cc Outdated
Comment thread flagcx/core/flagcx_p2p.cc Outdated
Comment thread flagcx/core/flagcx_p2p.cc
Comment thread flagcx/core/include/flagcx_mr_registry.h

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

Comment thread flagcx/core/flagcx_p2p.cc Outdated
Comment thread flagcx/core/flagcx_p2p.cc
Comment thread flagcx/core/flagcx_p2p.cc
Comment thread flagcx/core/flagcx_mr_registry.cc
Comment thread flagcx/core/flagcx_mr_registry.cc
Comment thread flagcx/core/flagcx_p2p.cc

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 15 comments.

Comment thread flagcx/core/flagcx_mr_registry.cc
Comment thread flagcx/core/flagcx_mr_registry.cc
Comment thread flagcx/core/flagcx_mr_registry.cc
Comment thread flagcx/core/flagcx_mr_registry.cc
Comment thread flagcx/core/flagcx_mr_registry.cc
Comment thread test/unittest/core/test_mr_registry.cpp
Comment thread test/unittest/core/test_mr_registry.cpp
Comment thread test/unittest/core/test_mr_registry.cpp
Comment thread test/unittest/core/test_mr_registry.cpp
Comment thread test/unittest/core/test_mr_registry.cpp
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.

2 participants