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

Duplicate ship_tank_karavan.png #11

Open
kervala opened this issue Jan 7, 2016 · 5 comments
Open

Duplicate ship_tank_karavan.png #11

kervala opened this issue Jan 7, 2016 · 5 comments

Comments

@kervala
Copy link
Contributor

kervala commented Jan 7, 2016

ship_tank_karavan.png is present in 2 directories :

  • stuff/caravan/agents/_textures/accessories
  • stuff/caravan/agents/_textures/actors

There are slightly differences between them and I think the good one is in "accessories", because this texture has nothing to do with "actors" :p

We should keep only one of them and remove the other one.

@kaetemi
Copy link
Member

kaetemi commented Jan 7, 2016

The ship is an actor.

Accessories is for hand-held equipment (like swords) and non-moving dynamically placed objects.

  • Actors: Movable entity (including armors) (anything with a skeleton basically)
  • Accessories: Hand-held equipment (swords) and non-moving dynamically placed objects (anything without a skeleton)
  • Decors: Static models (lightmapped)
  • Construction: Decors XRef'd by other files (landscape), not directly exported

@kaetemi
Copy link
Member

kaetemi commented Jan 7, 2016

Ship models are under actor as well, but textures all seem misplaced. :)

Please do double check with live data which of the textures is the latest version.

@kervala
Copy link
Contributor Author

kervala commented Jan 7, 2016

Yep, that's why I didn't remove it directly and prefer to create an issue :) If files were identical, there wouldn't have any problem :p

@kaetemi
Copy link
Member

kaetemi commented Jan 7, 2016

The trame.png under caravan actors also needs to be deleted.

@kaetemi
Copy link
Member

kaetemi commented Jan 7, 2016

The caravan ships are a special case, they are under actors because they do have a skeleton for some animations, and can move (also with specific movement animations and effects.)

@kaetemi kaetemi transferred this issue from another repository Mar 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants