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

Function estimate_dimension is missing from separated_de/estimate_dimension.jl #133

Closed
CNelias opened this issue Jan 10, 2024 · 9 comments
Closed

Comments

@CNelias
Copy link

CNelias commented Jan 10, 2024

Describe the bug
When trying to call the function estimate_dimension e.g. as estimate_dimension(signal, ...) the following error is returned:

ERROR: UndefVarError: `estimate_dimension` not defined
Stacktrace:
 [1] getproperty
   @ .\Base.jl:31 [inlined]
 [2] avg_dimension(sigs::Matrix{Float64})
   @ Main d:\Corentin\Julia\Glial_mitochondria\src\mitochondrial_dynamics.jl:34
 [3] top-level scope
   @ d:\Corentin\Julia\Glial_mitochondria\src\mitochondrial_dynamics.jl:57

Looking at the source code in separated_de/estimate_dimension.jl there seems to be no such function there, as opposed to the function estimate_delay, which is found in estimate_delay.jl

Minimal Working Example

using DynamicalSystems
using Random
test = rand(1000)
 estimate_dimension(signal)

Package versions

DyncamicalSystems 3.2.3
@Datseris
Copy link
Member

estimate_dimension does not appear to be an exported name of DelayEmbeddings.jl:

https://juliadynamics.github.io/DelayEmbeddings.jl/stable/search/?q=estimate_dimension

where did you find it from?

@CNelias
Copy link
Author

CNelias commented Jan 10, 2024

@Datseris
Copy link
Member

This links to version of the 1.3 docs, which is 6 years old, probably older. Current version of the software is 3.2.3 as also stated in your report of package versions. I assume this issue is because of consluting a by-now invalid documentation page. Use instead the current one (https://juliadynamics.github.io/DynamicalSystemsDocs.jl/delayembeddings/stable/) and if there are still problems with missing names re-open the issue!

@Datseris
Copy link
Member

Or, if you want to stick to using the v1.3 docs, that is also doable but you need to explicitly add v1.3 of DynamicalSystems.jl at your project.

@CNelias
Copy link
Author

CNelias commented Jan 11, 2024

Oh my bad, sorry for the trouble. I was using the first link that google provided me for the doc...

@Datseris
Copy link
Member

Perhaps I can fix that. Do you remember what was the query you used in google? All queries I've tried now bring me either to the latest stable version or to the development version (which is the same as latest stable).

@CNelias
Copy link
Author

CNelias commented Jan 11, 2024

I don't remember precisely, but for instance when I google dynamical systems jl, the fourth link that shows up is the version 1.3 (https://juliadynamics.github.io/DynamicalSystems.jl/v1.3/)

@kahaaga
Copy link
Member

kahaaga commented Jan 11, 2024

Same result here when using Google as the search engine. The first link leads to the correct docs, the fourth link leads to some very old docs.

Skjermbilde 2024-01-11 kl  13 30 02

@Datseris
Copy link
Member

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

3 participants