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

Rename to TimeSeriesClustering.jl #106

Merged
merged 4 commits into from
Aug 18, 2019
Merged

Rename to TimeSeriesClustering.jl #106

merged 4 commits into from
Aug 18, 2019

Conversation

holgerteichgraeber
Copy link
Owner

  • Rename the package to TimeSeriesClustering.jl
  • Update paper.md to reflect it's more general applicability
  • update package version to 0.5.0 to reflect name change

@holgerteichgraeber holgerteichgraeber self-assigned this Aug 18, 2019
@holgerteichgraeber
Copy link
Owner Author

The builds will likely fail due to renaming. Will rename the package after merge, give permissions on travis and codecov, and see if it works.

@holgerteichgraeber holgerteichgraeber merged commit 2a73aa0 into master Aug 18, 2019
Copy link
Collaborator

@YoungFaithful YoungFaithful left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. One question in the code was if JuMP needs to be part of TimeSeriesClustering.jl? It is necessary for the exact methods, correct?

@@ -12,7 +12,7 @@ using Distances
using StatsBase
using JLD2
using FileIO
using JuMP #QUESTION should this be part of ClustForOpt?
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we answer the question with yes?

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

Successfully merging this pull request may close these issues.

None yet

2 participants