search

julia-vscode / LanguageServer.jl

An implementation of the Microsoft Language Server Protocol for the Julia language.
Other
368 stars 81 forks source link

Go to definition problem within the ferrite package #1229

Open andreichalapco opened 1 year ago

andreichalapco commented 1 year ago

Hi,

I'm currently working with the ferrite package for FEM and I've encountered the problem of go to definition for some particular structs.

using Ferrite

ip = Lagrange{dim, RefCube, 2}()

the struct is located in: https://github.com/Ferrite-FEM/Ferrite.jl/blob/fd4cfd399587e76bd3974bbf7cc93ca3044ab5d3/src/interpolations.jl#L455-L459

I don't know if this is a recurrent problem with go to definition in julia packages. Found the pattern actually very curious

fredrikekre commented 1 year ago

The definition on the latest release is https://github.com/Ferrite-FEM/Ferrite.jl/blob/b937ea679c34effbd5055fc7e41c7328baff77b9/src/interpolations.jl#L315 which is just a standard struct definition. Note that jumping to the definition of RefCube (https://github.com/Ferrite-FEM/Ferrite.jl/blob/b937ea679c34effbd5055fc7e41c7328baff77b9/src/Ferrite.jl#L22) works just fine, so it is not like the LS is completely broken for this package or version.

fredrikekre commented 1 year ago

https://github.com/Ferrite-FEM/Tensors.jl/blob/8df6d26f4564e9eb534c94e69be051f12a4cd83e/src/Tensors.jl#L45-L48 similarly doesn't have a location. Both these structs use inner constructors, maybe that is a lead...