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

3 ➡️ 4 #120

Merged
merged 18 commits into from
Aug 7, 2020
Merged

3 ➡️ 4 #120

merged 18 commits into from
Aug 7, 2020

Conversation

chapulina
Copy link
Contributor

This ports #83, which changes the tests from ogre to ogre2. But I believe the LidarVisual test will need #116 to pass.

chapulina and others added 15 commits April 17, 2020 14:06
…dering2

[ign-rendering2] Update BitBucket links
* mv hgignore
* add gitignore
Signed-off-by: claireyywang <[email protected]>
* mv hgignore
* add gitignore

Signed-off-by: claireyywang <[email protected]>
add odd grid and fix vertical bars

Signed-off-by: John Shepherd <[email protected]>
* [ign-rendering3] Workflow updates

Signed-off-by: Louise Poubel <[email protected]>

* remove markdown-header

Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Kamal Aghayev <[email protected]>

Co-authored-by: Kamal Aghayev <[email protected]>
* use ogre2 test

Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
@chapulina chapulina requested a review from iche033 as a code owner August 4, 2020 21:35
@github-actions github-actions bot added the 🔮 dome Ignition Dome label Aug 4, 2020
@iche033
Copy link
Contributor

iche033 commented Aug 5, 2020

now that ogre2 tests are run with this PR, jenkins picked up a LidarVisual test failure, which may be fixed by pull request #116. cc @mihirk284

@chapulina chapulina force-pushed the chapulina/3_to_4 branch 2 times, most recently from de842cb to 25e6c7b Compare August 5, 2020 18:55
Signed-off-by: Louise Poubel <[email protected]>
@chapulina
Copy link
Contributor Author

I updated this PR now that #116 is merged, and INTEGRATION_lidar_visual is crashing. I can reproduce locally, it's a problem during destruction. here's a backtrace:

terminate called after throwing an instance of 'Ogre::InvalidParametersException'
  what():  OGRE EXCEPTION(2:InvalidParametersException): Vertex Buffer belongs to a Multisource pool, not this VaoManager in VaoManager::des
troyVertexBuffer at /var/lib/jenkins/workspace/ogre-2.1-debbuilder/repo/OgreMain/src/Vao/OgreVaoManager.cpp (line 127)

Thread 1 "INTEGRATION_lid" received signal SIGABRT, Aborted.
__GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:51
51      ../sysdeps/unix/sysv/linux/raise.c: No such file or directory.
(gdb) bt
#0  __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:51
#1  0x00007ffff6778801 in __GI_abort () at abort.c:79
#2  0x00007ffff716b957 in ?? () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#3  0x00007ffff7171ae6 in ?? () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#4  0x00007ffff7170b49 in ?? () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#5  0x00007ffff71714b8 in __gxx_personality_v0 () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#6  0x00007ffff6b39573 in ?? () from /lib/x86_64-linux-gnu/libgcc_s.so.1
#7  0x00007ffff6b39df5 in _Unwind_Resume () from /lib/x86_64-linux-gnu/libgcc_s.so.1
#8  0x00007ffff0cc2a3d in Ogre::SubMesh::destroyVaos(Ogre::FastArray<Ogre::VertexArrayObject*>&, Ogre::VaoManager*, bool) ()
   from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#9  0x00007ffff0cc2b66 in Ogre::SubMesh::~SubMesh() () from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#10 0x00007ffff0b56b80 in Ogre::Mesh::unloadImpl() () from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#11 0x00007ffff0bf6f37 in Ogre::Resource::unload() () from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#12 0x00007ffff0b56ef7 in Ogre::Mesh::~Mesh() () from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#13 0x00007ffff0b57019 in Ogre::Mesh::~Mesh() () from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#14 0x00007ffff0a87a25 in ?? () from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#15 0x00007ffff0c06059 in ?? () from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#16 0x00007ffff0c0006d in Ogre::ResourceGroupManager::_notifyAllResourcesRemoved(Ogre::ResourceManager*) ()
   from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#17 0x00007ffff0bfc8b5 in Ogre::ResourceGroupManager::shutdownAll() () from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#18 0x00007ffff0c0fc13 in Ogre::Root::shutdown() () from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#19 0x00007ffff0c110a0 in Ogre::Root::~Root() () from /usr/lib/x86_64-linux-gnu/libOgreMain.so.2.1.0
#20 0x00007ffff14d98fc in ignition::rendering::v4::Ogre2RenderEngine::Destroy (
    this=0x7ffff179df80 <ignition::common::SingletonT<ignition::rendering::v4::Ogre2RenderEngine>::GetInstance()::t>)
    at /home/chapulina/dome_ws/src/ign-rendering/ogre2/src/Ogre2RenderEngine.cc:119
#21 0x00007ffff7ba6762 in ignition::rendering::v4::RenderEngineManagerPrivate::UnloadEngine(std::_Rb_tree_iterator<std::pair<std::__cxx11::b
asic_string<char, std::char_traits<char>, std::allocator<char> > const, ignition::rendering::v4::RenderEngine*> >) ()
    at /home/chapulina/dome_ws/src/ign-rendering/src/RenderEngineManager.cc:334

@codecov
Copy link

codecov bot commented Aug 6, 2020

Codecov Report

Merging #120 into master will increase coverage by 44.58%.
The diff coverage is 62.03%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #120       +/-   ##
===========================================
+ Coverage    7.51%   52.09%   +44.58%     
===========================================
  Files          27      143      +116     
  Lines        1770    13070    +11300     
===========================================
+ Hits          133     6809     +6676     
- Misses       1637     6261     +4624     
Impacted Files Coverage Δ
include/ignition/rendering/ParticleEmitter.hh 100.00% <ø> (ø)
include/ignition/rendering/Scene.hh 0.00% <ø> (ø)
include/ignition/rendering/base/BaseScene.hh 0.00% <ø> (ø)
ogre/src/OgreGrid.cc 0.00% <0.00%> (ø)
ogre/src/OgreLidarVisual.cc 0.00% <0.00%> (ø)
ogre2/src/Ogre2Camera.cc 87.70% <0.00%> (ø)
ogre2/src/Ogre2DepthCamera.cc 94.23% <ø> (ø)
src/RenderEngineManager.cc 76.21% <ø> (+50.81%) ⬆️
...ude/ignition/rendering/base/BaseParticleEmitter.hh 41.55% <40.00%> (ø)
ogre2/src/Ogre2Grid.cc 66.23% <45.00%> (ø)
... and 146 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 528ba21...87db562. Read the comment docs.

@iche033
Copy link
Contributor

iche033 commented Aug 7, 2020

lasest ubuntu build is 🔵

Signed-off-by: Louise Poubel <[email protected]>
@chapulina chapulina merged commit 42e20ed into master Aug 7, 2020
@chapulina chapulina deleted the chapulina/3_to_4 branch August 7, 2020 00:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔮 dome Ignition Dome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants