Skeetendo

’Cause all games were better on the GBC

You are not logged in.

#1 2017-01-30 02:37:11

Maniac379
Member
Registered: 2015-01-15
Post 33/71

[pokecrystal] New Bag Section

I was looking at the disassembly for Polished Crystal, the engine > items.asm file and tried to emulate changes to make my own berry bag pocket.  It seemed simple enough but it doesn't work.  Not sure why, if there's another file that needs editing?

I figured out how to make a branch and put comments on my commits.

https://github.com/maniac379/White-Crys … e/berrybag

(Also curious on running shoes and physical/special split asm)


---
Pokémon White Crystal
A simple quality of life and gameplay improved hack

Offline

#2 2017-01-30 03:30:42

Rangi
Member
Registered: 2016-05-09
Post 498/631

Re: [pokecrystal] New Bag Section

Here are my commits for Physical/Special split, Running Shoes, and Medicine and Berry pockets.

Note that I don't usually get everything working in one commit. For instance, after implementing the new Bag pockets I noticed that the catching-tutorial dude's bag wasn't updated, so that got fixed. And the Physical/Special split was one of the first things I did with assembly, along with adding the Fairy type, so that commit might be incomplete.

Good luck!

Last edited by Rangi (2017-01-30 03:33:59)


ROM hack: Pokémon Polished Crystal — version 2.2.0 released!

Online

#3 2017-01-31 02:30:35

Maniac379
Member
Registered: 2015-01-15
Post 34/71

Re: [pokecrystal] New Bag Section

Thanks!  I got the running shoes and the physical/special split taken care of.  My code for the physical special split was a little weird because I had a bird type as well as numerous normal types for some reason.  I added fairy even though I won't use it.  Figured it out though.

Pocket is a lot of changes....coming back to that one later (I really only want the one pocket).


---
Pokémon White Crystal
A simple quality of life and gameplay improved hack

Offline

#4 2017-03-03 03:30:27

Maniac379
Member
Registered: 2015-01-15
Post 43/71

Re: [pokecrystal] New Bag Section

I tried again to do the bag section, this time straight copying both pockets:

https://github.com/maniac379/White-Crys … ca06921aa6

I get an error "Unable to load fixed WR00"

I didn't change a couple of the seemingly random numbers nor the day palette as you did.  I believe I copy/pasted only 1 of your files completely over: engine/pack.asm...I feel like I maybe did another but can't imagine and think that was going to be item attributes but I realized you changed a lot of items....

Dunno right now


---
Pokémon White Crystal
A simple quality of life and gameplay improved hack

Offline

#5 2017-03-05 22:51:26

Maniac379
Member
Registered: 2015-01-15
Post 45/71

Re: [pokecrystal] New Bag Section

I tried manually copying changes into engine/pack.asm

https://github.com/maniac379/White-Crys … 069e581dd0

Same error.  Unable to load fixed WR AMX section into bank $00

Last edited by Maniac379 (2017-03-06 01:06:00)


---
Pokémon White Crystal
A simple quality of life and gameplay improved hack

Offline

#6 2017-03-13 22:47:50

Rangi
Member
Registered: 2016-05-09
Post 569/631

Re: [pokecrystal] New Bag Section

I didn't change a couple of the seemingly random numbers nor the day palette as you did.

This is probably the cause of your compile error. It sounds like one of the banks in wram.asm has too much data in it. When I added space for the new pockets, I had to remove other unused space.

I'd suggest undoing all the bag-related commits and trying again, but to add just one pocket. And examine what each change is doing before you copy them over. For instance, I see you copied this:

-wKickCounter::
-wPresentPower:: ds 1 ; c689
+wKickCounter:: ds 1 ; c689

This was just me getting rid of an alternate name for that WRAM address, since I removed the Present attack and don't need wPresentPower any more. You don't need to copy it for the new bag pockets to work.

Also, my TM Pocket has a completely different implementation because I store TMs separately from other items. (You can only have one of each, can't throw them away, and it frees up 50 slots for more items.) So you'll need to ignore changes related to that.

Last edited by Rangi (2017-03-13 22:48:15)


ROM hack: Pokémon Polished Crystal — version 2.2.0 released!

Online

#7 2017-03-14 01:42:14

Maniac379
Member
Registered: 2015-01-15
Post 46/71

Re: [pokecrystal] New Bag Section

I appreciate the response.

I looked back through the code and didn't notice any other changes like wPresentPower which I then fixed as you mentioned.  Super helpful to understand that the error is due to lack of space in the ROM bank for WRAM, makes sense.  I went back through and tried to delete things I wouldn't use.  From what I can tell I added 4 lines/dunno how many bytes to the one bank of WRAM and 10 lines/bytes to a different bank in WRAM.  This is based off of your changes to WRAM, how much you actually added.  I did not add anything more than you.  I then removed lucky number codes to compensate for the 4 lines (I assume to avoid crashing later would have to remove the lucky item radio script).  I then removed the dude lines as seen which takes up like 12 lines and but I still get not enough room.  I tried deleting more from the dude bank.

https://github.com/maniac379/White-Crys … be8fdcaad2

I noticed that all of this is in BANK [1]?  Where as my error says WRAMX into bank $00 which seems like it would be the first bank which I did not change?  Once I undid removing present power it should be the same amount of code that was originally there? 

What is the wram.o file?

Last edited by Maniac379 (2017-03-14 01:42:29)


---
Pokémon White Crystal
A simple quality of life and gameplay improved hack

Offline

Board footer

Powered by FluxBB