Skip to content

Conversation

iche033
Copy link
Contributor

@iche033 iche033 commented Oct 4, 2024

➡️ Forward port

Port gz-rendering9 to main

Branch comparison: main...gz-rendering9

Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

iche033 and others added 19 commits August 25, 2024 18:06
Signed-off-by: Van Malleghem Antoine <[email protected]>
Signed-off-by: Van Malleghem Antoine <[email protected]>
Signed-off-by: Antoine Van Malleghem <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Signed-off-by: Van Malleghem Antoine <[email protected]>
Revert "Fix using lightmap and emissive map together (#1047)"
By adding the missing DL dependency

Signed-off-by: Silvio Traversaro <[email protected]>
Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

Windows CI is broken but we can fix this in a follow up PR

usage: colcon [-h] [--log-base LOG_BASE] [--log-level LOG_LEVEL]
              {build,extension-points,extensions,graph,info,list,metadata,test,test-result,version-check}
              ...
colcon: error: unrecognized arguments: -A x64 -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTING=OFF

@iche033
Copy link
Contributor Author

iche033 commented Oct 4, 2024

Windows CI is broken but we can fix this in a follow up PR

usage: colcon [-h] [--log-base LOG_BASE] [--log-level LOG_LEVEL]
              {build,extension-points,extensions,graph,info,list,metadata,test,test-result,version-check}
              ...
colcon: error: unrecognized arguments: -A x64 -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTING=OFF

Not sure why it's working on gz-rendering9 branch and not main. Took a quick look, colcon and pixi versions are the same. I also tried merging with #1052 but didn't help. Will continue looking into it

Signed-off-by: Ian Chen <[email protected]>
@iche033
Copy link
Contributor Author

iche033 commented Oct 9, 2024

merging this first. Will address windows CI github actions separately

@iche033 iche033 merged commit 115a179 into main Oct 9, 2024
8 of 10 checks passed
@iche033 iche033 deleted the merge_9_main_20241004 branch October 9, 2024 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

8 participants