Add nb_profilable metric counting profilable visits. #19745

Description:

Refs #16363

I'm finishing up #16773, and thought it would be better to review/merge it piecemeal. This PR adds nb_profilable as a metric to VisitsSummary.get. It's just the sum of all profilable visits, and will be used in further PRs.

Review

• [ ] Functional review done
• [ ] Potential edge cases thought about (behavior of the code with strange input, with strange internal state or possible interactions with other Matomo subsystems)
• [ ] Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
• [ ] Security review done
• [ ] Wording review done
• [ ] Code review done
• [ ] Tests were added if useful/possible
• [ ] Reviewed for breaking changes
• [ ] Developer changelog updated if needed
• [ ] Documentation added if needed
• [ ] Existing documentation updated if needed

This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers

@sgiehl modified the code and asked questions for other items

@diosmosis left another comment. Besides that the target still needs to be changed to 5.x-dev and a couple of tests are failing...

@sgiehl can you confirm with @mattab the target should be 5.x-dev? I was told it would be better to put this in 4.x. Or is 4.12 the last 4.x version?

@justinvelluppillai told me 4.12. was the last planned release before Matomo 5. There will be a patch release 4.12.1 though, but the changes in this PR might be a bit too big for that.

Ok, I'll switch to 5.x-dev

i'm thinking maybe we will have a 4.13 release i'm thinking? So maybe you could wait until switching to 5.x @diosmosis

@mattab please clarify that with @justinvelluppillai
We have already merged a couple of changes to 5.x-dev, as there was no plan for 4.13
Also we will need to wait with merging this one until 4.12.1 has been released if it should target 4.x-dev. On 5.x-dev we could directly merge

@mattab @justinvelluppillai any update on ^?

This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers

@mattab I am happy either way, you can maybe decide here? It is likely there will be a 4.13 to release a few things that need to be done for this quarter

This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers

This PR was last updated more than one month ago, maybe it's time to close it. Please check if there is anything we still can do or close this PR. ping @matomo-org/core-reviewers

@justinvelluppillai Has there been a decision whether to merge this into the next 4.x patch release or keep it for 5.x?

This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers