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

Unable to implement example #95

Closed
Deepakgthomas opened this issue Jul 1, 2019 · 3 comments
Closed

Unable to implement example #95

Deepakgthomas opened this issue Jul 1, 2019 · 3 comments

Comments

@Deepakgthomas
Copy link

Deepakgthomas commented Jul 1, 2019

Hello,

I am trying to test the ClustForOpt.jl software, however when I try running a variation of the example provided in the readme -

using ClustForOpt
ts_input_data = load_timeseries_data("DAM", "GER")

I get the following error:
MethodError: no method matching load_timeseries_data(::String, ::String)
Closest candidates are:
load_timeseries_data(::Any; region, T, years, att) at /home/nvidia/.julia/packages/ClustForOpt/YNrmS/src/utils/load_data.jl:19

Stacktrace:
[1] top-level scope at In[38]:2

@holgerteichgraeber
Copy link
Owner

holgerteichgraeber commented Jul 1, 2019

Thank you for testing ClustForOpt, and thank you for pointing that out!
The example seems to be an old version of load_timeseries_data().
See workflow_introduction.jl for a working example.

I am currently rewriting the documentation and README, the documentation branch has the newest version. I hope it should be up in dev and master in 1-2 weeks. Have a look if you like to see hopefully improved descriptions of how to use ClustForOpt.jl. Feedback welcome anytime.
In the documentation branch, there is also a new function to load_timeseries_data() where you just specify the data import by saying for example load_timeseries_data(:CEP_GER1).

Before the new documentation is deployed, you would have to read the actual files in the documentation branch, e.g. https://github.com/holgerteichgraeber/ClustForOpt.jl/blob/documentation/docs/src/load_data.md .

@Deepakgthomas
Copy link
Author

Thank you for the quick information :)

@holgerteichgraeber
Copy link
Owner

@Deepakgthomas the new documentation and also the load_timeseries_data() function is now in the current stable version of ClustForOpt.

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