New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds support for controlling the generic stats graph in a easy to use manner. #15801
base: master
Are you sure you want to change the base?
Conversation
Created this as we had a need to know the daily max average for some log stuff. |
Hi @VVelox |
@PipoCanaja Also added some docs for this on this and |
I'm not sure I understand what problem this PR is trying to solve. |
@murrant See the stats option line in the screen shot there. That adds that. Basically the purpose of this is to allow graph data to be dug into at way more depth than before. The original stats graph proved to be a handy start, put we started finding it a bit lacking with available information. This allows what is displayed via graphs supporting this to be controlled. This was originally born out of the need to get daily max stats as well. While that line is not that useful the bottom information line for like average, min, and max can be insanely useful for planning purposes. |
This is something I want to begin expanding to other stats graph enabled apps in time. :) |
Moving to draft to begin re-factoring to object based graphs. Thanks for the idea @murrant . <3 |
@VVelox start simple, don't try to do too much at once :) |
Adds support for controlling the generic stats graph in a easy to use manner.
Adds in control for hourly, daily, and weekly averages, mins, and maxs. Also allows control for if a user wants to display the 25th/50th/75th or either of the two user definable percentiles.
generic_stats_options.inc.php
is the new graph file that takes the options for that from vars in a generic fashion. Also sanity checks it and if it does not fine a sane value it just uses the default for that value.generic_stats_options_init.inc.php
inits using it for a apps graph page.generic_stats_options_line.inc.php
generates the options for use us in the options area for a app page.Defaults...
The defaults for these are configurable via the global options menu.
https://cdn.discordapp.com/attachments/463435361100627988/1203793773604376576/Untitled.png
Please note
Testers
If you would like to test this pull request then please run:
./scripts/github-apply <pr_id>
, i.e./scripts/github-apply 5926
After you are done testing, you can remove the changes with
./scripts/github-remove
. If there are schema changes, you can ask on discord how to revert.