-
Notifications
You must be signed in to change notification settings - Fork 74
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
CI: Calculate metrics based on previous results. #122
Comments
Hi. Do you wanna use gdrive for this? So in this case we should be able also to compare expectations not just with object set in config but also with some file with expectations. |
hey @denar90 I think gdrive is one of the option, but ideally you want to have several options, right? For example to store results locally. That's what I'm doing right now in my patched version of |
That's right. As a part of your calculation you will take the budget from the config file and the prev. results from the store(file/gdrive/s3/etc.). |
Will be nice to take a look 👀 . Can you create a pr or smth? I have couple questions. |
Sure, I'll open PR.
Very good question. Each time we perform a test a new file is getting created with a name that contains a timestamp (i.e.
I'm not sure if it's valid for Travis. We can use
It is definitely worth having it. |
Good points. I'm 👍 for this way, we also can have some tuning or something but for now, we should stick to this one. |
I'm ok with trying something like this, not exactly sure how to implement it though. I see two options:
These are just my 2 cents. I understand the desired for the functionality just not sure of whether pwmetrics as a project is the right scope for it. |
❤️ it. I would then also move google-sheets out of the |
If we agree on a plugin-based system, yes Google-sheets should move out. Perhaps as the first option (POC) for pwmetrics-storage |
If we wanna divide pwmetrics should we create org for this? or should we create subpackages? thoughts? |
I do not have experience in managing orgs or monorepos. I'll defer to your best judgment, however does this make sense as a plugin for pwmetrics? @julianusti correct me if I'm wrong but don't you want something more general? As in, depending on the CI system you have you create a script that runs pwmetrics, processes the output and passes that to the CI system that could, for example generate a regression performance report? In this scenario, it would be the responsibility of the CI system to store the previous reports and use that to evaluate metric variations over time. |
As a part of CI it would be nice to have a feature that allows you to calculate metrics based on your previous results, so we can alert in case if perf metrics are going down significantly.
The text was updated successfully, but these errors were encountered: