Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support aggregators with multiple parameters (i.e. percentile) #30

Closed
eatlakson opened this issue Feb 5, 2025 · 0 comments
Closed

Support aggregators with multiple parameters (i.e. percentile) #30

eatlakson opened this issue Feb 5, 2025 · 0 comments

Comments

@eatlakson
Copy link
Contributor

Describe the bug
Extra parameters are not detected for aggregators - such as percentile.

To Reproduce
Add a literal column with a "percentile(columns_name, p_value)" value. Current regex does not detect or extrect p_value, and it is not added to the summarize section.

Expected behavior
percentile literal column is added to the summarize section with the p_value value defined.

Screenshots
n/a

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Will follow up with PR.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants