pk-hack/coilsnake

coilsnake's Issues

Produced by Itoi/Presented by Nintendo Graphics

Should be simple

Change /s to os.sep

For portability

EB Items: Misc Flags

Should be separated into different options

Window Flavor Titles: Need to be relocated on write

If you rename the window flavor titles, it won't move the text if it's longer than the originals.

Town Map Icons

Like Ness's head, etc.

Should be fairly straightforward.

PSI Animations Module

Should import/export in animated gif format.

Gas Station Graphics

Only 2 palettes, so just need to export/import twice

sprite group unknowns

they have something to do with collision

Item names must be null-terminated

Item names must end with a 00 byte at the end, unlike enemy names. Currently in-game wackiness occurs if you have an item with 25 characters in it.

Town Map Icon Positions Module

import/export to/from yml

Warn user about inability to load native comp/decomp library

As it stands, the user has no way of knowing if the native library is being used successfully

Default Values for YML files

PSI Position:
<Lyrositor> Oh, I think I understand the coordinate thing. <Lyrositor> y is the row (there are three in total), and x is the location of the alpha, beta, gamma or omega symbol. I'll check out a bit more. <Lyrositor> Yup, it makes sense; every omega is at 15. <Lyrositor> Shouldn't the X be automated? <Lyrositor> Since it's always the same thing.

PSI Type:
(Also, "Type" should be automated)
1- Offense
2 - Recovery
4- Assist
8-Other

Add values for Unknown A to EBYAML in SpriteGroupModule

Apparently the values are known and are in JHack

ROM Expander gui does not work in Windows

Needs to be fixed, throws an exception in Windows

Better error message for when free space runs out

Remove redundant ROM expansion code

Replace all references to EbRomExpander.expand() with calls to Rom.expand()

Misc Text error

When changing the text in the status window, if your new text is shorter than the original, the extra characters at the end of the original will be printed after your new text
which status window text?
I'm not sure if there's some way to clear out the old text, and then put your new text in
For me, it was changing "Psychic Points:"
My "Magic Points:" was two characters shorter, so I just put two spaces at the front

Make the call to the CCScript compiler from CccInterfaceModule, not CoilSnakeGUI

Currently, the GUI frontend is the only way that CCC is automatically called.

This fix will make the command line client just as functional.

Misc Text Module

Should be able to edit the things in the Misc Text Editor

"Effect" in psi_ability_table.yml should be "Action"

Decompile/Compile/Decompile Cycle Broken

Cannot decompile/compile/decompile a fresh rom. Breaks upon 2nd decompile.

EB Death Screen

Should be a simple compressed graphic

Error messages for image importing

Should tell the user if it's

• Not a valid image
• Not an indexed PNG

Add Town Map attrs to map_sectors.yml

Town Map Arrow
Town Map Image
Town Map X
Town Map Y

Some kind of upgrade mechanism for projects

The format of files in a project may change between versions CS. Or a new file may be added in a new version of CS.

There needs to be some smooth way for a user to load their CS 1.0 project into CS 1.1

Enlarge maximum label size for table pointer fields

Text Pointer 1: Test.this_identifier_is_looooooooooooong

CoilSnake complains that it is an invalid label. Shortening the label solves the problem.

Effect should be renamed to Action

In various files (item_configuration_table.yml and psi_ability_table.yml), the term Effect is used to describe an Action; for consistency, it would be better to use Action.

Music Module

Should be able to import and export EBM files

Create a Test Plan

Put it in the wiki

At a minimum, should include some stuff like

• Decompiling a clean EB ROM
• Compiling a clean project to an EB ROM
• Testing the re-compiled EB ROM in zsnes
• Decompiling a re-compiled EB ROM and diffing the new project with a clean project

Does EB++ use byte 0x2e9200?

Does EB++ use byte 0x2e9200?

If it does, this is a bug in EB++ since 0x2e9200 is still EB data, and there must be workaround in CoilSnake since 0x2e9200 is not part of the free range.

Better error message for when a user provides invalid pointer table entry data

ie: if a user does "Text Pointer: 0" (instead of $0) or "Text Pointer: a.non_existent_label", the message is very unclear

TilesetModule: Moonside Issue

Moonside is "completely black" in compiled roms from CS 1.2 and 1.1 (it is fine in 1.0)

Probably has something to do with palette animation

Change Import/Export to Decompile/Compile in GUI

The import/export terminology is extremely confusing

Organize files

move modulelist.txt, romtypes.yaml, structures/, and ips/ to resources/

"scummers" should be "summers"

Typo for the "Town Map" option in the sector data.

Not only does the typo have to be corrected, but "scummers" has to be accepted in new versions. This can possibly be done through a version upgrade (or not).

Clarify unclear values in enemy_configuration_table.yml

There are a few unclear values in enemy_configuration_table.yml:

• Run Flag should be either true instead of 7 or false instead of 8,
• Mirror Success Rate should display as a percentage, instead of an integer going from 0 to 100.
• Miss Rate should maybe be reworked to a percentage if possible? It currently seem to be an integer on 16.
• Item Rarity should be a percentage instead of 2^i/128, i being the specified value.
• Boss Flag should probably be true/false instead of 1/0.

Python-version of comp/decomp are broken

Replace with Michael1's version

Re-sync with the main EBYAML repo

Self explanatory

Unit Tests for TilesetModule

Currently might have a bug, the palette is messed up for Deep Darkness when saving:

Write unit tests to check:

• if importing is being done correctly
• if compiling is being done correctly

Module for a ROM other than EB

Just to make sure that this'll work. Just wanna make sure the framework allows this without problems.

EB0 might be good.

Numbering issue with PSI Names

<Lyrositor> It uses a different numbering system. <Lyrositor> So PSI Name starts at 1... and psi_name_table starts at 0. <Lyrositor> That should probably be fixed too?

Deep Darkness Tileset Not Saved Correctly

The palette becomes garbled

Steps to reproduce:

• Import a project from a ROM
• Compile the project to a new ROM
• Load the ROM in emulator, go to Deep Darkness

Gas Station Screen Module

The problem here is how to allow the user to edit the animated palette in a straightforward way.

v1.2 Release

• New logo image
• Update version strings
• Update the Test Plan
• Go through the Test Plan
• Release

Credits Font Module

Should be fairly straightforward.

Update Version Number & Help Text for Version 1.1

File Select Battle Background Saving Bug

The file select battle bg (background 0xe6) is not saved correctly to the ROM.

Steps to reproduce:

• Create a new project
• Compile the new project to a rom
• Go to the file select screen OR enter a battle with bg 0xe6

Battle Swirl Module

PNG for each frame, folder for each swirl

Font #3 Width Not Editable

As it stands, the widths for the characters in font 3 (the battle font) are not editable. That is, when you make a change, the change is not reflected in actual gameplay.

JHack uses a patch created by MrA to bypass this.

Title Screen Module

Challenges:

• How to allow the user to edit the palette animation in a straightforward way
• How to allow the user to edit the title screen enimation in a straightforward way