[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/climate/pull/111 --- 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 ena

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-07-14 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-121488538 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 d

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-07-14 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-121488437 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 project

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-07-14 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-121488445 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 do

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-07-14 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-121487964 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 project

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-07-14 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-121487982 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 do

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-07-14 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-121487998 Merged build finished. 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 d

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-05-11 Thread MJJoyce
Github user MJJoyce commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-100943337 PR == Pull Request DS == Dataset That would be my guess at least --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-05-01 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-98251796 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 does not

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-05-01 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-98251621 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 does not h

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-05-01 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-98251611 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 project does no

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-05-01 Thread huikyole
Github user huikyole commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-98246836 What do PR and DS stand for? Can we use the word ticket rather than tix? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-04-26 Thread MJJoyce
Github user MJJoyce commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-96450855 The dataset_processor changes should be in separate tickets since they don't actually relate to this ticket. Would be worthwhile to update this ticket to note that these

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-04-24 Thread kwhitehall
Github user kwhitehall commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-96043711 @huikyole This PR needs cleaning. For e.g. what your tix says, and what work has been done here don't align. Please be certain to include the other functionality

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83946302 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 do

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83946207 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 project

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83946216 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 doe

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83945996 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 do

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread huikyole
Github user huikyole commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83945925 All of MJJoyce's comments have been addressed. Testing using examples have been successful. --- If your project is set up for it, you can reply to this email and have

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83945897 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 doe

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83945891 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 project

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83944142 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 do

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83944041 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 doe

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83944037 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 project

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83942921 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 project

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83942947 Merged build finished. 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 do

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2015-03-20 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-83942930 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 doe

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2014-11-20 Thread huikyole
Github user huikyole commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-63893960 That is okay. Thank you for taking care of this finally! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2014-11-20 Thread MJJoyce
Github user MJJoyce commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-63854027 Hi @huikyole. Sorry for the horrible horrible delay on getting this reviewed. We definitely dropped the ball. I thought I had commented on this already as was awaiting a

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2014-09-15 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-5556 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 do

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2014-09-15 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-55588766 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 doe

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2014-09-15 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-55588751 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 project

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2014-09-15 Thread MJJoyce
Github user MJJoyce commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-55588718 test this please --- 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 featu

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2014-09-15 Thread MJJoyce
Github user MJJoyce commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-55588731 add to whitelist --- 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 featu

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2014-09-11 Thread OCWJenkins
Github user OCWJenkins commented on the pull request: https://github.com/apache/climate/pull/111#issuecomment-55335308 Can one of the admins verify this patch? --- 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 proje

[GitHub] climate pull request: CLIMATE-467: Replace the existing time decod...

2014-09-11 Thread huikyole
GitHub user huikyole opened a pull request: https://github.com/apache/climate/pull/111 CLIMATE-467: Replace the existing time decoding function with netCDF4 mod... ...ules. You can merge this pull request into a Git repository by running: $ git pull https://github.com/huikyole/