Home › Forums › Destiny of an Emperor › DoaE / DoaEditor Prioritized Bug List
- This topic has 151 replies, 10 voices, and was last updated 1 year, 4 months ago by DragonAtma.
-
AuthorPosts
-
September 7, 2013 at 4:48 pm #12543MiDKnighTModerator
Most Important DoaE / DoaEditor Bugs as of 1/20/2014:
—————————————————————————————————–
Bugged item: IPS 1.2 (1.3 untested) – Enemy uses Raise and attacks their own party
Impact: : Game Functionality Loss
Discovered by: sonic.penguin and Shingen01
Description: http://www.lordyuanshu.com/forums/topic/destiny-of-an-emperor-20-rise-of-lu-bu-beta-mod/page/35
I have also sent you a link to this via email with the mod and savestate a few weeks ago. When an enemy uses raise it does weird things.
—————————————————————————————————–
Dead Bugs
Fixed bugs after 1/2013:
—————————————————————————————————–
DoaEditor: Ding Yuan unbillitable issue
Impact: Somewhat annoying
Discovered by: sonic penguin
Description: Officer billetable when he’s supposed to be unbillletable.
MiDKnighT: You can try messing with the unbilletable list from Meterostrike’s post as a way around it:
The game maintains a list of 7 possible Generals that cannot be stashed away:
0x39580 = $9570,Y = A8 A9 AA AC AD BC D5More info: http://www.lordyuanshu.com/forums/topic/destiny-of-an-emperor-rom-hacking-guide-1/page/63#post-38981
Fixed in DoaEditor 0.98r_4+
—————————————————————————————————–
DoaEditor: 3 original officers bugImpact: Mildly Annoying
Discovered by: sonic penguin
Description: I manually changed the POL values for all the officers in game, exported and restarted DOAEditor, however, I am getting the following error upon selecting the Liu Bei, Zhang Fei, and Guan Yu officers (AA, A9, A6) and everyone else seems to be fine even the other tigers. It just seems specific to those 3 main guys.
MiDKnighT: The LDR tables are at 0x7AA## where ## is the officer ID. I looked at the ROM you sent me and Cao Cao’s LDR is at “FF 00 00″ (A6-A8). When I opened the game his LDR was down to 195. Just traced it and the game is looking up the A8 value which is 00 so it’s deriving its own (thanks to my code FLEXX )
So until I fix DoaEditor the solution is to just manually put FF (255) in the table at all 3 spots:
0x7AAA6: FF FF FF
The DoaEditor bug is with officer with multiple officer IDs. Way around is to manually update the tables:
To set the LDR for an officer, put the value at 0x7AA## where ## = officer ID
To set the VIT for an officer, put the value at 0x7AB## where ## = officer ID
To set the POL for an officer, put the value at 0x7AC## where ## = officer IDI’ll have to see how Niahak handled those cases in his code.
More info: http://www.lordyuanshu.com/forums/topic/new-mod-hero-of-chaos-cao-cao-mod/page/4/#post-52985
Fixed in DoaEditor 0.98r_4+
—————————————————————————————————–
DoaEditor: Level 31-60 Tactics Not Saving in Profiles
Impact: Somewhat annoying
Description: Changed a tactic on page 2 to Lian Huo and when exported it reverted it back to (None)
MiDKnighT: The way around this would be to update the tactic profiles directly:
0xE9000-0xE903F = Level up profile 00
0xE9040-0xE907F = Level up profile 01
0xE9080-0xE90BF = Level up profile 02
0xE90C0-0xE90FF = Level up profile 03
0xE9100-0xE913F = Level up profile 04
0xE9140-0xE917F = Level up profile 05
0xE9180-0xE91BF = Level up profile 06
0xE91C0-0xE91FF = Level up profile 07
0xE9200-0xE923F = Level up profile 08
0xE9240-0xE927F = Level up profile 09
0xE9280-0xE92BF = Level up profile 0A
0xE92C0-0xE92FF = Level up profile 0B
0xE9300-0xE933F = Level up profile 0C
0xE9340-0xE937F = Level up profile 0D
0xE9380-0xE93BF = Level up profile 0E
0xE93C0-0xE93FF = Level up profile 0F
0xE9400-0xE943F = Level up profile 10
0xE9440-0xE947F = Level up profile 11
0xE9480-0xE94BF = Level up profile 12
0xE94C0-0xE94FF = Level up profile 13
0xE9500-0xE953F = Level up profile 14
0xE9540-0xE957F = Level up profile 15
0xE9580-0xE95BF = Level up profile 16
0xE95C0-0xE95FF = Level up profile 17
0xE9600-0xE963F = Level up profile 18
0xE9640-0xE967F = Level up profile 19
0xE9680-0xE96BF = Level up profile 1A
0xE96C0-0xE96FF = Level up profile 1BFix: Added a save button to the profiles page. -MiDKnighT
—————————————————————————————————–
IPS 1.3: AC bar too short
Impact: Somewhat annoying
Discovered by: sonic penguin
Description: The AC bar, even with 200 AC is shorter than the AP bar which might have a weapon with a numerically less number. Is there a way to actually display an officers base AC without it just being black? For example, see below. 150 AC and that tiny bar.
More Info: http://www.lordyuanshu.com/forums/topic/destiny-of-an-emperor-rom-hacking-guide-1/page/62?message=spammed#post-38718
Fix: http://www.lordyuanshu.com/forums/topic/new-mod-hero-of-chaos-cao-cao-mod/page/6/#post-53012Fixed by MiDKnighT
Solution:
0x385C0: 2019CCE8AD4F7E
0×78620: EAEA4C5086B930868503EAEA4C508600
0×78660: 8D4F7EA9004CA185—————————————————————————————————–
IPS 1.3: Highest tactic learned bug.
Impact: Major game functionality loss
Discovered by: sonic penguin
Description: Setting the highest tactic learned is not working properly.
More Info: http://www.lordyuanshu.com/forums/topic/new-mod-hero-of-chaos-cao-cao-mod/page/4/#post-52976
Fixed the tactic bug. Problem was that it was comparing the tactic priority of Raise to the slot of Life. It should have been comparing priority vs priority. Change is very simple:
0xEB7FD: “85″–>”86″
—————————————————————————————————–
IPS 1.3: Armor / profile bug.
Impact: Highly annoying
Discovered by: sonic penguin
Fixed by: MiDKnighT 9/19/2013
Solution:
0xE9BC0: 4B4C4D4E4F (this is a list of non-weapon icons that can be equipped by anyone)
http://www.lordyuanshu.com/forums/topic/tactic-expansion-ideas/page/6?replies=110#post-35657
This was in IPS 1.3 patch 1 but not 2. I removed it from patch 1 and put it in patch 2.
—————————————————————————————————–
Description: It won’t let you equip armor unless it’s in the officer’s profile.
IPS 1.2 and 1.3: The infamous “last guy going first” freeze bug.
Impact: Major game functionality loss
Discovered by: Everybody
Fixed by: MiDKnighT 9/13/2013
Solution:
0x202B8: 2040BFEAEAEA
0x23F50: 18A90065CD85CD60
0x7A950:
0000A90E2082C46868E6AEE6AFE6ADBD
0062C91DF004C91ED008A9019DD0614C
50834C00800000000000000000000000Also corrected in IPS 1.2 and 1.3 downloaded after 9/13/2013.
—————————————————————————————————–
The infamous last guy going first freeze bug!
Description: This one has been an absolute monster to trap and catch. Both ludmeister and I have tried to trap it but have been unsuccessful thus far. If someone can get a save state with the latest IPS patch version we can hopefully nail this one.
More info: http://www.lordyuanshu.com/forums/topic/ludmeisters-remix-v30-released/page/3
Fix: http://www.lordyuanshu.com/forums/topic/doae-doaeditor-prioritized-bug-list#post-45862Solution:
0x202B8: 2040BFEAEAEA
0x23F50: 18A90065CD85CD60
—————————————————————————————————–Please reply with any open bugs not mentioned here or if the priority should be raised on one. When raising a new bug, please give in the following format:
Bugged item: (DoaEditior, IPS 1.3, etc..)
Impact:
Discovered by:
Description:
More Info:Recommended impacts are:
– Major game functionality loss
– Game functionality loss
– Highly annoying
– Somewhat annoying
– Mildly annoying- This topic was modified 10 years, 5 months ago by Lord Yuan Shu.
- This topic was modified 2 years, 8 months ago by MiDKnighT.
September 8, 2013 at 1:30 pm #45858sonic.penguinModeratorBugged item: IPS 1.3 – Base AC Issue
Impact: : Mildly Annoying
Discovered by: sonic.penguin
Description: Even modest INT officers are getting a large AC bump making them far superior to STR officers. Since stupid bruisers such as Zhang Fei, Wang Shuang, Gan Ning, etc have such low INT, they are far inferior to TACTIC users in that they have no natural defense since INT blocks against TACTICS, especially end-game. With the new VIT formula, I'm getting my STR officers having the same AC boost as my INT officers, especially end-game when AGI is higher due to chapters. This makes even modest INT officers far superior to STR officers, and high INT officers maxing out their base AC because they have stats like 240 INT, 120 STR. Base AC only defends against physical attacks.
More Info: My fix would be to lower the impact of INT and AGI and have VIT and subsequently Base AC based more on STR and VIT, like 80%, 10% agility and 10% INT. since that would indicate that the officer is actually buff and strong FLEXX and can resist the physical damage blows much better than the bookish type. Granted you could always have special cases ie: Jiang Wei, Tiger generals, Sima Yi types. It just makes more sense in my head I guess
Ignore Troop Count, and just look at AC which only affects Physical Attack Defense:
In my opinion, Lu Su should only have like 20 AC while Gan Ning would be near perfect… and hopefully a bar fix can be made too but that was mentioned above >_>
September 8, 2013 at 7:39 pm #45859sonic.penguinModeratorBugged item: IPS 1.2 (1.3 untested) – Enemy uses Raise and attacks their own party
Impact: : Game Functionality Loss
Discovered by: sonic.penguin and Shingen01
Description: http://www.lordyuanshu.com/forums/topic/destiny-of-an-emperor-20-rise-of-lu-bu-beta-mod/page/35
I have also sent you a link to this via email with the mod and savestate a few weeks ago. When an enemy uses raise it does weird things.
More Info:
September 9, 2013 at 1:25 pm #45860MiDKnighTModeratorQuote:Bugged item: IPS 1.3 – Base AC IssueThis one really isn't a bug per say but is a request to change the formula right? Probably instead of changing the patch we should make the formula configurable. I had documented it here:
Quote:Another interesting usage idea for VIT. If it resists damage so much, maybe have an offense tactic called virus or plague or something that essentially drops the targets VIT and causes them to take more damage. I was also setting a breakpoint to check something else and stumbled across your base AC code (at least I think it is) here:1E:987F:B9 9C 61 LDA $619C,Y @ $619E = #$B4
1E:9882:4A LSR
1E:9883:65 03 ADC $0003 = #$00
1E:9885:85 03 STA $0003 = #$00
1E:9887:B9 A8 61 LDA $61A8,Y @ $61AA = #$E1
1E:988A:4A LSR
1E:988B:4A LSR
1E:988C:65 03 ADC $0003 = #$00
1E:988E:85 03 STA $0003 = #$00
1E:9890:B9 B4 61 LDA $61B4,Y @ $61B6 = #$31
1E:9893:4A LSR
1E:9894:4A LSR
This formula is almost exactly what I used for deriving a default value for VIT. But I do allow the modder to override the derived default. So how about this? In the above code:
– Replace 619C with 6410 (Replace STR with VIT)
– Replace 61A8 with 619C (Replace INT with STR)
– Leave AGI as it is.
http://www.lordyuanshu.com/forums/topic/destiny-of-an-emperor-rom-hacking-guide-1/page/55#post-35197
In the IPS patch it is:
50% VIT
25% STR
25% AGI
You could mess with the above code to give the formula you want. Your numbers (80% 10% 10%) would take more code to come up with.
September 9, 2013 at 1:32 pm #45861MiDKnighTModeratorThis looks like a good trace log – digging through it…
Quote:I managed to run a trace log file that is relatively small and had the bug occur. Last guy went and it froze during Trace Log. Let me know if I did it right. Here is the file from Flames of Wu titled FOWSeptember 9, 2013 at 2:11 pm #45862MiDKnighTModeratorI think I fixed the infamous last guy going first freeze bug!!!
Findings:
$D4 = BA87 (use 87BA for lookups)
Y value is too high (15/16)
A:87 X:0A Y:15 S:20 P:NvUbdIzc $8556:85 D5 STA $00D5 = #$01
A:87 X:0A Y:15 S:20 P:NvUbdIzc $8558:B1 D4 LDA ($D4),Y @ $87CF = #$20
A:20 X:0A Y:15 S:20 P:nvUbdIzc $855A:AA TAX
A:20 X:20 Y:15 S:20 P:nvUbdIzc $855B:C8 INY
A:20 X:20 Y:16 S:20 P:nvUbdIzc $855C:B1 D4 LDA ($D4),Y @ $87D0 = #$2D
A:2D X:20 Y:16 S:20 P:nvUbdIzc $855E:85 D5 STA $00D5 = #$87
A:2D X:20 Y:16 S:20 P:nvUbdIzc $8560:86 D4 STX $00D4 = #$BA
A:2D X:20 Y:16 S:20 P:nvUbdIzc $8562:6C D4 00 JMP ($00D4) = $2D20
^^^ Jumps to wrong location
A:2D X:20 Y:16 S:20 P:nvUbdIzc $2D20:91 1E STA ($1E),Y @ $011A = #$16
A:2D X:20 Y:16 S:20 P:nvUbdIzc $2D22:11 00 ORA ($00),Y @ $0015 = #$11
A:6F X:20 Y:16 S:20 P:nvUbdIzc $2D24:F8 SED
A:6F X:20 Y:16 S:20 P:nvUbdIzc $2D26:4F UNDEFINED
So the Y value is too high. Chasing backwards we see:
A:01 X:3A Y:00 S:1E P:nvUbdIZc $8939:B1 D4 LDA ($D4),Y @ $018D = #$0A <– 0A is too high
A:0A X:3A Y:00 S:1E P:nvUbdIzc $893B:AA TAX
…
A:00 X:0A Y:01 S:1E P:nvUbdIZc $854E:8A TXA
A:0A X:0A Y:01 S:1E P:nvUbdIzc $854F:0A ASL
A:14 X:0A Y:01 S:1E P:nvUbdIzc $8550:A8 TAY
A:14 X:0A Y:14 S:1E P:nvUbdIzc $8551:C8 INY
As I found before, I'm thinking that 01 ($D5) is not correct. That value comes from:
A:8D X:3A Y:01 S:1E P:nvUbdIzc $8934:B1 CC LDA ($CC),Y @ $08D2 = #$01
line num: 208910
$CD is 1 number too high I think. Usually it is #07 when doing lookups. How does $CD go from 07 to 08?
I see $CC and $CD getting changed here:
A:17 X:22 Y:FB S:22 P:nvUbdIzC $82A8:A9 00 LDA #$00
A:00 X:22 Y:FB S:22 P:nvUbdIZC $82AA:65 CD ADC $00CD = #$07
A:08 X:22 Y:FB S:22 P:nvUbdIzc $82AC:85 CD STA $00CD = #$07
Bingo, the carry flag is set when it should not be. Later we see:
A:3C X:3A Y:01 S:22 P:nvUbdIzc $82A1:A9 1F LDA #$1F
A:1F X:3A Y:01 S:22 P:nvUbdIzc $82A3:18 CLC
A:1F X:3A Y:01 S:22 P:nvUbdIzc $82A4:65 CC ADC $00CC = #$B2
A:D1 X:3A Y:01 S:22 P:NvUbdIzc $82A6:85 CC STA $00CC = #$B2
A:D1 X:3A Y:01 S:22 P:NvUbdIzc $82A8:A9 00 LDA #$00
A:00 X:3A Y:01 S:22 P:nvUbdIZc $82AA:65 CD ADC $00CD = #$08
A:08 X:3A Y:01 S:22 P:nvUbdIzc $82AC:85 CD STA $00CD = #$08
Which did not have the carry flag. Chasing down how this carry flag got set…
There is CLC at the top here but adding F8 to 1F sets the carry flag. F8 is too high:
A:1F X:22 Y:FB S:22 P:nvUbdIzc $82A3:18 CLC
A:1F X:22 Y:FB S:22 P:nvUbdIzc $82A4:65 CC ADC $00CC = #$F8
A:17 X:22 Y:FB S:22 P:nvUbdIzC $82A6:85 CC STA $00CC = #$F8
A:17 X:22 Y:FB S:22 P:nvUbdIzC $82A8:A9 00 LDA #$00
A:00 X:22 Y:FB S:22 P:nvUbdIZC $82AA:65 CD ADC $00CD = #$07
A:08 X:22 Y:FB S:22 P:nvUbdIzc $82AC:85 CD STA $00CD = #$07
Best way to fix would be to sneak a CLC before 82AA. Which is at 0x202BA. Unfortunately there is no room to do that. So maybe best to add a little subroutine.
I'll try to sneak a little subroutine into 0x23F50 which clears the carry flag first:
0x23F50: 18A90065CD85CD60
Then call the subroutine here:
0x202B8: 2040BFEAEAEA
So the fix is:
0x202B8: 2040BFEAEAEA
0x23F50: 18A90065CD85CD60
FINALLY FIXED!
http://www.youtube.com/watch?v=76RrdwElnTU
Please test this and see if this fixes the freeze. I'll mark this as fixed unless it happens again with the above fix.
September 10, 2013 at 1:13 am #45863sonic.penguinModeratorHALLELUJAH!!!
Did that log file work? I tried really hard to trap only a snippet and not make a huge file.
September 10, 2013 at 1:40 am #45864sonic.penguinModeratorSeptember 10, 2013 at 2:14 am #45865MiDKnighTModeratorYes I used the log file to resolve it. That did the trick :)
September 13, 2013 at 12:33 pm #45871MiDKnighTModeratorUpdated the IPS patches and list with the additional "last guy going first" freeze fix.
September 13, 2013 at 4:57 pm #45872MiDKnighTModeratorQuote:
IPS 1.3: Armor / profile bug.
Impact: Highly annoying
Discovered by: sonic penguin
Description: It won't let you equip armor unless it's in the officer's profile.
I just tested this and I'm not seeing this. I'm having no problem equipping or equipping armor in IPS patch 1.3.
September 14, 2013 at 12:02 am #45873sonic.penguinModeratorOkay, here is a SS of the event. Download my latest version of FOW, apply the 1.3 IPS patch to it, and try to equip some basic equipment to an officer and it won't work. See below.
September 18, 2013 at 1:33 pm #45876MiDKnighTModeratorFor the armor thing make sure your code matches what I have here:
http://www.lordyuanshu.com/forums/topic/tactic-expansion-ideas/page/5?replies=110#post-35480
Then I had documented an armor equip bug here so make sure you have these changes too:
http://www.lordyuanshu.com/forums/topic/tactic-expansion-ideas/page/6?replies=110#post-35657
September 18, 2013 at 10:15 pm #45877sonic.penguinModeratorAll I did was apply the IPS patch 1.3 to a fresh, unaltered rom and it wasn't working. I can try the above method and see if it fixes it.
September 18, 2013 at 11:48 pm #45878MiDKnighTModeratorYa I'll check the IPS patch to make sure it has all that.
-
AuthorPosts
- You must be logged in to reply to this topic.