[GitHub] climate pull request: CLIMATE-646 Move the routine to detect tempo...

2015-08-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/climate/pull/211 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] climate pull request: CLIMATE-646 Move the routine to detect tempo...

2015-08-02 Thread MJJoyce
Github user MJJoyce commented on the pull request: https://github.com/apache/climate/pull/211#issuecomment-127059711 :+1: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] climate pull request: CLIMATE-646 Move the routine to detect tempo...

2015-07-31 Thread huikyole
Github user huikyole commented on the pull request: https://github.com/apache/climate/pull/211#issuecomment-126828498 @MJJoyce The routine was originally inside CLI. I found out that this would be a useful routine for other examples. First I added the routine to dataset_processor but

[GitHub] climate pull request: CLIMATE-646 Move the routine to detect tempo...

2015-07-29 Thread MJJoyce
Github user MJJoyce commented on the pull request: https://github.com/apache/climate/pull/211#issuecomment-126122030 Hey @huikyole, I don't really see any problems here but why are we changing the API just to move it from dataset_processor to utils? I'm not really in favor of the

[GitHub] climate pull request: CLIMATE-646 Move the routine to detect tempo...

2015-07-29 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/211#issuecomment-125862679 Merged build finished. Test Passed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] climate pull request: CLIMATE-646 Move the routine to detect tempo...

2015-07-29 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/211#issuecomment-125862126 Merged build started. Test Failed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] climate pull request: CLIMATE-646 Move the routine to detect tempo...

2015-07-29 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/211#issuecomment-125862118 Merged build triggered. Test Failed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your