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

Compatibility with more recent versions of Ogre #1538

Merged
merged 60 commits into from
Jan 18, 2025

Conversation

rolalaro
Copy link

@rolalaro rolalaro commented Jan 8, 2025

  • Fix compatibilities issues with Ogre 1.12.10, 13.1.0 and 14.0.0
  • Created 2 CIs to check the compilation of ViSP when Ogre is used as 3rd party. Unit tests for Ogre cannot be run because Ogre requires a valid X11 context

LAGNEAU Romain and others added 30 commits December 4, 2024 15:15
…d" and "Ogre::WindowEventListener : expected class-name before '{' token"
Moved detection of the visp-images outside the vp_add_test method, to make the detection possible even when tests are disabled
…cause it seemed to cause a problem when running successive configure steps
Copy link

codecov bot commented Jan 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (master@abf3419). Learn more about missing BASE report.
Report is 46 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1538   +/-   ##
=========================================
  Coverage          ?   54.01%           
=========================================
  Files             ?      441           
  Lines             ?    53865           
  Branches          ?        0           
=========================================
  Hits              ?    29095           
  Misses            ?    24770           
  Partials          ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

- Introduce vpDisplayFactory
- Remove useless try/catch and other code
@fspindle
Copy link
Contributor

@rolalaro AROgre and AROgreBasic are now working with Ogre 1.12.10 installed from ubuntu package.

There remains a segfault when I run ./mbtEdgeKltTracking -c -o or ./mbtGenericTracking -c -w -o

LAGNEAU Romain added 3 commits January 14, 2025 08:38
The segfault came from the fact that the rendering window was added to the list of listening windows but not removed when destroyed
@rolalaro
Copy link
Author

@fspindle the bug should have been fixed in the latest commit, could you please pull and try again ?

@fspindle
Copy link
Contributor

Closes #1547 #1519 #1546

@fspindle fspindle merged commit f6faf50 into lagadic:master Jan 18, 2025
104 checks passed
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