Reuse CSR pattern in PhasorDynamics system Jacobian

[nkoukpaizan]

Description

This adapts the approach to reuse system Jacobian CSR pattern, introduced in #339 for PowerElectronics, to PhasorDynamics.

Proposed changes

• Updated the PhasorDynamics system model to reuse the CSR Jacobian pattern.
• Updated IDA interface to remove the temporary CsrJac function.

Checklist

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

Further comments

This only touches the system-level Jacobians. I'll follow up with a PR that changes the component-level Jacobians.

[pelesh]

The code executes correctly. On my machine I get Hawaii case to run in 6.8s compared to 8.4s before, so we are going in the right direction.

I made a couple of nitpicking comments, but I think this is ready to be merged.

[pelesh]

Consider using iostream here:

Suggested change

	printf("%lu : %e \n", i, fres[i]);
	}
	std::cout << i << " : " << << std::scientifi << fres[i] << "\n";
	}

[pelesh]

Do we need to include matrix header here? Could we use forward declarations instead?

Suggested change

	#include <GridKit/LinearAlgebra/SparseMatrix/COO_Matrix.hpp>
	#include <GridKit/LinearAlgebra/SparseMatrix/CooMatrix.hpp>
	#include <GridKit/LinearAlgebra/SparseMatrix/CsrMatrix.hpp>
	#include <GridKit/Model/VariableMonitor.hpp>
	#include <GridKit/ScalarTraits.hpp>
	#include <GridKit/LinearAlgebra/SparseMatrix/COO_Matrix.hpp>
	#include <GridKit/Model/VariableMonitor.hpp>
	#include <GridKit/ScalarTraits.hpp>
	namespace GridKit
	{
	namespace LinearAlgebra
	{
	template <typename RealT, typename IdxT>
	class CsrMatrix<RealT, IdxT>;
	template <typename RealT, typename IdxT>
	class CooMatrix<RealT, IdxT>;
	}
	}

[pelesh]

CC @lukelowry @abirchfield

[kakeueda]

Enzyme isn't enabled on my machine yet so I haven't evaluated speedup, but it looks good to me.