Small changes to the constraints module

[tsmbland]

Better checks for the constraints, mandating that extra arguments are kwargs, removing some redundant calls to cast. Nothing major.

[Copilot]

Pull Request Overview

This pull request updates the constraints module to enforce that all constraint results are matrices with appropriately labelled dimensions, rather than scalars. Key changes include:

• Updating test expectations in tests/test_constraints.py for constraint dimension labels.
• Adding runtime checks for constraint data variables and labeled dimensions in src/muse/constraints.py.
• Modifying functions (e.g., max_capacity_expansion, demand, and max_production) to ensure matrix outputs.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
tests/test_constraints.py	Tests updated to verify that produced matrices include the expected labels for dimensions.
src/muse/constraints.py	Constraint functions now enforce the use of matrices with labelled dimensions via runtime checks and updated operations.

Comments suppressed due to low confidence (1)

src/muse/constraints.py:393

• [nitpick] The variable name 'capa' used shortly after is somewhat ambiguous. Consider renaming it to something more descriptive like 'capacity_matrix' to improve code clarity.

if b.region.dims == ():

[Copilot]

Using assert to check for non-empty dimensions can lead to unexpected behavior if Python is run with optimizations; consider replacing the assertion with an explicit runtime error (e.g., raising a RuntimeError with a descriptive message).