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

Warnings in Julia v1.9 #108

Open
juliohm opened this issue Jan 11, 2023 · 2 comments
Open

Warnings in Julia v1.9 #108

juliohm opened this issue Jan 11, 2023 · 2 comments

Comments

@juliohm
Copy link

juliohm commented Jan 11, 2023

Appreciate if you can take a look at these warnings:

WARNING: method definition for broadcasted at C:\Users\julio\.julia\packages\TypedTables\dycVq\src\DictTable.jl:301 declares type variable N but does not use it.
WARNING: method definition for broadcasted at C:\Users\julio\.julia\packages\TypedTables\dycVq\src\DictTable.jl:305 declares type variable N but does not use it.
WARNING: method definition for broadcasted at C:\Users\julio\.julia\packages\TypedTables\dycVq\src\DictTable.jl:309 declares type variable N but does not use it.
WARNING: method definition for broadcasted at C:\Users\julio\.julia\packages\TypedTables\dycVq\src\DictTable.jl:313 declares type variable N but does not use it.
WARNING: method definition for broadcasted at C:\Users\julio\.julia\packages\TypedTables\dycVq\src\DictTable.jl:317 declares type variable N but does not use it.
@dmbates
Copy link
Contributor

dmbates commented Feb 11, 2023

Fixed in #107 I think

@dmbates
Copy link
Contributor

dmbates commented Feb 11, 2023

Another series of warnings of the form

Test Summary: | Pass  Total  Time
Table         |  119    119  3.2s
┌ Warning: use values(kwargs) and keys(kwargs) instead of kwargs.data and kwargs.itr
│   caller = #FlexTable#72 at FlexTable.jl:27 [inlined]
└ @ Core ~/.julia/dev/TypedTables/src/FlexTable.jl:27
┌ Warning: use values(kwargs) and keys(kwargs) instead of kwargs.data and kwargs.itr
│   caller = FlexTable(ts::FlexTable{1}; kwargs::Base.Pairs{Symbol, Vector{Bool}, Tuple{Symbol}, NamedTuple{(:c,), Tuple{Vector{Bool}}}}) at FlexTable.jl:27
└ @ TypedTables ~/.julia/dev/TypedTables/src/FlexTable.jl:27
┌ Warning: use values(kwargs) and keys(kwargs) instead of kwargs.data and kwargs.itr
│   caller = FlexTable(::FlexTable{1}, ::Vararg{Any}; kwargs::Base.Pairs{Symbol, Union{}, Tuple{}, NamedTuple{(), Tuple{}}}) at FlexTable.jl:27
└ @ TypedTables ~/.julia/dev/TypedTables/src/FlexTable.jl:27
┌ Warning: use values(kwargs) and keys(kwargs) instead of kwargs.data and kwargs.itr
│   caller = FlexTable(ts::FlexTable{1}; kwargs::Base.Pairs{Symbol, Nothing, Tuple{Symbol}, NamedTuple{(:b,), Tuple{Nothing}}}) at FlexTable.jl:27
└ @ TypedTables ~/.julia/dev/TypedTables/src/FlexTable.jl:27
...

are produced when testing on v1.8.5 and later. I will create a PR to suppress those.

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