Skip to content

Add Nintendo Opus (.nop) format#24

Open
Geo6453 wants to merge 12 commits intokinnay:masterfrom
Geo6453:test
Open

Add Nintendo Opus (.nop) format#24
Geo6453 wants to merge 12 commits intokinnay:masterfrom
Geo6453:test

Conversation

@Geo6453
Copy link

@Geo6453 Geo6453 commented Feb 4, 2026

No description provided.

@kinnay
Copy link
Owner

kinnay commented Feb 4, 2026

Hi,

Thank you for providing this information. Before I am merging this, could you make some adjustments to the PR?

  • Please use Offset / Size / Description tables instead of Offset / Value / Description. It should be clear from the description which values can be found in a field.
  • Please capitalize the first letter of each entry of the description column.
  • Please change XB2 / Xenoblade 2 to XBC 2 / Xenoblade Chronicles 2, as this is the correct name of the game.
  • Could you mention in the documentation that the file extension is .nop?
  • What is a hexa byte?
  • Fix the typos: duracy is probably duration, and lenght should be length.
  • Finally, please make sure to use ascii characters if possible, so replace the quotes by ' and ", and replace stéréo by stereo.

@Geo6453
Copy link
Author

Geo6453 commented Feb 5, 2026

  • I use Offset / Size / Value / Description table, i hope it's a good compromise
  • Done
  • I don't like the "Chronicles" added by NoA, everyone writes XC instead of XB which doesn't follow the logic of XenoGears (XG) and XenoSaga (XS). If it's still important, I'll make the change; I'm not stubborn.
  • Done
  • It's just a value written on 4 bytes, i removed to make it clear
  • Done
  • Done

Thanks for the feedback

@kinnay
Copy link
Owner

kinnay commented Feb 5, 2026

Thanks for improving the documentation so far.

I use Offset / Size / Value / Description table, i hope it's a good compromise

I would prefer Offset / Size / Description tables. This is important for consistency with the rest of the documentation. If a value always has a specific value, such as a magic number, this value can be mentioned in the description column. The description column should be clear enough to understand which values can be found in a field.

While it can be good to provide examples, a 'value' column is not the best way to do this in my opinion. It would be better if we could provide complete example files for each format (although we would have to be careful not to infringe any copyright with that).

I don't like the "Chronicles" added by NoA, everyone writes XC instead of XB which doesn't follow the logic of XenoGears (XG) and XenoSaga (XS). If it's still important, I'll make the change; I'm not stubborn.

Interesting. The reason that I asked for this, is that all resources that I can find (wikipedia, reddit, gbatemp, etc.) refer to the game as Xenoblade Chronicles 2. Therefore, I believe that referring to the game by its full name would cause the least amount of confusion. If I type XB2 into Google, I do not get any results related to Xenoblade, while XC2 and XBC2 give much better results. If everyone writes XC, it would make sense if we did the same.

@Geo6453
Copy link
Author

Geo6453 commented Feb 6, 2026

I'll write in this format then :
| Offset | Size | Description |
| 0x00 | 4 | 73 61 64 66 = sadf |
| 0x04 | 4 | ?? ?? ?? ?? : File size / total number of bytes |

Ok, i made the change, it's true that can improve the referencement too.

@kinnay
Copy link
Owner

kinnay commented Feb 6, 2026

I guess that for magic numbers that would work, though for consistency it would be best to describe it as a magic number, and then its value in ASCII text. For example, see here: https://nintendo-formats.com/libs/aal/barslist.html

I don't really see the point of adding the ?? ?? ?? ??. How do those question marks improve the documentation?

@kinnay
Copy link
Owner

kinnay commented Feb 6, 2026

(I don't see the change btw)

@Geo6453
Copy link
Author

Geo6453 commented Feb 6, 2026

(I doesn't push yet)

@Geo6453
Copy link
Author

Geo6453 commented Feb 6, 2026

I think write Magic number each time is a lost of space

What can i put instead of ?? ?? ?? ??

@Geo6453
Copy link
Author

Geo6453 commented Feb 6, 2026

Nvm, know the size of a data is enough, i'll delete the ?? ?? ?? ?? and that will be good

@kinnay
Copy link
Owner

kinnay commented Feb 6, 2026

Yeah, the size is enough. I'll probably merge the PR this weekend and make some remaining formatting changes myself. Thanks again for the contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants