wakystuf / ESG-Mod

An Endless Space 2 balance and overhaul mod
31 stars 4 forks source link

Update for WeightTableDefinitions.xml #1492

Closed GitKopfkino closed 1 day ago

GitKopfkino commented 2 weeks ago

Using WeightTableDefinitions.xml from Development-1-5-1, I've done a mostly visual update. It's also my first time using Github to actually commit and push something, so I hope everything is ok and done correctly. I'm trying to push this to Development-1-5-1 and not the Main. The DIFF Software doesn't understand that some things just changed from one line to another, and that I deleted a lot of blank space, so it thinks I changed values, which I didn't. Feel free to check manually, but most of the changes I made were:

Edit: Had to edit this message because GitHub was hiding the entries if fully written.

Tat011 commented 2 weeks ago

Don't leave them unassigned, add them.

Tat011 commented 2 weeks ago

looking at this i noticed Curiosity_PlanetAnomaly49 is missing

GitKopfkino commented 2 weeks ago

Don't leave them unassigned, add them.

Will do. I believe they aren't Localized and I can't find their description anywhere, but I might not be looking in the right place.

GitKopfkino commented 2 weeks ago

looking at this i noticed Curiosity_PlanetAnomaly49 is missing

Went to check and it's true. It wasn't present originally. I'll add it as well.

Tat011 commented 2 weeks ago

did you intend to change the harmony spawn planets? id look to see if others are different but the github comparison tool breaks down in that section

Tat011 commented 2 weeks ago

which ones are missing localization?

GitKopfkino commented 2 weeks ago

which ones are missing localization?

The ones I left unassigned. I'm rechecking nonetheless, and these don't have an entry on ES2_Localization_Names.xml: "%StarNameRW2" "%ActualStarName88" "%ActualStarName241" "%ActualStarName334" "%ActualStarName385" "%ActualStarName386" "%ActualStarName387" "%ActualStarName388" "%ActualStarName419"

  1. I'm unsure about this one because it's marked as Vanilla, so it might not have localization? - "%ConstellationName8" (it's under Name="ConstellationNames", and below --Vanilla--)
  2. I'm also unsure about "PlanetAnomaly03" under "Name="AnomalyVisibleGeneration_Default", because it might be on purpose so it's invisible?
GitKopfkino commented 2 weeks ago

did you intend to change the harmony spawn planets? id look to see if others are different but the github comparison tool breaks down in that section

That's why I mentioned the problem with the DIFF. It doesn't understand I changed some lines and spaces. I'll attach a printscreen so you can see the before and after. The values remain the same. It's the DIFF that it comparing wrong lines.

01  Before 02  After

Tat011 commented 2 weeks ago

"%StarNameRW2" has been fixed, it wasnt broken tho, just skipped over "%ActualStarName88" and all the other ones are very likely me messing up the number and not stuff missing, it doesn't really matter that the numbering is scuffed, just leave it as is

captaincobbs commented 1 week ago

Is this good to review? Also, there are conflicts you need to resolve before merging. Sorry, don't check the github often, feel free to message me if you have something that needs to be reviewed.

Tat011 commented 1 week ago

What's conflicting? i tried to look at "resolve conflicts" but just got confused