[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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-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-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

[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

[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

[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

[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-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-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

[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

[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

[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

[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

[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

[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-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

[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