Bradley Dice
Bradley Dice
This is a meta-issue with several independent tasks. We can cross-link the PR in the checklist and assign the PR to you once it’s open. Thanks!
Hi, unfortunately I won't be much help here. I have done some packaging work for this library but am unfamiliar with its internals. I believe this library is also scheduled...
This may become irrelevant with the ongoing work to unify rapids.ai and docs.rapids.ai.
Thanks for the request! I believe that there are some ongoing website revisions will improve the overall navigation experience when it's ready. cc: @exactlyallan @ajschmidt8 for awareness of this request.
@ajschmidt8 Yes, we can add the sidebar -- but this was the original goal: > I'm mostly interested a fast way to land on the installation page (https://docs.rapids.ai/install). I think...
Closing for now -- I think this is superseded by some ongoing work by @trxcllnt.
@shriramch You can use an owning object like a `std::deque` or `std::list` where references are stable (`std::vector` can reallocate and invalidate references). The lifetime of the AST expressions would then...
For an example of constructing an expression tree, see this benchmark: https://github.com/rapidsai/cudf/blob/23bb2ed156d164b59e608e7e791c74db5cb4bce8/cpp/benchmarks/ast/transform.cpp#L65-L87
Closing as the original question about managing AST expressions was answered.
@raybellwaves I tried to review this but I don't see any changes in the docs. Does this PR affect anything yet, or is it just setup for some future change?