Forward AD tests and cleanup for vector interface

[kshyatt]

No description provided.

[github-actions]

Your PR no longer requires formatting changes. Thank you for your contribution!

[lkdvos]

Cool! Do you have any idea if this would make sense to implement at the VectorInterface level? Alternatively, to not necessarily mark everything as @is_primitive there but to already provide the implementation?

[kshyatt]

We could do it there if it's generic enough? I don't have a strong opinion tbh

[lkdvos]

It looks like all rules are implemented solely in terms of Mooncake or VectorInterface functions, so that might just work?

[kshyatt]

Nice, I can move these over tomorrow then

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines	Coverage Δ
ext/TensorKitMooncakeExt/tangent.jl	65.06% <ø> (ø)
ext/TensorKitMooncakeExt/utility.jl	100.00% <ø> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[lkdvos]

I think you still need the @is_primitive calls, since the VI rules are only defined for AbstractArray

[kshyatt]

Addressed by another PR over there 😉

[lkdvos]

Alright, I think I've been annoying for long enough about the VectorInterface compat, so I think the main thing here is to just bump the VI version and drop the versions that don't have these rules, and go from here. Otherwise this PR looks ready for merging to me, thanks @kshyatt !

[kshyatt]

OK the Enzyme fail is unrelated, something that intermittently didn't show up previously, I'll fix separately in the Enzyme VI PR