Skip to content

Comments

Animations#581

Merged
cesarl merged 43 commits intomasterfrom
animations
Oct 9, 2015
Merged

Animations#581
cesarl merged 43 commits intomasterfrom
animations

Conversation

@cesarl
Copy link
Member

@cesarl cesarl commented Oct 2, 2015

  • Animation (skeleton commun, une instance d'animation par mesh skinné)
  • Shared animation (skeleton, animation en commun). Genre pour faire une foule ou plein de monstres qui dansent.
  • Animations jobbed. C'est encore largement optimizable. Mais la quasi totalité du temps d'animation peut etre foutu dans des jobs. La je me suis pas pris la tete et j'attends dans le main thread que tout les jobs soit finis, donc c'est pas le plus top, mais on gagne deja enormement lorsqu'il y a bcp d'animations, mais suffit d'etaler les jobs entre 2 frames et le tour est joué et plus d'attentes sur le main thread. Bref.
  • Les objets skinnés castent des shadow
  • Mise en place dans l'editeur / serialization / load asynchrone etc
  • Le skin des mesh n'est actif que si l'occlusion culling l'est aussi, j'ai eu la flemme de le mettre sur toutes les pipelines.
  • Plein de scene de test et de benchmark
  • Faut faire attention : spotlight + skinned mesh ne font pas bon menage, ca m'empeched de faire de l'instancied rendering pour les shadow et donc on perd pleinnnnnnnnnnnnn de temps

@DorianPinaud c'est bon tu peux review 🐗

https://youtu.be/DDCvZz3lkw4

image

image
image

@cesarl
Copy link
Member Author

cesarl commented Oct 5, 2015

@DorianPinaud are you alive ? 😄

@DorianPinaud
Copy link
Contributor

yep, dsl en ce moment c'est la merde, la connection insternet creve chez moi et je suis obligé de resincrco le drive apres une mauvaise manip, je te tiens au courant quand je l'ai fait remarché chez moi. Peut etre demain soire

@cesarl
Copy link
Member Author

cesarl commented Oct 5, 2015

Hahaha ! Pas de soucis !

@cesarl
Copy link
Member Author

cesarl commented Oct 7, 2015

image

cesarl added 3 commits October 8, 2015 11:37
Disgusting implementation onto the existing crap but it looks like it's
working
Close #585
Load material is not thread safe but you know what ? Fuck it for the
moment hahaha
@cesarl
Copy link
Member Author

cesarl commented Oct 8, 2015

@DorianPinaud problem fixed

@DorianPinaud
Copy link
Contributor

nice, you can merge

@cesarl
Copy link
Member Author

cesarl commented Oct 9, 2015

@DorianPinaud thanks ! Yeah !

cesarl added a commit that referenced this pull request Oct 9, 2015
@cesarl cesarl merged commit 4025184 into master Oct 9, 2015
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

Successfully merging this pull request may close these issues.

2 participants