Guys, I was given this script created by Josh Camas.
Its supposed to change the paperdoll background according to day/night and weather + dungeon location etc,
Its all described in the readme there.
Issue is:
when we started and it had only day/night variant, it worked. When it was extended with weather, it just stopped doing anything and since than we did not move anywhere (we bug fixed some issues on my side but it did not solve the issue)
So if any coder can take a look, and maybe see whats wrong there, I would be really happy.
I write it here because I want to release DREAM this year, and time is running up, and Josh does not have as much time as before. I hope it all right with him, I know its his script, but bugfixing it is always good for everyone. Plus its public on Git.
Thanks a lot
Link to script
https://github.com/joshcamas/dfu-charac ... nd-swapper
(Bug Fix) Can someone take a look at this script made by JoshCamas please
- King of Worms
- Posts: 4753
- Joined: Mon Oct 17, 2016 11:18 pm
- Location: Scourg Barrow (CZ)
- Contact:
(Bug Fix) Can someone take a look at this script made by JoshCamas please
DREAM Nexus https://www.nexusmods.com/users/73250038?tab=user+files
DREAM Forums viewtopic.php?f=27&t=1168
DISCORD: king_of_worms_cz
DREAM Forums viewtopic.php?f=27&t=1168
DISCORD: king_of_worms_cz
- pango
- Posts: 3359
- Joined: Wed Jul 18, 2018 6:14 pm
- Location: France
- Contact:
Re: (Bug Fix) Can someone take a look at this script made by JoshCamas please
From a quick read, there's a few intriguing stuff, but something that's very certainly a bug is texture being assigned in loops l. 120, without a check whether it got assigned null or something else; So the value of texture after the loops will just be the last assigned value, no matter if that was the "successful" assignment of not (I'm not sure I'm very clear, I hope it makes sense).
By the way, fixing this will require to make sure it's the most "specific" texture that's picked by those loops, in case they're several candidate textures...
I think it would help to require tags to be in a specific order, say weather then day/night, to lower the number of filename checks. That should also simplify the logic.
If you have trouble getting it right, add unit tests.
By the way, fixing this will require to make sure it's the most "specific" texture that's picked by those loops, in case they're several candidate textures...
I think it would help to require tags to be in a specific order, say weather then day/night, to lower the number of filename checks. That should also simplify the logic.
If you have trouble getting it right, add unit tests.
Mastodon: @pango@fosstodon.org
When a measure becomes a target, it ceases to be a good measure.
-- Charles Goodhart
When a measure becomes a target, it ceases to be a good measure.
-- Charles Goodhart
- King of Worms
- Posts: 4753
- Joined: Mon Oct 17, 2016 11:18 pm
- Location: Scourg Barrow (CZ)
- Contact:
Re: (Bug Fix) Can someone take a look at this script made by JoshCamas please
Thanks a lot I have no idea how hard that would be to do. Also Im reluctant because the script is not mine.
Either way I apreciate you pointing us to the direction of possible issue. I hope Josh will come back. And also implement the last part he said he can do - dungeon background switch.
Either way I apreciate you pointing us to the direction of possible issue. I hope Josh will come back. And also implement the last part he said he can do - dungeon background switch.
DREAM Nexus https://www.nexusmods.com/users/73250038?tab=user+files
DREAM Forums viewtopic.php?f=27&t=1168
DISCORD: king_of_worms_cz
DREAM Forums viewtopic.php?f=27&t=1168
DISCORD: king_of_worms_cz
- Hazelnut
- Posts: 3016
- Joined: Sat Aug 26, 2017 2:46 pm
- Contact:
Re: (Bug Fix) Can someone take a look at this script made by JoshCamas please
Do you use this in Dream KoW? And did you get it fixed yet?
See my mod code for examples of how to change various aspects of DFU: https://github.com/ajrb/dfunity-mods