-
Notifications
You must be signed in to change notification settings - Fork 108
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
Priorities other than A,B,C #312
Comments
Fixed - #313 |
To expand on #313 (comment) a bit, ideally a general solution to this issue would use |
That would work for the predicate body, but it might preclude regexp-based optimizations. It would need to be studied carefully to avoid performance regressions. That parent-priority function in that SE answer wouldn't be necessary in org-ql, because we have the |
Makes sense!
To be clear, I only included that link as one example of a custom priority function, which can be arbitrary. So even if Org QL has optimizations that improve upon that particular function, ultimately it can’t account for arbitrary priority functions without simply calling the function, I think? |
That's right. In the case of custom priority functions, hopefully the regexp-based optimizations can be used to find potential matches and then call the function to get the computed value, like several other predicates do anyway. |
I use priorities below C constantly and for everything, and have just spent way too long being confused about whether I'm using org-ql wrongly, before finally finding a temporary workaround. It'd be really nice to update, or if nothing else change the docs to state that it ignores most values.
Quote from the source:
The text was updated successfully, but these errors were encountered: