-
Notifications
You must be signed in to change notification settings - Fork 77
Make bg3 plugin more robust #204
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
base: master
Are you sure you want to change the base?
Make bg3 plugin more robust #204
Conversation
…er is using the functionality
… a function to share the creating of the xml strings for ModuleShortDesc
…utable finishes, log folders removed
…cutable does not launch too fast to see it
…ocuments folder if they dont exist in the mapping function, log all directories mapped if debug is enabled
|
@spectrbro @TheForgotten69 as the only users I know of of the bg3 plugin, id be interested in your thoughts on these changes, if you have some time. Thanks 🤝 |
|
@Holt59 this is ready for review. |
|
I will test more thoroughly but so far:
I never really dig into what was the root cause but it seems that this MR mitigates this but don't entirely fix it. Not sure if there are logs I can dig/share regarding this @daescha |
How big are we talking @TheForgotten69 ? i have it working with 300+ mods just fine. |
I have ~450 mods including textures that were auto converted, not sure what is the culprit though. I think it tries to do some operation and crashes. |
|
@TheForgotten69 a stacktrace or any further context (crash message, or just log messages immediately before crash) on the crash would help. Might be a stacktrace in mo_interface.log. I cant think of any reason itd be able to handle 300 mods but not 450, so definitely will need that context to solve anything. |
|
Okay the log isn't that helpful sadly: And it's with the maximum logging level. It only occurs with this extension, the previous I used/all the other games worked fine. The crash dump is also unhelpful: |
|
@TheForgotten69 yeah that stack trace is useless (to me, at least: im not well versed in reading core dumps) unfortunately. Heres how I would debug if I had this issue: go through the code and comment out functionality until you can get it to launch, then work backward to find where the crash is happening. Its probably during the initialization phase.
have you tried reducing the number of mods to find out what that limit is? |
|
Hello, I'm writing to report an issue with the MO2 plugin for Baldur's Gate 3. When using MO2 version 2.5.2, the plugin fails to detect Baldur's Gate 3—the game does not appear in the selectable games list. This doesn't seem to be an isolated issue, as others have experienced it as well, though the exact cause is unclear. Interestingly, an alternative script from Nexus Mods (https://www.nexusmods.com/site/mods/807?tab=files) is able to detect the game normally, although that script currently has other issues. On a related note, I noticed your team has successfully implemented file mapping and load order management for BG3. Would it be possible to create a similar MO2 support plugin for Divinity: Original Sin 2? I've actually made a script that allows MO2 to recognize and manage DOS2, but I still have to manually edit the modsettings.lsx file to handle load order—my attempts to map the MO2 mod list order automatically were unsuccessful. The mod structure for DOS2 is very similar to BG3, involving extracting meta.lsx from unpacked .pak files. Thank you for your work and consideration. |
|
If I had to guess I would say that the issue you are experiencing is due to not having the yaml library available locally @MeifanuQAQ. This pr makes it so the yaml library is only imported if the feature using it is enabled. @Holt59 I think this should be merged to address these issues, as it will continue to be a problem until there is a new release of MO2. |
Uh oh!
There was an error while loading. Please reload this page.