Skip to content

E07: Remove the v1 Analyzer logger - #250

Merged
jacobdenobel merged 4 commits into
masterfrom
wp/E07-remove-analyzer-v1
Nov 9, 2026
Merged

jacobdenobel merged 4 commits into
masterfrom
wp/E07-remove-analyzer-v1

Conversation

@jacobdenobel

Copy link
Copy Markdown
Contributor

Requested by the repository owner: remove the v1 Analyzer logger, make v2 the only Analyzer, and remove the v1 bindings and docs. This is a breaking change for code that uses ioh.logger.old, Experiment(old_logger=...) or names analyzer::v1/v2 in C++. The CHANGELOG (### Removed) and the upgrade guide list each removed item and what to use instead. The Analyzer output is unchanged: the byte-for-byte golden tests pass.

Work package E07: Remove the v1 Analyzer logger

Branch: wp/E07-remove-analyzer-v1
Status: ready for review

Done

  • step 1/3: folded analyzer::v1 and v2 into one analyzer::Analyzer, removed the .info writer and its C++ tests; 118 C++ tests pass (Debug).

  • step 2/3: removed ioh.logger.old, Experiment(old_logger=), stubs, tests, tutorial, example README.

  • step 3/3: CHANGELOG ### Removed and upgrade guide sections. Python module built in .venv; all Python tests pass in a
    copy outside the tree (test_golden.py and test_logger.py included). The one error test_array_layout.py:: tester_inc_one_transform is pytest collecting a helper function, unrelated.

Next step

None. Controller: push once E05 is merged and open the draft PR.

Decisions

  • No analyzer::v2::Analyzer alias: the class is ioh::logger::analyzer::Analyzer, ioh::logger::Analyzer is an alias to it.
  • Dropped the use_old_data_format constructor argument with v1.
  • Kept E05's full_precision and "precision" JSON key.

Open questions / Blocked

  • none

Work package from the IOHanalyzer rebuild plan (plan/ in IOHprofiler/ioh-web). Review and merge are done by a person.

🤖 Generated with Claude Code

jacobdenobel and others added 4 commits November 6, 2026 12:33
…writer

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…cs, CHANGELOG, upgrade guide

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jacobdenobel
jacobdenobel marked this pull request as ready for review November 9, 2026 10:33
@jacobdenobel
jacobdenobel merged commit 1d713b8 into master Nov 9, 2026
37 checks passed
@jacobdenobel
jacobdenobel deleted the wp/E07-remove-analyzer-v1 branch November 9, 2026 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant