Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign up[REVIEW]: ClustForOpt: Time-series aggregation for optimization in Julia #1573
Comments
whedon
assigned
danielskatz
Jul 16, 2019
whedon
added
the
review
label
Jul 16, 2019
whedon
referenced this issue
Jul 16, 2019
Closed
[PRE REVIEW]: ClustForOpt: Time-series aggregation for optimization in Julia #1562
This comment has been minimized.
This comment has been minimized.
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @jgoldfar, @ahwillia it looks like you're currently assigned to review this paper If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews To fix this do the following two things:
For a list of things I can do to help you, just type:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Jul 16, 2019
Note: @ahwillia is traveling for the next few weeks. It will be about 3 weeks until he can work on this review. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Jul 16, 2019
whedon
assigned
jgoldfar and
ahwillia
Jul 17, 2019
This comment has been minimized.
This comment has been minimized.
holgerteichgraeber
commented
Jul 23, 2019
|
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Jul 26, 2019
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 2, 2019
This comment has been minimized.
This comment has been minimized.
Reminder set for @ahwillia in 7 days |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 9, 2019
@jgoldfar - any update on this? |
This comment has been minimized.
This comment has been minimized.
Looking through this now. I'm generally very impressed and think we should be able to approve this in short order. I am confirming that I can install and execute the package now. My biggest piece of feedback is that the README and description of the package should emphasize even more applications. I think this package will be broadly useful to many fields! For example, the first sentence of the README might lead users to think the package is for a very specialized purpose I would recommend editing to something like... Current: "ClustForOpt is a julia implementation of unsupervised machine learning methods for finding representative periods for energy systems optimization problems." Revised: "ClustForOpt is a julia implementation of unsupervised machine learning methods for detecting motifs, clustering, and quantifying similarity between time series datasets." Likewise, in the subsequent paragraphs, I recommend adding some more example applications with citations. Segmentation and clustering of audio datasets should be an easy one to find. It is of course okay to say something like "this package was originally developed for energy systems optimization" but I think emphasizing the generality of the package and the methods as much as possible will increase the impact of this work. Full disclosure, I've worked on using very simple time warping methods for neural data (https://www.biorxiv.org/content/10.1101/661165v1), though those data show very different statistics and call for different modeling approaches. But I'm quite enthusiastic about this area of research. |
This comment has been minimized.
This comment has been minimized.
One final thought, I don't insist on changing the name of the repo, but something like "TimeSeriesClustering.jl" would seem to better capture the function of the package. The name ClustForOpt doesn't make it super clear what the package does... Also can the authors comment on the differences between this package and other time series packages in julia (e.g |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
holgerteichgraeber
commented
Aug 10, 2019
Thank you for your feedback, these are great ideas! I am out for the weekend, and will get back to this next week. |
This comment has been minimized.
This comment has been minimized.
holgerteichgraeber
commented
Aug 10, 2019
In case that there are any papers that you can recommend to read in the suggested application areas, suggestions are greatly appreciated. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 15, 2019
This comment has been minimized.
This comment has been minimized.
Checked my boxes...
Eamonn Keogh has a variety of methods and application papers to check out (e.g. https://www.cs.ucr.edu/~eamonn/MatrixProfile.html). I'm sure many other research groups have relevant papers as well. Please don't worry about being comprehensive, but the more references you can find the better. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 17, 2019
Can you please go ahead and do this? (and check the boxes for items that are complete) |
whedon commentedJul 16, 2019
•
edited by ahwillia
Submitting author: @holgerteichgraeber (Holger Teichgraeber)
Repository: https://github.com/holgerteichgraeber/ClustForOpt.jl
Version: v0.4.2
Editor: @danielskatz
Reviewer: @jgoldfar, @ahwillia
Archive: Pending
Status
Status badge code:
Reviewers and authors:
Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)
Reviewer instructions & questions
@jgoldfar & @ahwillia, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @danielskatz know.
Review checklist for @jgoldfar
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?Review checklist for @ahwillia
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?