-
Notifications
You must be signed in to change notification settings - Fork 9
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
Limited number of arguments for queries #30
Comments
Some more discussion at numpy/numpy#4840 |
See this as well numpy/numpy#4398 |
Won't fix. |
Why not? |
@153957: We cannot change the numpy query limit and a work-around (in kernel query for large querys) is too much hassle. I'm happy with leaving this open, but I was trying to close issues that seem too much hassle to fix. (Too low return on investment) |
It would actually be nice if this is fixed, currently simply selecting 'Amsterdam' as cluster in the Coincidence download form returns the 'to many stations' error: Is it to much trouble to convince NumPy to change it on their side? I know various attempts have been made, with little progress.. |
Ok, that make sense. Let's at least keep this open and hope numpy fixes this. (Or do the workaround thing...). |
I think this should at least be kept open as long as the numpy issue above is still open. |
The value was increased in NumPy 2, you can even manually set it higher it seems. However, PyTables does not fully support NumPy 2 yet PyTables/PyTables#1172 |
Awaiting the next release of PyTables which includes this fix; PyTables/PyTables#1208 |
Due to some (arbitrary?) limit in NumPy (numpy/numpy#4398) there is a limit to the number of inputs in a query in PyTables (PyTables/PyTables#286).
This mostly affects the coincidence_queries module where queries may include many station columns.
For instance, if looking for coincidences that include all stations in the Amsterdam cluster, more than 32 columns will be queried. This will fail.
Example using all station columns:
We could work around this by evaluating the query in python instead of in-kernel.
However that would slow execution..
Perhaps we can work around this limit or see if we can also vote for an increase in the limit.
The text was updated successfully, but these errors were encountered: