Skip to content
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

CMEnergies as float_range involves passing filters around a lot #199

Open
alisonrclarke opened this issue Apr 24, 2020 · 0 comments
Open
Labels

Comments

@alisonrclarke
Copy link
Contributor

Code is currently on my fork, but the initial fixes for #143 involve passing the currently selected filters around a lot, e.g. to add the aggregations in es_config.py, and when parsing the aggregations in aggregations.py.

Part of the reason for this is that support for ranges in elasticsearch is not (yet) as comprehensive as support for float types: see elastic/elasticsearch#34644

One potential option would be to see if we can use painless scripting in elasticsearch to so some of the work for us.

We need to do the migration soon so I'm not going to spend any more time on the issue now, but am logging it as something to look into later on if we have time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: To do
Development

No branches or pull requests

2 participants