I am not planning on migrating to patch 8 until I complete my current playthrough so I cannot personally say if this mod works or not.
If you're having any issues with the mod on patch 8 please leave a bug report or comment and I'll look into it.
For anyone having issues double check that you have the latest version of Script Extender. If you've done that and are experiencing any issues please still make a bug report or leave a comment.
Edit: I've migrated to patch 8 and did some initial testing. I didn't find any issues so I'm going to call this patch 8 compatible. I don't have that many saves to test with so I wasn't able to check all possibilities, so if you encounter any troubles please let me know.
I can only put gnome corpses in the bag, I was hoping I'd be able to carry all the corpses ive picked for my necromancy character. I also have the weight limit turned off, I was wondering if theres something wrong on my end, or if carrying corpses was not the intended purpose.
I didn't test with corpses at all. I wouldn't expect there to be an issue. Can you not add the other corpses to the container at all or do they go inside but the weight isn't reduced? Also I'm going to move this into the Bugs section.
While the mod is working in Patch 8, SE is throwing this message anytime something is added to the bag
bg3se::esv::lua::OsirisCallbackManager::RunHandler(): Osiris event handler failed: [string "BoHReforged/Server/BagOfHolding.lua"]:475: attempt to index a nil value (local 'objectEntity') stack traceback: BoHReforged/Server/BagOfHolding.lua:475: in function <BoHReforged/Server/BagOfHolding.lua:471>
No issues when pulling something out, but I found it odd and thought it might be something to look into.
Thank you for making this mod! It's absolutely lovely <3
Thanks for letting me know. Could you also please provide the following information:
Your game version as displayed in the lower right of the BG3 main menu
Your SE version as displayed in the lower left of the BG3 main menu
Your Bag of Holding Reforged version as displayed at the bottom of the Bag of Holding Reforged MCM menu.
Do you still get the errors in a new save file or only in an existing save?
This looks like an error from an unexpected SE function return value. This could be related to changes in SE which I may need to look into.
The part of the code it's coming from is one that checks to make sure the weight reduction status isn't applied to items that shouldn't have it. The status can be incorrectly applied to items when spitting a stack if the destination of the new stack is outside of the bag. This is also related to ensuring that the bag's stored weight remains correct when removing items from a stack inside the bag. These issues are more subtle but if this error is being thrown I'd expect one of them to be happening.
(Sadly I have to check anytime the weight reduction status is applied to ensure it's a correct application, so it does make sense to get this error when adding things to the bag.)
oohh~ Nifty! I always love learning about how mods are coded
Game Version: Patch 8, but I'll check today's hotfix (I installed it before I got your message) SE Version: 23 (now v24 with the hotfix 30) BoH Version: 2.1.4
This is a new save I'm on with Patch 8. I killed all my old saves from back in the days of hotfixes for Patch 6. (I got burned out doing mod support and needed a break)
The error had been happening for a few days when putting items in the BoH, whether at the trader, looting, or rummaging in the camp chest... but I can't seem to replicate the thrown error post-hotfix 30. So, now I'm wondering if Larian did something to fix the issue 🤣
Thank you for your time! I'll keep an eye on the console if it randomly starts happening again!
I've tried some testing since receiving your report and can't seem to reproduce the error. For testing I use either a new save or one of the completely unmodded saves I have specifically for mod testing. This means that sometimes it can be hard for me to identify an issue if it's caused by migrating an older save to a new patch/hotfix. I would be quite surprised if the error you reported is related to this however.
I've mostly been doing some mod work learning to make hairs and tattoos since patch 8 came out so I haven't done very much play testing of the Bag of Holding myself. So I haven't seen any patch 8 issues but it's possible I just missed them.
I don't recall exactly what's inside of the SE function that's likely the source of the error you reported so it's possible the Larian update changed that somehow. Or it could be that something changed with the SE update. Please do let me know if the error pops up again for you.
Hi, the bag doesn't seem to reduce weight... Can't figure it out... Script Extender is v20, game is latest. The order is correct ('Mod Configuration Menu' and then 'Bag of Holding Reforged'). Any idea what I'm missing? (https://disk.yandex.ru/d/PVaSq899bH8cyA)
I've no idea why, but this mod doesn't seem to be working for me. After checking the SE Console, I found the following; any ideas or insight would be greatly appreciated: Error while dispatching event GameStateChanged: [string "BoHReforged/BootstrapServer.lua"]:10: attempt to call a nil value (field 'GameStateRunning') stack traceback: BoHReforged/BootstrapServer.lua:10: in function <BoHReforged/BootstrapServer.lua:8> [C++ Code]: in function 'xpcall' builtin://Libs/Events/SubscribableEvent.lua:124: in method 'Throw' builtin://Libs/Events/EventManager.lua:17: in method 'ThrowEvent' builtin://Libs/Events/EventManager.lua:107: in function <builtin://Libs/Events/EventManager.lua:106>
bg3se::lua::State::LoadScript(): Failed to execute script: [string "BoHReforged/Server/BagOfHolding.lua"]:32: attempt to index a nil value (field 'BG3MCM') stack traceback: BoHReforged/Server/BagOfHolding.lua:32: in main chunk [C++ Code]: in field 'Include' builtin://Startup.lua:31: in field 'Require' BoHReforged/BootstrapServer.lua:6: in main chunk [C++ Code]: in field 'Include' builtin://BuiltinLibrary.lua:57: in function <builtin://BuiltinLibrary.lua:27>
Hi I had the same error message, checked my load order, MCM happened to be loaded after BoH, fixed it, moved the bag out of my inventory and now it works. Check your load order, make sure MCM loads before the BoH.
I responded to your DM but I've also now included the localization file under the miscellaneous category in the files section of this mod page for others who are interested in localizing but aren't that familiar with packing and unpacking mods.
For those interested in localizing please note that the provided file does need to be either packed as it's own mod or included with the main mod to work. If you simply want to submit the localized file back to me I'll happily include it with the main mod and credit you.
58 comments
If you're having any issues with the mod on patch 8 please leave a bug report or comment and I'll look into it.
For anyone having issues double check that you have the latest version of Script Extender. If you've done that and are experiencing any issues please still make a bug report or leave a comment.
Edit: I've migrated to patch 8 and did some initial testing. I didn't find any issues so I'm going to call this patch 8 compatible. I don't have that many saves to test with so I wasn't able to check all possibilities, so if you encounter any troubles please let me know.
bg3se::esv::lua::OsirisCallbackManager::RunHandler(): Osiris event handler failed: [string
"BoHReforged/Server/BagOfHolding.lua"]:475: attempt to index a nil value
(local 'objectEntity')
stack traceback:
BoHReforged/Server/BagOfHolding.lua:475: in function <BoHReforged/Server/BagOfHolding.lua:471>
No issues when pulling something out, but I found it odd and thought it might be something to look into.
Thank you for making this mod! It's absolutely lovely <3
This looks like an error from an unexpected SE function return value. This could be related to changes in SE which I may need to look into.
The part of the code it's coming from is one that checks to make sure the weight reduction status isn't applied to items that shouldn't have it. The status can be incorrectly applied to items when spitting a stack if the destination of the new stack is outside of the bag. This is also related to ensuring that the bag's stored weight remains correct when removing items from a stack inside the bag. These issues are more subtle but if this error is being thrown I'd expect one of them to be happening.
(Sadly I have to check anytime the weight reduction status is applied to ensure it's a correct application, so it does make sense to get this error when adding things to the bag.)
Game Version: Patch 8, but I'll check today's hotfix (I installed it before I got your message)
SE Version: 23 (now v24 with the hotfix 30)
BoH Version: 2.1.4
This is a new save I'm on with Patch 8. I killed all my old saves from back in the days of hotfixes for Patch 6. (I got burned out doing mod support and needed a break)
The error had been happening for a few days when putting items in the BoH, whether at the trader, looting, or rummaging in the camp chest... but I can't seem to replicate the thrown error post-hotfix 30. So, now I'm wondering if Larian did something to fix the issue 🤣
Thank you for your time! I'll keep an eye on the console if it randomly starts happening again!
I've mostly been doing some mod work learning to make hairs and tattoos since patch 8 came out so I haven't done very much play testing of the Bag of Holding myself. So I haven't seen any patch 8 issues but it's possible I just missed them.
I don't recall exactly what's inside of the SE function that's likely the source of the error you reported so it's possible the Larian update changed that somehow. Or it could be that something changed with the SE update. Please do let me know if the error pops up again for you.
Error while dispatching event GameStateChanged: [string "BoHReforged/BootstrapServer.lua"]:10: attempt to call a nil value (field 'GameStateRunning')
stack traceback:
BoHReforged/BootstrapServer.lua:10: in function <BoHReforged/BootstrapServer.lua:8>
[C++ Code]: in function 'xpcall'
builtin://Libs/Events/SubscribableEvent.lua:124: in method 'Throw'
builtin://Libs/Events/EventManager.lua:17: in method 'ThrowEvent'
builtin://Libs/Events/EventManager.lua:107: in function <builtin://Libs/Events/EventManager.lua:106>
bg3se::lua::State::LoadScript(): Failed to execute script: [string "BoHReforged/Server/BagOfHolding.lua"]:32: attempt to index a nil value (field 'BG3MCM')
stack traceback:
BoHReforged/Server/BagOfHolding.lua:32: in main chunk
[C++ Code]: in field 'Include'
builtin://Startup.lua:31: in field 'Require'
BoHReforged/BootstrapServer.lua:6: in main chunk
[C++ Code]: in field 'Include'
builtin://BuiltinLibrary.lua:57: in function <builtin://BuiltinLibrary.lua:27>
I had the same error message, checked my load order, MCM happened to be loaded after BoH, fixed it, moved the bag out of my inventory and now it works.
Check your load order, make sure MCM loads before the BoH.
Boy, does this game need a mod sorter.
For those interested in localizing please note that the provided file does need to be either packed as it's own mod or included with the main mod to work. If you simply want to submit the localized file back to me I'll happily include it with the main mod and credit you.