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

Bug in the example of DataFramesMeta doc. #333

Closed
zdlspace0528 opened this issue Jul 5, 2022 · 2 comments
Closed

Bug in the example of DataFramesMeta doc. #333

zdlspace0528 opened this issue Jul 5, 2022 · 2 comments

Comments

@zdlspace0528
Copy link

df = DataFrame(x = [1, 1, 2, 2], y = [1, 2, 101, 102]);
@select(df, :x2 = :x, :x3 = ^(:x))
ERROR:
LoadError: ArgumentError: This path should not be reached
Stacktrace:
 [1] fun_to_vec(ex::Expr; nolhs::Bool, gensym_names::Bool, wrap_byrow::Bool)
   @ DataFramesMeta ~/.julia/packages/DataFramesMeta/z0mc6/src/parsing.jl:302
 [2] (::DataFramesMeta.var"#30#31"{Bool})(ex::Expr)
   @ DataFramesMeta ./none:0
 [3] iterate(::Base.Generator{Vector{Any}, DataFramesMeta.var"#30#31"{Bool}})
   @ Base ./generator.jl:47
 [4] select_helper(::Symbol, ::Expr, ::Vararg{Expr})
   @ DataFramesMeta ~/.julia/packages/DataFramesMeta/z0mc6/src/macros.jl:976
 [5] var"@select"(__source__::LineNumberNode, __module__::Module, x::Any, args::Vararg{Any})
   @ DataFramesMeta ~/.julia/packages/DataFramesMeta/z0mc6/src/macros.jl:1078
in expression starting at REPL[3]:1

@pdeffebach
Copy link
Collaborator

Apologies for being so long delayed in fixing this.

I have a PR #347 which fixed this problem.

pdeffebach added a commit that referenced this issue Feb 6, 2023
* fix problem

* Fix #328 parsing bug try 2 (#346)

* simpler diff

* hygiene

* better fix

* update versions

* improve subsetting explanations (#345)

* Update TagBot.yml (#350)

* update news and version (#353)

* add tests

---------

Co-authored-by: Bogumił Kamiński <[email protected]>
Co-authored-by: Carlo Lucibello <[email protected]>
@pdeffebach
Copy link
Collaborator

Fixed in #347

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

No branches or pull requests

2 participants