Skip to content

Conversation

@spirali
Copy link
Collaborator

@spirali spirali commented Sep 26, 2025

This PR adds hq journal report <journal> <output-filename> that exports journal statistics as follows:

image
  • This implementation now takes the journal file as the input. In the case of need it is possible to create a report from a live server or JSON export.
  • This version is not rebased on the newest main, because Waiting mechanism refactor + user notification mechanism. #1024 adds new events and v0.23.0-v0.24.0 journal would be not loadable. I am leaving it as it is for now so potential testers may use their current journals. (btw: It it non conflicting with the current main)
  • I have just implemented basic charts and basic time filtering. There are a lots of option what to implement but I would let it be user use-case driven.
  • Known bug: It does not work when resource variants are used (needs to update journal to implement this; it will be in the next PR).

@Kobzol
Copy link
Member

Kobzol commented Sep 26, 2025

This version is not rebased on the newest main

CI will always rebase on main, so you won't be able to share the CI artifacts with the main that this PR is based on.

@spirali
Copy link
Collaborator Author

spirali commented Sep 27, 2025

This version is not rebased on the newest main

CI will always rebase on main, so you won't be able to share the CI artifacts with the main that this PR is based on.

Ah, ok. So in that case, an enthusiastic tester may build own HQ version:)

I will leave it here for few days, meanwhile I will independently fix the problem with resource variants (that also breaks the journal compatibility) and then rebase this.

@spirali
Copy link
Collaborator Author

spirali commented Sep 30, 2025

Rebased on main + fixed variant bugs.

@spirali spirali merged commit f6cf8c2 into main Oct 1, 2025
9 checks passed
@spirali spirali deleted the journal-charts2 branch October 1, 2025 20:49
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.

3 participants