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