Code

Refactor and extend History class#552

Code

Open

timmens wants to merge 24 commits intomainoptimagic-dev/optimagic:mainfrom remote-history-arraysoptimagic-dev/optimagic:remote-history-arraysCopy head branch name to clipboard

+1,100-233

Commits

Commits on Nov 30, 2024

• Add direction argument to History class

  timmens

  committed
  c829c75

• Create 3.13 compatible envs using pre-commit hooks

  timmens

  committed
  125d313

• Add attributes of HistoryArrays to History

  timmens

  committed
  d18b63c

• Use History in convergence report creation

  timmens

  committed
  7ad2edd

• Use History and not HistoryArrays in history_plots.py

  timmens

  committed
  bf6cbe6

• Remove history_tools.py

  timmens

  committed
  e6b6066

• Update error message for direction init argument in History class

  timmens

  committed
  d8fefc1

• Add additional tests for history.

  timmens

  committed
  6a991af

• Refactor History class

  timmens

  committed
  b4a186d

Commits on Dec 1, 2024

• Fix tests for refactored History class

  timmens

  committed
  0b7cfe0

• Add stop_time and init arg validation to History

  timmens

  committed
  1ff4765

• Implement cost model and use in History

  timmens

  committed
  7bbc079

• Align codebase with new History class

  timmens

  committed
  f15e173

• Add unit tests

  timmens

  committed
  eac0bac

• Add tests for fun and params

  timmens

  committed
  dff7c4c

• Finish test suite for History class

  timmens

  committed
  df8bb95

• Fix mypy errors

  timmens

  committed
  e72608b

• Try to fix some warnings

  timmens

  committed
  fe27a3b

Commits on Dec 3, 2024

• Validate cost_model argument in _get_time function

  timmens

  committed
  c86421c

• Handle batch case

  timmens

  committed
  da279ac

• Use plural

  timmens

  committed
  e089347

Commits on Dec 5, 2024

• Add more test cases for invalid user funcs; check that batch time aggregator returns scalars

  timmens

  committed
  bd50a5d

• Some minor changes after first round of own review

  timmens

  committed
  0c5ead3

• Add explicit assumption to _apply_to_batch docstring

  timmens

  committed
  33c5a18