Mutable Animation Clips about bevy

[NthTensor]

It would be lovely if you would write a PR for this. Otherwise it may be some time before someone starts working on it.

[nzhao95]

Nice ! I'll get on it thanks a ton

[NthTensor]

Each AnimationGraphNode contains an Handle<AnimationClip>. You can get a mutable interference to the underlying clip from Assets<AnimationClip>. So it looks like we are just missing methods

fn curves_mut(&mut self) -> &mut HashMap<AnimationTargetId, Vec<VariableCurve>>
fn curves_for_target_mut(&mut self, target_id: AnimationTargetId) -> Option<&mut Vec<VariableCurve>>

on AnimationClip. Seem correct?

[nzhao95]

Hello, thanks for getting back so quickly. Exactly those mutable getters would be really nice to have. For my use case it's really crucial. I don't know if there are good reasons not to have them but if they aren't it would be super cool to have them !

[NthTensor]

I think we just never bothered to add them. Looks like a good addition to me. I've tagged this as a good first issue.

[nzhao95]

Cool ! Should I write the code and ask for a PR or is it going to be taken care of ?