Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get the error with gen-json command. #6

Closed
chetuna opened this issue Mar 30, 2021 · 6 comments
Closed

Get the error with gen-json command. #6

chetuna opened this issue Mar 30, 2021 · 6 comments

Comments

@chetuna
Copy link

chetuna commented Mar 30, 2021

I got an error with gen-json. (Title ver.1.0.0)

Error: Generating large monsters

Caused by:
    0: enemy/em001/00/user_data/em001_00_datatune.user
    1: Left over data
@wwylele
Copy link
Owner

wwylele commented Mar 30, 2021

Thanks for the report. I already have some fix locally, but I want to give myself a spoiler-free week, I am not testing and pushing it until this weekend. If you are interested in fixing it yourself, You can generate new TDB and look up the DataTune structure, and update the code accordingly.

@wwylele
Copy link
Owner

wwylele commented Apr 4, 2021

Pushed the updated code

@wwylele wwylele closed this as completed Apr 4, 2021
@chetuna
Copy link
Author

chetuna commented Apr 5, 2021

Error: Generating large monsters

Caused by:
    0: enemy/em002/00/collision/em002_00_colliders.rcol
    1: Unsupported type 732706C9

I continue to get errors and no results are being output.
Is this a known issue? Should I not report it for a while?

@wwylele wwylele reopened this Apr 5, 2021
@wwylele
Copy link
Owner

wwylele commented Apr 5, 2021

That's interesting. I didn't encounter this problem when generating with the 1.1.1 patch (and as you can see I updated the website with it). I retried it with the vanilla game data and got this error. Maybe the original version used an old format for this file. Since 1.1.1 is a day 1 patch and I wanted to regard it as the effective 1.0, I am not super interested in fixing this. I'll leave this open if anyone is interested in looking into it.

Tip: to generate with patch, dump the patch RomFS as well, and supply all .pak file to the --pak parameter, ordered from the oldest to the newest.

@chetuna
Copy link
Author

chetuna commented Apr 5, 2021

Worked well with 1.1.1.
I overlooked that there is a cool command to merge patches.

@wwylele
Copy link
Owner

wwylele commented Jan 17, 2022

Closing this as I am not going to maintain anything before the first pc version (3.6.1.0)

@wwylele wwylele closed this as completed Jan 17, 2022
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

No branches or pull requests

2 participants