Skip to content

Test coverage gaps: AMICA.save/load and plot_components #15

Description

@neuromechanist

Surfaced during Phase 1 review (epic #9, PR #14).

  • AMICA.save/AMICA.load (pyAMICA/amica.py:244-297) do a real torch.save/load state_dict roundtrip but have zero test coverage. The old AMICATorch.save/load test was deleted (those methods no longer exist); coverage moved to the wrapper and was never re-added.
  • amica_viz.plot_components (pyAMICA/amica_viz.py:84) has a current signature (results_dir, data=None, ...) -> None and is fully untested since the stale test_visualization was deleted.

Add real (non-mock) tests: save/load a fitted model to tmp_path and assert roundtrip equality of W/A; render plot_components from a real saved results directory and assert files are written.

Metadata

Metadata

Assignees

No one assigned

    Labels

    featureNew feature or enhancement

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions