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

Update aiebu to use main-ge branch #8742

Merged
merged 1 commit into from
Feb 6, 2025
Merged

Update aiebu to use main-ge branch #8742

merged 1 commit into from
Feb 6, 2025

Conversation

stsoe
Copy link
Collaborator

@stsoe stsoe commented Feb 3, 2025

Problem solved by the commit

Make the aiebu submodule point at
https://github.com/Xilinx/aiebu/tree/main-ge. The main-ge branch is a copy of gitenterprise version of aiebu with amendments to build on both Linux and Windows.

How problem was solved, alternative solutions (if any) and why they were rejected

There will be additional fine tuning of the the aiebe:main-ge branch while we validate the branch with XRT.

What has been tested and how, request additional testing if necessary

unit tests of aiebu

@stsoe stsoe requested a review from rozumx as a code owner February 3, 2025 23:10
@stsoe stsoe requested review from sonals and removed request for rozumx February 3, 2025 23:10
@stsoe stsoe force-pushed the aiebu-ge branch 2 times, most recently from f20aae7 to 43a7ab6 Compare February 4, 2025 22:49
Make the aiebu submodule point at
https://github.com/Xilinx/aiebu/tree/main-ge.  The main-ge branch is a
copy of gitenterprise version of aiebu.

There will be additional fine tuning of the the aiebe:main-ge branch
while we validate the branch with XRT.

Update XRT's version of ELFIO to match that of aiebu (182248f)

Signed-off-by: Soren Soe <[email protected]>
@stsoe stsoe merged commit aebdfc7 into Xilinx:master Feb 6, 2025
20 checks passed
@stsoe stsoe deleted the aiebu-ge branch February 6, 2025 16:46
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