-
Notifications
You must be signed in to change notification settings - Fork 11
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
volume analysis #201
volume analysis #201
Conversation
b0ecc7f
to
d0fa66b
Compare
d0fa66b
to
2fff74d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also test it with adex-explorer and do a PR there to support this
routes/analytics.js
Outdated
value: { | ||
$sum: { | ||
$map: { | ||
input: { $objectToArray: `$events.IMPRESSION.eventPayouts` }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should work on counts too; it should be a parameter like event-aggregates
the event type (IMPRESSION) should be a parameter too
@samparsky I will approve/merge that when the relevant PR is made in adex-explorer |
bugs/issues:
don't touch for now, I'm rewriting |
another one
|
Fix #200