[nkoukpaizan]

Description

This fixes a latent bugs in the Enzyme sparse_store dispatch and drop the unused primal buffer.

Checklist

• All tests pass.
• Code compiles cleanly with flags -Wall -Wpedantic -Wconversion -Wextra.
• [N/A] The new code follows GridKit™ style guidelines.
• [N/A] There are unit tests for the new code.
• [N/A] The new code is documented.
• The feature branch is rebased with respect to the target branch.
• [N/A] The CHANGELOG.md has been updated to reflect the changes. If this is a minor PR that is part of a larger fix already included in the file, state so.

Further comments

[read-the-docs-community]

Documentation build overview

📚 GridKit | 🛠️ Build #35332935 | 📁 Comparing 4dae43a against latest (f3d5dd6)

  🔍 Preview build  

4 files changed

± api/reference/file_GridKit_AutomaticDifferentiation_DependencyTracking_VariableOperators.hpp.html
± api/reference/file_GridKit_Testing_Testing.hpp.html
± api/reference/file_GridKit_Utilities_Enum.hpp.html
± api/reference/namespace_GridKit__DependencyTracking.html

[lukelowry]

Great! Thank you for making this more robust. I can build this on my system and all tests pass

I am not sure if this is in scope, but I found one type issue that fails if I instantiate with float

  GridKit::Testing::TestingResults              result;
  GridKit::Testing::EnzymeTests<double, size_t> test;
  GridKit::Testing::EnzymeTests<float, size_t>  test_float;

  result += test.scalarFunction();
  result += test.vectorFunctionSparse();
  result += test_float.vectorFunctionSparse();

I think everything is double, so unsure if this is an actual issue

[nkoukpaizan]

  GridKit::Testing::TestingResults              result;
  GridKit::Testing::EnzymeTests<float, size_t>  test_float;

  result += test.scalarFunction();
  result += test_float.vectorFunctionSparse();

I think everything is double, so unsure if this is an actual issue

Right, that doesn't work now because we have yet to add template instantiations for the models on float.