kaptain-kavern / ADogSaid

Add basic animal surgeries and bionics to Rimworld - Originally made by Latta
https://ludeon.com/forums/index.php?topic=29310.0
6 stars 0 forks source link

Possible conflict reported #16

Closed nattiejim closed 8 years ago

nattiejim commented 8 years ago

Sarelth reported a possible conflict with another mod here: https://ludeon.com/forums/index.php?topic=21788.30

I do not know the name of the mod, so I have asked for further information.

inkyspoon commented 8 years ago

yeah I saw that on vegetable garden thread too. he thinks its us but dismar..the author of veg garden said it would be a mod that overwrites the plant parent def. and we don't even have that so I'm kinda sure its not us. but I will test it right now

inkyspoon commented 8 years ago

so like I said. just tested with mod order ccl cclvanilla epoe veggarden and us..no errors. if it was us we would have overwritten veggarden and there would be errors.

nattiejim commented 8 years ago

All right then - I'll close this issue.

kaptain-kavern commented 8 years ago

Sareth pm'ed me in Ludeons forum and he sent me his mod list. There was so many mods that I can't be 100% sure of the compatibilities but it turns out that he use Fluffe mod that add new animals. I'm pretty sure this clash with our changes in ThingsDefRaces

The way I see it we may need to enforce people to place ADogSaid in the highest reasonable position in mod order (Just after CCL+tweak) In order to try to make other animal mods try to catch and inherit the changes in AnimalThingBase class (because this is where we link the animal race to all the possible recipes).

The other most clean/compatible solution is to copy the class AnimalThingBase in our-modified-AnimalThingBase and redo all the files for ThingsDefRaces in order to reattach the animal to our modded AnimalThing class. That way the mod can be placed anywhere in mod order without throwing errors but we'll need to do patch for each new animals mods (or enforce modders to use our modded races)

kaptain-kavern commented 8 years ago

It's Dismar comment about parent def that make me think about that

inkyspoon commented 8 years ago

the thing is..I don't get how he gets the veggie error when we don't even have that? xD bc I did test with vegetable garden and there were no errors

kaptain-kavern commented 8 years ago

If you have seen the mod list ... I'm pretty sure he got several compatibility errors in the same time.

ADS(ADogSaid from now for me) was clashing with Fluffe's random animal mods

And VegGarden with Ghillie s mod

inkyspoon commented 8 years ago

but if theres several mods conflicting, I'm really not sure how and why we should help him...If he's got a ton of uncompatible mods we can't help anyway if they crash with each other? Or does the whole problem go away everytime he disables our mod? did he try to place it high above everything? like right under ccl, do you know?

kaptain-kavern commented 8 years ago

That's what i've advise him to do. And also to trim down is modlist too. For me this particular problem is solved/closed. I re-comment on the closed issue but i guess i should have open a new one. But the reasoning is the same and it's about parent def bases.

Dismar's comment that made me think about all of that.

To me ADS is a special case as it aims to apply modification to all animals (including modded one's if possible) so it's seem legit to me to enforce the high placement in mod order

inkyspoon commented 8 years ago

lol https://i.gyazo.com/3c5392463dde867dfb19550b84070174.png

inkyspoon commented 8 years ago

that mod already doesn't even work on its own! xD

kaptain-kavern commented 8 years ago

no look there is two folders in the archive. It's weirdly packaged :p

But it contain ThingsDefRaces files, so it will conflict with ADS (i already updated the readme)

inkyspoon commented 8 years ago

oh I see its two versions of the mod?

inkyspoon commented 8 years ago

soo..I get no errors, when our mod is above.

inkyspoon commented 8 years ago

how about we just say..If you have mods that add new animals etc place our mod as high as possible

kaptain-kavern commented 8 years ago

lol

the screenshot of the modlist : https://www.dropbox.com/s/gmeuphi6ehy2g5m/2016-07-23.png?dl=0

inkyspoon commented 8 years ago

you are fucking kidding me!!! xD

kaptain-kavern commented 8 years ago

lol i know what you mean ;) but srs i just explained that nobody can guarantee compatibility on that scale and advise him to reduce his modlist and also that the more mods he add the more i will find bug. Especially at first when you're beginning. This steam workshop mess ... it will bring tons and tons of problem (especially savegames broken by mods autoupdate). Anyway ...

how about we just say..If you have mods that add new animals etc place our mod as high as possible.

Simple and clear. Adopted :p

inkyspoon commented 8 years ago

so I just went ingame with our mod above the two fluffe mods..it gives no errors and I can still operate on the vanilla animals but ofc not on the added animals I don't think its that bad actually

kaptain-kavern commented 8 years ago

We both try to edit the same comment at the same time i see :p nearly telepathic there

inkyspoon commented 8 years ago

yeah and the steam thing..seems nattie had to upload a complete new version of the mod now bc steam wouldn't let him just update the old one. I'm guessing its because the modfolder he was trying to update from is named differently than his initial vers 1.0.2 update. I hope people won't get angry when the mod is suddenly gone from their game and gives errors lol xD

kaptain-kavern commented 8 years ago

I don't think its that bad actually

Neither do i. We clearly can't guarantee this

nattiejim commented 8 years ago

Just to clarify, I didn't delete the mod entirely, rather, I just set it to be hidden from the workshop. Hopefully, people won't have to deal with a broken game as if it had been just deleted entirely, but we'll see.

kaptain-kavern commented 8 years ago

I'm guessing its because the modfolder he was trying to update from is named differently than his initial vers 1.0.2 update

This could be my fault as i haven't add the .zip in the 1.0.2 release (so the folder he download from Github source button was named Adogsaid-master.zip) :-(

inkyspoon commented 8 years ago

yeah I saw ^^ it was just hidden. do you know what the problem was nattie? or what did steam say when you tried to update?

nattiejim commented 8 years ago

It didn't give me any errors when I tried to update - it just gave me the "Upload to Workshop" button on the Mods screen, when what I needed to use was the "Update" button.

inkyspoon commented 8 years ago

well the folder i got from the zip was just ADogSaid

nattiejim commented 8 years ago

My folder was ADogSaid as well.

inkyspoon commented 8 years ago

Do you think its bc of the old folder that was called LT-ADogSaid--1.0.2? or something

inkyspoon commented 8 years ago

maybe steam thought its a different mod bc of it

nattiejim commented 8 years ago

That's most likely what it was.

inkyspoon commented 8 years ago

so..we just stick with how it is now? and hope people using the old mod see the new update?

nattiejim commented 8 years ago

I think that's what we can do for now.

inkyspoon commented 8 years ago

yeah.. maybe write a comment or something explaining. maybe its for the best so now people can see the issue with old saves before they change over and not be surprised by it :)

kaptain-kavern commented 8 years ago

About steam workshop : It was a "thing" in the RW mod community since the last few weeks (see this post

kaptain-kavern commented 8 years ago

I'm doing PR there ^^