RSS Parrot

BETA

🦜 openmw activity

@gitlab.com.openmw.openmw@rss-parrot.net

I'm an automated parrot! I relay a website's RSS feed to the Fediverse. Every time a new post appears in the feed, I toot about it. Follow me to get all new posts in your Mastodon timeline! Brought to you by the RSS Parrot.

---

Your feed and you don't want it here? Just e-mail the birb.

Site URL: gitlab.com/OpenMW/openmw

Feed URL: gitlab.com/OpenMW/openmw.atom

Posts: 316

Followers: 1

Abdu Sharif commented on merge request !5492 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5492#note_3675718993

Published: August 12, 2026 06:06

Adding those here so no one else has doubts that people add their own music: This shows people knew about this for over a decade and a half: https://gamefaqs.gamespot.com/boards/935550-the-elder-scrolls-iii-morrowind-game-of-the-year-edition/51270358 A…

S3kshun8 Games commented on merge request !5498 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5498#note_3675673586

Published: August 12, 2026 05:46

Uhhhhh well I was trying to draw a distinction in phrasing between the bone not existing on ANY rig versus not presently existing on the one you are actually using but yes it makes no sense and is overcomplicated, 1707ae26

S3kshun8 Games commented on merge request !5498 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5498#note_3675673401

Published: August 12, 2026 05:46

It's thread-safe/dangerous in the same way the entirety of openmw.animation is, which is to say, not exactly, but, since we're running during Cull, it's fine. This bind actually could theoretically be a bit dangerous, but, hasBone is just as dangerous in…

Cody Glassman commented on merge request !5498 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5498#note_3675550821

Published: August 12, 2026 04:52

... or is not attached to the scene graph Is there some way for the bone to exist but not be attached, I'm not seeing how this is possible. In any case, better to avoid terminology like this to begin with.

Cody Glassman commented on merge request !5498 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5498#note_3675550811

Published: August 12, 2026 04:52

Have you verified this is thread safe? I'm not versed too well on how the node map is updated but from a cursory glance bone positions are updated in update thread, which means this is safe, but it's worth mentioning to double check.

S3kshun8 Games opened merge request !5498: FEAT: Add a bind to get bone positions at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5498

Published: August 12, 2026 02:33

Not really a lot to it but it did reveal that Transforms were broken in the same way the VFX path was so I fixed that and added some tests for it.

Cody Glassman commented on task #9242 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/issues/9242#note_3674917470

Published: August 12, 2026 01:36

It's a bit more involved then I imagined since ideally we have a flexible system to enable/disable clip planes just as we do with the FFP version. If we just deal with a single clip plane this is much simpler, but not sure we want to corner ourselves into…

AnyOldName3 commented on merge request !5493 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5493#note_3674531759

Published: August 11, 2026 22:40

Ooof. Anyway, most of what I've said is still fine - we still want setDefaults(osgUtil::SceneView::NO_SCENEVIEW_LIGHT) as osgUtil::SceneView::APPLY_GLOBAL_DEFAULTS and osgUtil::SceneView::CLEAR_GLOBAL_STATESET weren't set by the old code, unless there's…

Hyamen Tar opened issue #9271: [Crash] qt.core.qobject.connect at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/issues/9271

Published: August 11, 2026 21:12

Issue Description Have encounter specific CTD/system stall a few times now. CS Log Entry: [22:05:27.667 W] qt.core.qobject.connect: QObject::connect: Cannot connect (nullptr)::focusObjectChanged(QObject*) to…

AnyOldName3 accepted merge request !5486: Provide more debug information on crash the CI integration tests job at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5486

Published: August 11, 2026 20:48

Install debug symbols for dependencies. Keep more artifacts. Example job with crash: https://gitlab.com/elsid/openmw/-/jobs/15795073583 I have read the CONTRIBUTING guidelines My code runs locally My code passes CI on my fork

jvoisin approved merge request !5486: Provide more debug information on crash the CI integration tests job at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5486

Published: August 11, 2026 20:14

Install debug symbols for dependencies. Keep more artifacts. Example job with crash: https://gitlab.com/elsid/openmw/-/jobs/15795073583 I have read the CONTRIBUTING guidelines My code runs locally My code passes CI on my fork

AnyOldName3 accepted merge request !5496: Finish background Lua GC before input processing at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5496

Published: August 11, 2026 20:03

Related to #9043 Finish background Lua GC before input processing InputManager::update can run Lua synchronously: uiModeChanged fires on any input-driven UI mode change and races the background collection step. Reported as a menu-key freeze after !5487…

AnyOldName3 commented on merge request !5486 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5486#note_3673941906

Published: August 11, 2026 19:56

It should be reliable (at least as reliable as anything involving Ubuntu ever is, especially if the elfutils server's available as a backup), so it's potentially a sign of some kind of weirdness that it didn't work. It's beyond this MR's scope, but we're…

elsid commented on merge request !5486 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5486#note_3673857271

Published: August 11, 2026 19:36

The approach is too unreliable for CI anyway. If packages cannot be installed nothing would work anyway but debugging why symbols are not shown due to unavailable server does not worth it.

S3kshun8 Games approved merge request !5496: Finish background Lua GC before input processing at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5496

Published: August 11, 2026 19:17

Related to #9043 Finish background Lua GC before input processing InputManager::update can run Lua synchronously: uiModeChanged fires on any input-driven UI mode change and races the background collection step. Reported as a menu-key freeze after !5487…

Andrii Gook commented on merge request !5336 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5336#note_3673711553

Published: August 11, 2026 19:11

shouldn't this be table.records instead of types.records? And Utils.getAnyRecord also checks if the records are nil - shouldn't such check either be here too, or removed from that function?

Andrii Gook commented on merge request !5336 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5336#note_3673687763

Published: August 11, 2026 19:07

This function needs automatic type detection too, so it can be easily used standalone. autoType() would need to be be rewritten to accept tooltip it detects type for, instead of using variable from outer scope.

Mads Buvik Sandvei commented on merge request !5392 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5392#note_3673580437

Published: August 11, 2026 18:44

I should have looked more closely at the engine code before commenting. I had forgotten the cursed subtle difference in the cost formula for create. I guess the current split does make sense. I'm not sure i like that we get two…

Thomas Portal commented on merge request !5492 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5492#note_3673509005

Published: August 11, 2026 18:29

Update on the FFmpeg front, I didn't know they moved from the old trac + mailing list setup to forgejo last summer! With that I was able to send a PR for the option directly. If they accept it we'll be down to just setting the flag. (Found another per-open…

Mads Buvik Sandvei commented on merge request !5392 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5392#note_3673507169

Published: August 11, 2026 18:29

As far as i can tell, the only real difference between getEffectListCostCreation and getEffectListCost is the minimum value of 1 per effect. Could reduce the code duplication by adding a minimum effect value parameter to getEffectListCost and just…

S3kshun8 Games commented on merge request !5487 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5487#note_3673411839

Published: August 11, 2026 18:03

By the way, since you seem interested in this subject - LJ absolutely will do emergency GC cycles during big spikes in allocations. The easiest way to induce it is to set a timer function that runs every few seconds and constructs, say, 100K Vec4s or so. I…

Evil Eye approved merge request !5496: Finish background Lua GC before input processing at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5496

Published: August 11, 2026 18:02

Related to #9043 Finish background Lua GC before input processing InputManager::update can run Lua synchronously: uiModeChanged fires on any input-driven UI mode change and races the background collection step. Reported as a menu-key freeze after !5487…

Thomas Portal commented on merge request !5487 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5487#note_3673370507

Published: August 11, 2026 17:51

Gotcha! I couldn't reproduce the freeze locally even with a heavy Lua modlist, so if the Discord folks can confirm the fix build behaves (or point me at a mod triggering that), that would be great so I can add it to my "bunch of Lua" setup. MR at !5496

Thomas Portal opened merge request !5496: Finish background Lua GC before input processing at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5496

Published: August 11, 2026 17:50

Related to #9043 Finish background Lua GC before input processing InputManager::update can run Lua synchronously: uiModeChanged fires on any input-driven UI mode change and races the background collection step. Reported as a menu-key freeze after !5487…

S3kshun8 Games commented on merge request !5487 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5487#note_3673322824

Published: August 11, 2026 17:39

input processing happens before the finishGc call here, which then synchronously calls uiModeChanged, so actually I think this race can be triggered on every UI mode change and they just happened to mention escape first.

Alexei Kotov commented on merge request !5220 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5220#note_3672458488

Published: August 11, 2026 14:37

There might not be an elegant solution. You probably have to revert to the original behavior when the focal point offset is non-zero. Something like this might not be too complicated (completely untested): const MWRender::Camera& camera =…

Thomas Portal opened merge request !5492: Skip embedded cover art when opening audio files at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5492

Published: August 11, 2026 14:03

Depends on !5490 libavformat always parses attached pictures and neither demuxer exposes a skip option, meaning a file with embedded cover art reads the picture on every open, on the thread that starts the sound. Real case: The Lossless Soundtrack (Nexus…

AnyOldName3 commented on merge request !5492 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5492#note_3672257456

Published: August 11, 2026 14:01

I was actually leaning towards this probably being worthwhile (at least if we can't get FFMPEG to handle it for us in a timely manner) after Abdu's comment, as I'm pretty sure that's a thing people do.

AnyOldName3 commented on merge request !5493 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5493#note_3672194854

Published: August 11, 2026 13:50

Things like the stats handler have no view, so the Renderer constructor defaults options to osgUtil::SceneView::HEADLIGHT, but for the places where there is a view, like the renderer for the default viewer for the whole engine, setting its lighting mode…

Alexei Kotov closed issue #9268: F4 profiler pages are not showing up at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/issues/9268

Published: August 11, 2026 13:12

Happens with the latest development build from August 9 (5cf42a03), and doesn't happen with 0.51 release.

Alexei Kotov accepted merge request !5491: Fix F4 menu at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5491

Published: August 11, 2026 13:12

Closes #9268 setDefaults actually modifies the camera beyond the stateset, so it's overriding some the modifications we're doing just before.3 I have read the CONTRIBUTING guidelines My code runs locally My code passes CI on my fork

Alexei Kotov accepted merge request !5490: "Name the audio demuxer from the file extension and cap stream probing" at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5490

Published: August 11, 2026 13:11

Related to #4880. Depends on !5488 Follow-up to the mp3 question on !5488: after avformat_open_input, an mp3 stream has its codec id, but sample rate and channel count arrive only with the first decoded frame; the scan cannot be skipped for it, only…

jvoisin approved merge request !5485: Reset weapon state to equipped during mid-equip/unequip knockdown (#9258) at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5485

Published: August 11, 2026 13:00

Closes #9258 Try this @AbduSharif. In any case, I believe this to be an improvement. The None reset added and totally undocumented in 8455ed62 is incomplete and doesn't properly let the equip animation restart as it seemed to intend to do. That aside, the…

jvoisin approved merge request !5495: Execute werewolf refusal action on GUI mode pickup (#9249) at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5495

Published: August 11, 2026 12:49

Closes #9249. Deduplicates refusal action generation logic and uses it in the inventory. This is somewhat fragile as it assumes the structure of item activation and pickUpObject will remain roughly as it is. It's simple and does the job, though. I have…

Thomas Portal closed merge request !5492: Draft: Skip embedded cover art when opening audio files at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5492

Published: August 11, 2026 12:39

Depends on !5490 libavformat always parses attached pictures and neither demuxer exposes a skip option, meaning a file with embedded cover art reads the picture on every open, on the thread that starts the sound. Real case: The Lossless Soundtrack (Nexus…

Thomas Portal commented on merge request !5492 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5492#note_3671829840

Published: August 11, 2026 12:38

No negativity perceived, and I appreciate the concern and preface nevertheless! I think we both have reasonable arguments, but trust your point about the maintenance cost itself above all others - the best code is no code. A quick win isn't always one when…

AnyOldName3 commented on merge request !5493 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5493#note_3669427386

Published: August 11, 2026 00:27

I figured it out. It appears that disableFFPLightModelForRenderer was written under the assumption that osgUtil::SceneView::setDefaults(unsigned int options) was called from osgUtil::SceneView::setDefaults(), defined as virtual void setDefaults() {…

Chris Robinson approved merge request !5490: "Name the audio demuxer from the file extension and cap stream probing" at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5490

Published: August 10, 2026 23:59

Related to #4880. Depends on !5488 Follow-up to the mp3 question on !5488: after avformat_open_input, an mp3 stream has its codec id, but sample rate and channel count arrive only with the first decoded frame; the scan cannot be skipped for it, only…

Abdu Sharif commented on merge request !5492 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5492#note_3669371952

Published: August 10, 2026 23:53

pop tracks that definitely have album art Modding aside, its a known "tips & tricks" thing that you can get Morrowind to play tracks by dropping music in the folders so I don't doubt anyone had done this or would do it.

AnyOldName3 commented on merge request !5486 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5486#note_3669363278

Published: August 10, 2026 23:49

Weird. It's set on my Ubuntu 24.04 installation, so I'd expect it to also be set in the Ubuntu 24.04 docker image. It seems that it's coming from my ~/.bashrc. It's not in /etc/skel/.bashrc, though, even though the Ubuntu manual says the variable's set…

AnyOldName3 commented on merge request !5492 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5492#note_3669343543

Published: August 10, 2026 23:41

I'll preface this by saying that none of this is automatically a rejection of this MR, we're just making sure it's all thought through, so don't get demoralised if it seems like a wall of negativity. Embedded cover art is the default in most encoders and…

Thomas Portal commented on merge request !5490 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5490#note_3669287591

Published: August 10, 2026 23:12

Yep, double-checked both points! On the free: no leak, already handled. The buffer dies in the existing AVIOContextDeleter at the top of ffmpegdecoder.cpp, which runs av_freep(&ptr->buffer) before avio_context_free; since it frees through ptr->buffer…

Alexei Kotov approved merge request !5491: Fix F4 menu at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5491

Published: August 10, 2026 23:11

Closes #9268 setDefaults actually modifies the camera beyond the stateset, so it's overriding some the modifications we're doing just before.3 I have read the CONTRIBUTING guidelines My code runs locally My code passes CI on my fork

Alexei Kotov opened merge request !5495: Execute werewolf refusal action on GUI mode pickup (#9249) at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5495

Published: August 10, 2026 23:10

Closes #9249. Deduplicates refusal action generation logic and uses it in the inventory. This is somewhat fragile as it assumes the structure of item activation and pickUpObject will remain roughly as it is. It's simple and does the job, though. I have…

Chris Robinson commented on merge request !5490 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5490#note_3669244650

Published: August 10, 2026 22:46

According to the docs, the given buffer needs to be freed later with av_free. Can you double-check to make sure this isn't leaking when the ioCtx is later freed? I notice this is also passing 0 for the write_flag parameter, indicating the buffer isn't…

Alexei Kotov accepted merge request !5488: Skip avformat_find_stream_info when the header fills the codec parameters at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5488

Published: August 10, 2026 21:57

Related to #4880 (removes the frame-stalling cost, mp3 note at the end). FFmpegDecoder::open() calls avformat_find_stream_info unconditionally. For wav files the scan reads packets across the whole file to estimate duration and bitrate; the decoder reads…

elsid commented on merge request !5493 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5493#note_3669139882

Published: August 10, 2026 21:54

Build OpenSceneGraph with following diff to see: diff --git a/src/osgUtil/SceneView.cpp b/src/osgUtil/SceneView.cpp index c6310a090..770bb31a8 100644 --- a/src/osgUtil/SceneView.cpp +++ b/src/osgUtil/SceneView.cpp @@ -313,6 +313,8 @@ void SceneView::init()…

elsid commented on merge request !5486 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5486#note_3669070039

Published: August 10, 2026 21:24

debuginfod.ubuntu.com does not seem to work so I'm not sure it's a good idea to use it. Basically any debug symbols server. Package installation is more reliable. I will revert the change.

Thomas Portal commented on merge request !5492 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5492#note_3669042646

Published: August 10, 2026 21:15

Fair point on the sample, the one art case I found is a maintained OpenMW-era mod; the other packs I probed are all clean. If it were only that one mod, stripping the art would be the best way. I think the class is bigger than my sample though. Embedded…

AnyOldName3 commented on merge request !5493 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5493#note_3668993477

Published: August 10, 2026 20:57

I don't think the analysis in #9269 is quite right, as before 65cfba7f made it so we explicitly called setDefaults with COMPILE_GLOBJECTS_AT_INIT, it was being done implicitly by OSG internals as it's part of STANDARD_SETTINGS, and we've not changed when…

AnyOldName3 commented on merge request !5492 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5492#note_3668992889

Published: August 10, 2026 20:57

As it seems like it's just mods that were made for OpenMW and are still maintained that are affected, it might be more prudent to just declare this to be something that obviously won't produce great results and tell the mod authors to strip the cover art…

AnyOldName3 approved merge request !5486: Provide more debug information on crash the CI integration tests job at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5486

Published: August 10, 2026 20:32

Install debug symbols for dependencies. Keep more artifacts. Example job with crash: https://gitlab.com/elsid/openmw/-/jobs/15795073583 I have read the CONTRIBUTING guidelines My code runs locally My code passes CI on my fork

Thomas Portal commented on issue #4880 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/issues/4880#note_3668874215

Published: August 10, 2026 20:19

Re: the measurements I promised, I built and benchmarked both layers from the 2025 discussion. The MRs stack behind !5490 once it and !5488 get merged. The detailed-ish shape: layer 1 is an LRU cache of file heads, not whole files. It learns prefix/suffix…

elsid commented on merge request !5486 at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5486#note_3668815337

Published: August 10, 2026 20:01

Changing crash catcher is out of scope for this MR. It's basically a new feature when here I'm just trying to make it simpler to debug CI failures. SDL popup would not work anyway so I'd need a setting in some form. I will change to use .gdbinit.

elsid opened merge request !5493: Fix crash on concurrent scene view initialization and cell loading at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5493

Published: August 10, 2026 19:52

Do not perform traversal on scene view initialization. This creates a race condition with cell loading scene graph modification. Closes #9269. I have read the CONTRIBUTING guidelines My code runs locally My code passes CI on my fork

S3kshun8 Games approved merge request !5469: Draft: assume GMST floats are imprecise decimals when promoting into lua doubles at OpenMW / openmw

https://gitlab.com/OpenMW/openmw/-/merge_requests/5469

Published: August 10, 2026 19:11

Closes #9251 I have read the CONTRIBUTING guidelines My code runs locally My code passes CI on my fork

~ 216 additional posts are not shown ~