rscustom / rocksmith-custom-song-toolkit

Custom song toolkit for Rocksmith and Rocksmith 2014
http://www.rscustom.net/
343 stars 80 forks source link

Crash in Rocksmith 2012 just before tuning the guitar with package created by Toolkit v2.3 #107

Closed Chlipouni closed 10 years ago

Chlipouni commented 10 years ago

I have this issue with all of my customs.

Toolkit v2.3 :

Toolkit v1.4 :

Thanks

L0FKA commented 10 years ago

it would me great if you diff 1.4 dat and 2.3 dat

I've resolved rough issues with exceptions to allow produce 2012 dlc but cant validate its worked, thanks :)

Chlipouni commented 10 years ago

Here are the links for downloading the two .dat files :

http://www.mediafire.com/download/09a04etfpb1xqlp/Accept%20-%20Balls%20To%20The%20Wall%20-%20TK%20v1.4.dat

http://www.mediafire.com/download/fo4c0n5fdchq328/Accept%20-%20Balls%20To%20The%20Wall%20-%20TK%20v2.3_p.dat

Thanks

2014/1/10 L0FKA notifications@github.com

it would me great if you diff 1.4 dat and 2.3 dat

I've resolved rough issues with exceptions to allow produce 2012 dlc but cant validate its worked, thanks :)

— Reply to this email directly or view it on GitHubhttps://github.com/rscustom/rocksmith-custom-song-toolkit/issues/107#issuecomment-32019651 .

fabianosan commented 10 years ago

Crash before or after tuning step? Just confirm, because if crash after probably is a problem with .bnk if crash before maybe is a problem with .SNG.

Can you post here the title of your toolkit with exactly version?

Chlipouni commented 10 years ago

Crash is before the tuning step.

The Toolkit v2.3 release is : v2.3.0.0-24da9151 beta The Toolkit v1.4 release is : v1.4.0.0 beta

Chlipouni commented 10 years ago

Sorry for the "closed" status, it was a mistake ...

L0FKA commented 10 years ago

In: song.xblock

become 9

song.manifest "RelativeDifficulty": 7, become 9 different precision on floating numbers in manifest "IsSolo": false become true if its solo(just working bug fix) in attributes added "Prebends", hope RS ignore that like "Vibrato" "Prebends": false, become true "RelativeDifficulty": 1, become 0

sng are different(in new tkit missing some bytes, but its not encrypted, thats good

fabianosan commented 10 years ago

Forget about manifest, if have a problem with manifest the song don't appear in song list or just don't show a specific info in song list, the only thing to see in manifest is tuning info.

All the differences in manifest you related are from new fixes of the new toolkit (RelativeDifficulty) is not manually defined anymore, is obtained from song xml, IsSolo now is setup in toolkit, the 1.4 doesn't setup solo sections, is the same as Prebends, because all techniques are now automatically obtained from song xml, I did this fixes, I believe that is not the problem.

Here is the path of the problem:

  1. Check tuning data in manifest (I checked and is the same, so is not the problem);
  2. Soundbank file (.bnk);
  3. Sound game file (.sng)

@Maveth Maybe you can help us with .SNG file? @snakez Maybe you can help us with .BNK file?

All about RS1 songs.

Me and @L0FKA verifying some other things.

We need to put RS1 customs working again on lastest toolkit.

fabianosan commented 10 years ago

@Chlipouni Can you download old releases like 2.2, 2.1 to see if it work?

https://github.com/rscustom/rocksmith-custom-song-toolkit/releases

Chlipouni commented 10 years ago

Yes, I can do this ...

Chlipouni commented 10 years ago

I don't have the 2.1 and 2.2 releases on my computer. Where can I download these old builds ?

L0FKA commented 10 years ago

2.2 -> https://github.com/rscustom/rocksmith-custom-song-toolkit/releases/download/2.2.0.0/RocksmithCustomSongToolkit-v2.2.0.0.zip

2.2.2.1 -> https://github.com/rscustom/rocksmith-custom-song-toolkit/releases/download/2.2.2.1/RocksmithCustomSongToolkit-2.2.2.1-3dbc716ff7cf.zip

L0FKA commented 10 years ago

But i'm sure its about SNG, we changed some field types for RS1 too, so it could be there.

Chlipouni commented 10 years ago

Toolkit v2.2.0.0 :

Toolkit v2.2.2.1 :

L0FKA commented 10 years ago

so 2.2.0.0 got no problems?

Chlipouni commented 10 years ago

I can't generate the "dat" file with the release v2.2.0.0 for other reasons, so I can't test the package ...

fabianosan commented 10 years ago

Just rename to .dat, it's only wrong extension.

Chlipouni commented 10 years ago

Crash before tuning with the package generated by Toolkit v2.2.0.0.

Chlipouni commented 10 years ago

Yesterday, I saw that "ArdGuitar" has released a new CDLC for RS2012 ("Green Day - Nice Guys Finish Last").

I downloaded it and try this :

Hope that helps

fabianosan commented 10 years ago

Yep, you confirmed the problem. Thanks, we need to track the change that broke the rs1 sng now.

Maveth commented 10 years ago

ill take a look when i get home tonight, if you havent already fixed it. i was just going to suggest testing it like that.

Maveth commented 10 years ago

do have the dlc, xml stuff handy? for the .dat that fails with new toolkit, but works with 1.4?

Maveth commented 10 years ago

i think i tracked it down using the xml2sng creator tab, i get a diff in the sngs, in the following commit c61c32be7f86908689b8c7a34f6bb40ea7aba4ce

there are a number of changes to the data types, its looks like it was used to base the initial 2014sng format off of, but wasnt reset back to normal. some of it might be ok, but not all of it... looking at the raw sng in hexeditor, its definitely the data types, byte vrs int.. stuff like that.. because all the data is there, its just not right sizes.

ill see what i can do. i dunno that changing it back to original, might cause some other issues, but ill see.

L0FKA commented 10 years ago

Some of sng values are changed, but it required by RS2014, so if you change one back for RS2012 - override for RS2014. or find what is wrong...

Maveth commented 10 years ago

i thought there are 2 processes, 1 for 2014, using song2014,cs and one for 2012 using song.cs it looks like song was copied to song2014, but it was changed before being copied? i dunno yet. but thats where it breaks.

L0FKA commented 10 years ago

its used some fields from RS1, so it was changed, you can compare XML\song.cs from 1.4 to last beta, and find what is wrong..

also my hsl not equals rs1 original sng what can be wrong?

Maveth commented 10 years ago

i dunno why its using the XML\song.cs for 2014 when there is a XML\song2014.cs, that doesnt make sense to me. if i try and change it back to what it used to be then i get compile errors for 2014filewriter? something isnt right there.

fabianosan commented 10 years ago

Make sense because is a object oriented language that uses inheritance. Basically all fields in song.cs are used in song2014 with only new fields.

If changes it's necessary, just create new object with new values and set song2014 variable to use this object and not inheritance.

If need create all new objects for 2014 we need to create a new RocksmithToolkitLib for 2014, it that's not make sense to me.

Maveth commented 10 years ago

i guess so, my old ways die hard, i just about got it though. i might duplicate a few more than HAVE to be. but that shouldnt matter

Maveth commented 10 years ago

i can get rs2012 working again,but getting errors for 2014 {"The best overloaded method match for 'RocksmithToolkitLib.DLCPackage.Manifest.ManifestFunctions.GetNoteCount(float, float, System.Collections.Generic.ICollection)' has some invalid arguments"}

Maveth commented 10 years ago

finally got sng, to match 1:1 for rs2012, to toolkit 1.4, also matched to 1:1 for rs2014 vs 2.3 toolkit. but now every package i make crashes the game??? i reverted my changes on my system and it still does it. you guys change something else in the meantime?

Maveth commented 10 years ago

yes, i revert to a week ago, and apply my changes and its ok. so something else done has broken the code. might be if showlights needs the original xml loaded again? dunno i dunno what it is. somewhere within the last week it starts crashing. im too tired to figure out what

Chlipouni commented 10 years ago

Thanks a lot for the quick resolution