[GitHub] climate issue #522: Climate 964
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/522 Thank you @markschn thanks for following up here. ---
[GitHub] climate issue #520: Copyedit overview.rst
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/520 Brilliant @StephenBarlow thanks, this documentation will be published when we next make a release. ---
[GitHub] climate issue #513: [CLIMATE - 957] Include Absolute Biasing under Metrics
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/513 +1, @prateekiiest thank you ---
[GitHub] climate pull request #516: CLIMATE-958 Disable Pydap for Python 2 build, act...
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/516 CLIMATE-958 Disable Pydap for Python 2 build, activate for Python 3 build Addresses https://issues.apache.org/jira/browse/CLIMATE-958 You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate CLIMATE-958 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/516.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #516 commit a3b65da98bbb80a02f1f15bb6495bf549e39de56 Author: Lewis John McGibbney Date: 2018-06-29T21:42:03Z CLIMATE-958 Disable Pydap for Python 2 build, activate for Python 3 build ---
[GitHub] climate issue #506: CLIMATE - 803 Initial Stucture for Handling mask for non...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/506 Hi @prateekiiest please close and reopen again, I added ```https://pypi.org/project/netcdftime/``` to the build profiles ---
[GitHub] climate issue #503: CLIMATE-937 ETCCDI
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/503 @prateekiiest correct, this is with all builds right now. Please see the mailing list and comment there, I've opened a thread for that. Thank you ---
[GitHub] climate issue #506: CLIMATE - 803 Initial Stucture for Handling mask for non...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/506 @huikyole @agoodm can you guys please scope out this PR. Thanks, we are working on the build issues. ---
[GitHub] climate pull request #514: Add cython to build dependency files
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/514 Add cython to build dependency files You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/514.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #514 commit 367ba9e8f3268c4413c2d41b27a5fbf2bd41f15d Author: Lewis John McGibbney Date: 2018-06-26T19:20:32Z Add cython to build dependency files ---
[GitHub] climate issue #513: [CLIMATE - 957] Include Absolute Biasing under Metrics
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/513 @prateekiiest can you please provide a Unit test? ---
[GitHub] climate issue #512: CLIMATE - 950 Convert Readme.md to reStructured Text
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/512 Excellent @prateekiiest I will merge ---
[GitHub] climate issue #506: CLIMATE - 803 Initial Stucture for Handling mask for non...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/506 @prateekiiest do you mean it seems to *fail*? Can you elaborate? ---
[GitHub] climate issue #506: CLIMATE - 803 Initial Stucture for Handling mask for non...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/506 @prateekiiest once we get an example of ndarray data set having missing values it would be great tp create a unit test for this. ---
[GitHub] climate issue #506: CLIMATE - 803 Initial Stucture for Handling mask for non...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/506 Thank you @prateekiiest Over on Jira this issue was reported by @huikyole, he is on vacation right now but I am sure he can answer your question when he returns. ---
[GitHub] climate issue #504: CLIMATE-948 cleanup podaac_datasource to accomodate Poda...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/504 I am not quite sure why the tests are failing with Python 3.5 and 2.7. Can anyone else have a look? I a running Python 3.6.1 :: Continuum Analytics, Inc. ---
[GitHub] climate pull request #504: CLIMATE-948 cleanup podaac_datasource to accomoda...
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/504 CLIMATE-948 cleanup podaac_datasource to accomodate Podaacpy 2.1.0 This PR addresses https://issues.apache.org/jira/browse/CLIMATE-948 You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate CLIMATE-948 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/504.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #504 commit ad5757dbd6ed7043d9565c34cc7460e0408f862c Author: Lewis John McGibbney <lewis.mcgibbney@...> Date: 2018-03-14T01:03:10Z CLIMATE-948 cleanup podaac_datasource to accomodate Podaacpy 2.1.0 ---
[GitHub] climate issue #501: CLIMATE-891 Make (non-ESGF and pydap) examples script py...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/501 @MichaelArthurAnderson can you please update PR to resolve conflicts? Thanks ---
[GitHub] climate issue #498: CLIMATE-744 Cannot load TRMM data from RCMED
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/498 @agoodm @huikyole I have scoped the code out and all looks good here. The issue is well documented over at JIRA. I think we just need to ensure that this patch doesn't break any existing functionality. Any comments? ---
[GitHub] climate issue #494: CLIMATE-946 - CORDEX script does not necessarily use cor...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/494 +1 @agoodm ---
[GitHub] climate issue #486: CLIMATE-942 Integer Passed To Temporal Rebin Rather Than...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/486 Can you please update to master branch @MichaelArthurAnderson thank you ---
[GitHub] climate issue #488: CLIMATE-943 TypeError: Cannot set property uiDateValidat...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/488 Can you please update to master branch @MichaelArthurAnderson ---
[GitHub] climate issue #489: CLIMATE-944 Grunt Errors and Karma Errors
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/489 Can you please update to master branch @MichaelArthurAnderson thank you ---
[GitHub] climate issue #490: CLIMATE-945 Grunt Errors On Build Task
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/490 Can you please update to master branch @MichaelArthurAnderson thank you ---
[GitHub] climate issue #483: CLIMATE-917 Bocumentation build error with Python3
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/483 +1 @MichaelArthurAnderson please push to remote master ---
[GitHub] climate issue #485: CLIMATE-366 Add full temporal rebin option to UI
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/485 +1 @MichaelArthurAnderson please push to remote master ---
[GitHub] climate issue #479: CLIMATE-940 File List Does Not Display On File Selection...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/479 +1 @MichaelArthurAnderson please push to remote master ---
[GitHub] climate issue #489: CLIMATE-944 Grunt Errors and Karma Errors
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/489 +1 @MichaelArthurAnderson please push to remote master ---
[GitHub] climate issue #490: CLIMATE-945 Grunt Errors On Build Task
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/490 +1 @MichaelArthurAnderson please push to remote master ---
[GitHub] climate issue #491: CLIMATE-374 UI runEvaluation doesnt use $window or $loca...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/491 +1 @MichaelArthurAnderson please push to remote master ---
[GitHub] climate issue #492: CLIMATE-413 Extend browser support for ocw-ui/frontend/c...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/492 +1 @MichaelArthurAnderson please push to remote master ---
[GitHub] climate issue #493: CLIMATE-626 Update doc strings in Downscaling class
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/493 +1 @MichaelArthurAnderson please push to remote master ---
[GitHub] climate issue #469: CLIMATE-762 Automate the linking of the frontend directo...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/469 Note that the TravisCI test only fails on Python 2.7 which is fine as it is related to the Pydap dependency. ---
[GitHub] climate issue #469: CLIMATE-762 Automate the linking of the frontend directo...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/469 Hi @MichaelArthurAnderson yes I am seeing 7 changed files and 4 commits. Is this intended? I think you may have merged them into each other. ---
[GitHub] climate issue #469: CLIMATE-762 Automate the linking of the frontend directo...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/469 If we can separate these PR's into individual ones then I think we can begin to merge some of your contributions into master @MichaelArthurAnderson thank you ---
[GitHub] climate pull request #459: CLIMATE-931 Documentation build error - missing m...
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/459 CLIMATE-931 Documentation build error - missing module This issue addresses https://issues.apache.org/jira/browse/CLIMATE-931 and includes ocw-parrallel code as well. You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate CLIMATE-931 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/459.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #459 commit 5c7a9ef1881b0c8875d8a9181afaceb567e072f3 Author: BrianWilson1 <brian.wil...@jpl.nasa.gov> Date: 2017-09-07T23:43:22Z Added ocw-parallel directory for first version of multicore support. commit b2c0b14f27ef24536af78a6258222004f9aa0f7a Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2017-09-11T15:34:31Z Merge branch 'master' of https://github.com/BrianWilson1/climate commit 28c92396d649fe5eacd00a235176fed1667ef80a Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2017-09-11T15:55:41Z CLIMATE-923 ocw-parallel multicore support Python3 support commit 73765a3e99852e67d0f9cd61a06d30221478459e Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2017-10-31T21:33:06Z CLIMATE-931 Documentation build error - missing module commit 4f13a40a7712126169bf1a650bd239872df4a43b Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2017-10-31T21:35:15Z Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/climate commit 4cf79f3d0ad9ed580eacf93f853d693692952aad Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2017-10-31T21:35:26Z Merge branch 'master' into CLIMATE-931 ---
[GitHub] climate issue #449: CLIMATE-887 - test_local.py speed and style guide improv...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/449 Thank you @justinlulejian --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #445: optimizing traversing
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/445 @Kejia can you please update this PR to resolve conflict? Thank you in advance. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #444: optimizing code
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/444 Thank you @Kejia in the future, if you could open an issue at our JIRA instance it would be greatly appreciated. Thank you very much. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #439: CLIMATE-912 Upgrade mccSearch code from Python2 > 3
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/439 Hi @kwhitehall I am sorry I didn't get back to you on this. We need to update the PR to resolve conflicts. I will take a wee look on Monday. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #448: import os
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/448 In the future could you please open an issue at our issue tracker https://issues.apache.org/jira/browse/CLIMATE It erely helps us to track project development. Thank you very much @cclauss --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #448: import os
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/448 Thank you @cclauss --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #445: optimizing traversing
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/445 Yeah, the failure is intermittent and OK for the time being. @kwhitehall can you try this patch out? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #444: optimizing code
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/444 @huikyole @agoodm can you scope this out? It LGTM. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #445: optimizing traversing
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/445 @Kejia did you succesffuly run this with sample MCC-related data? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #445: optimizing traversing
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/445 I just checked and these build failures have nothing to do with the proposed patch. They are dependency related.. I assume they failed due to network issue on build machines. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #439: CLIMATE-912 Upgrade mccSearch code from Python2 > 3
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/439 Hi @kwhitehall I just went to look at some more OCW code and my local branch was set to this issue. I had to come here to remind myself what the issue was about. Did you get any time at all to take a look through this? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #439: CLIMATE-912 Upgrade mccSearch code from Python2 > 3
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/439 Thank you @kwhitehall --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #439: CLIMATE-912 Upgrade mccSearch code from Python2 > 3
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/439 @kwhitehall if you are able to pull this PR locally and try it out with some test data (and reference the data here so I can also try it out) it would be very much appreciated. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #439: CLIMATE-912 Upgrade mccSearch code from Python2 >...
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/439 CLIMATE-912 Upgrade mccSearch code from Python2 > 3 This issue addresses [CLIMATE-912](https://issues.apache.org/jira/browse/CLIMATE-912) You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate CLIMATE-912 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/439.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #439 commit 5c86f3a87afd8c9ed136d59d3d517207efadb20a Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2017-04-26T01:40:44Z CLIMATE-912 Upgrade mccSearch code from Python2 > 3 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #418: CLIMATE-886 Updating front-end dependencies.
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/418 +1 please squash merge. Thanks On Sun, Nov 13, 2016 at 3:57 AM, Omkar <notificati...@github.com> wrote: > @lewismc <https://github.com/lewismc> I have replicated the README in the > root directory of climate and added to climate/ocw-ui/frontend. The warning > was raised because npm was searching for readme data in ocw-ui. I also > added a field readme in package.json, If this is alright then let me know > and I will squash the commits and merge into master. Thanks :) > > â > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/climate/pull/418#issuecomment-260181758>, or mute > the thread > <https://github.com/notifications/unsubscribe-auth/ABHJl6jhCRGWzoirUauToSFgkum_cNpwks5q9vs2gaJpZM4KiyTx> > . > -- *Lewis* Dr. Lewis J. McGibbney Ph.D, B.Sc Director, MCMA Associates Phone: +1(626)498-3090 Skype: lewis.john.mcgibbney Email: lewis.mcgibb...@gmail.com --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #418: CLIMATE-886 Updating front-end dependencies.
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/418 @Omkar20895 http://stackoverflow.com/questions/18057466/npm-how-to-fix-no-readme-data Please squash your commits if you could as well. Thanks, appreciated. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #420: CLIMATE-863 - Address warning, etc within Travis-CI log ...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/420 ACK, please merge into master --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #418: CLIMATE-886 Updating front-end dependencies.
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/418 Thats exactly right. There is nothing we can do about the deprecated transitive dependencies. Please merge in master branch. One thing is that we need to run the tests as well https://github.com/apache/climate/tree/master/ocw-ui/frontend/test Can someone please open a new issue for this? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #420: CLIMATE-863 - Address warning, etc within Travis-CI log ...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/420 @jarifibrahim can you please update this PR and resolve the merge conflict? Thank you --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #418: CLIMATE-886 Updating front-end dependencies.
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/418 @Omkar20895 have you tried deleting the npm-generated material in the ocw-ui directory and trying again? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #419: CLIMATE-889 - Fix Travis Build
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/419 +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 enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #418: CLIMATE-886 Updating front-end dependencies.
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/418 @Omkar20895 can you delete the node dependencies folder, try again with the fresh PR applied and post your output here? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #420: CLIMATE-863 - Address warning, etc within Travis-CI log ...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/420 I've tested this locally and the test output is looking much cleaner. I am +1 to merge into master. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #419: CLIMATE-889 - Fix Travis Build
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/419 I'm +1 for merging this into master folks. Heads up, we've just updated Pydap for Python 3.X support and will be releasing a new version shortly. I think making the basic check and handling the exception is fine in this case. stabilizing the test is more important just now as it means we can drive towards an OC release. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #418: CLIMATE-886 Updating front-end dependencies.
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/418 You've not addressed the README issue in the build. Also can you remove the PO.DAAC stuff? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #418: CLIMATE-886 Updating front-end dependencies.
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/418 Hi @Omkar20895 please merge the following into your remote branch and then test it out. https://github.com/Omkar20895/climate/pull/1 Thanks --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #418: CLIMATE-886 Updating front-end dependencies.
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/418 Actually, hold off on the merge. This is not right. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #418: CLIMATE-886 Updating front-end dependencies.
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/418 @Omkar20895 this is looking good. If you could just add the data to the README then I am +1 to merge into master branch. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #418: CLIMATE-886 Updating front-end dependencies.
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/418 #dynamite --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #417: CLIMATE-877 Adding PODAAC data source to OCW GUI
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/417 @Omkar20895 if I were you, I would attempt to address https://issues.apache.org/jira/browse/CLIMATE-886 and then get the tests passing for the frontend. Once we have it stable again then we can think about adding new data sources. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #416: CLIMATE-883 - Ensure python 3 builds pass unit tests
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/416 +1 from me On Monday, October 24, 2016, Alex Goodman <notificati...@github.com> wrote: > Fixed. Once again CI build failure seems to be due to connectivity issues, > so I will merge later today if no further discussion. > > â > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/climate/pull/416#issuecomment-255831609>, or mute > the thread > <https://github.com/notifications/unsubscribe-auth/ABHJlyWYQjNexzz1Y68qFZIPlNXC_ZWBks5q3P7AgaJpZM4KdtJr> > . > -- *Lewis* Dr. Lewis J. McGibbney Ph.D, B.Sc Director, MCMA Associates Phone: +1(626)498-3090 Skype: lewis.john.mcgibbney Email: lewis.mcgibb...@gmail.com --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #417: CLIMATE-877 Adding PODAAC data source to OCW GUI
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/417 This is a good start. Let me know when you want code review. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #416: CLIMATE-883 - Ensure python 3 builds pass unit te...
Github user lewismc commented on a diff in the pull request: https://github.com/apache/climate/pull/416#discussion_r84599868 --- Diff: ocw/data_source/rcmed.py --- @@ -74,7 +79,8 @@ def _make_mask_array(values, parameter_id, parameters_metadata): ''' for each in parameters_metadata: -if each['parameter_id'].encode() == str(parameter_id): +if str(each['parameter_id']) == str(parameter_id): +print('ok') --- End diff -- Can remove this logging @agoodm --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #415: CLIMATE-875 Upgrade to Podaacpy 1.4.0
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/415 OK folks, so this PR passes against Python 2.7... which is great. Thank you @agoodm for stabilizing and significantly simplifying our builds and packaging. There are a number of issue here with Python 3.4 and 3.5 which we will address over on additional tickets. Note, that the podaac_datasource tests actually pass fine on subsequent versions of Python. If anyone could review this clean PR it would be greatly appreciated. Ta --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #411: CLIMATE-875 Upgrade to Podaacpy 1.4.0
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/411 new PR in https://github.com/apache/climate/pull/415 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #415: CLIMATE-875 Upgrade to Podaacpy 1.4.0
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/415 CLIMATE-875 Upgrade to Podaacpy 1.4.0 Commit to address https://issues.apache.org/jira/browse/CLIMATE-875 You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate CLIMATE-875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/415.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #415 commit 088706d8209204869fbaf24d80c7cc7b7b9aeda3 Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2016-10-21T02:08:23Z CLIMATE-875 Upgrade to Podaacpy 1.4.0 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #411: CLIMATE-875 Upgrade to Podaacpy 1.4.0
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/411 I totally screwed this up. I'll submit another PR. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #411: CLIMATE-875 Upgrade to Podaacpy 1.4.0
Github user lewismc closed the pull request at: https://github.com/apache/climate/pull/411 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #414: CLIMATE-882 - local data source is not Python 3 compatib...
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/414 +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 enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #413: CLIMATE-874 - Remove Easy-OCW
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/413 +1 merge to master 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 feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #411: CLIMATE-875 Upgrade to Podaacpy 1.3.0
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/411 Hi @Omkar20895 what is the ping? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #411: CLIMATE-875 Upgrade to Podaacpy 1.3.0
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/411 Thanks Omkar. Can you please send a Pr to podaacpy to add these status codes to the function documentation? Thanks On Friday, September 30, 2016, Omkar <notificati...@github.com> wrote: > *@Omkar20895* commented on this pull request. > -- > > In ocw/data_source/podaac_datasource.py > <https://github.com/apache/climate/pull/411#pullrequestreview-2402455>: > > > +'''Subset Granule service allows users to Submit subset jobs. \ > +Use of this service should be preceded by a Granule Search in \ > +order to identify and generate a list of granules to be subsetted. > + > +:param input_file_path: path to a json file which contains the \ > +the request that you want to send to PO.DAAC > +:type input_file_path: :mod:`string` > + > +:returns: a token on successful request reception. This can be \ > +further used to check the status of the request. > + > +''' > +podaac = Podaac() > +status = podaac.subset_status(podaac.granule_subset(input_file_path)) > +print("Granule subsetting initiated with request tracking token '%s'." % status) > +while status is not "200": > > Instead of 200 it should be the string "done" here, because as much as I > have noticed subset_token returns the following status > >- "submitted" : returned on successful submission >- "error" : returned when there is error in the JSON POST request. >- "unknown" : I am uncertain about this. >- "done" : returned when subsetting is done. > > â > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/climate/pull/411#pullrequestreview-2402455>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/ABHJlzeAVFbzikngcPfoPhk_kRKUjV-mks5qvXnlgaJpZM4KHN4K> > . > -- *Lewis* Dr. Lewis J. McGibbney Ph.D, B.Sc Director Phone: +1(626)498-3090 Skype: lewis.john.mcgibbney Email: lewis.mcgibb...@gmail.com --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #411: CLIMATE-875 Upgrade to Podaacpy 1.3.0
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/411 BTW, I am unsure right now how we actually obtain the returned subset granule... right now we are only obtaining a token... which is not the data itself. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #411: CLIMATE-875 Upgrade to Podaacpy 1.3.0
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/411 @Omkar20895 I've updated this. See what you think. I also suggest that we try using something like https://pypi.python.org/pypi/progressbar to show progress on the granule subset. Wdyt? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #411: CLIMATE-875 Upgrade to Podaacpy 1.3.0
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/411 @Omkar20895 > I don't think replacing extract_l4_granule is a good idea, because this function is a real time function and it downloads the dataset right away to create an ocw dataset. OK, yes that sounds logical. I mean, if the user wants to download one entire granule then they should be able to do exactly that... so retaining the podaac.extract_l4_granule is good. > One thing I can suggest is instead of replacing extract_l4_granule with granule_subset, we can write and additional function that will make use of granule_subset, in this way the user has a flexibility, ACK > you wouldn't call the granule_subset WS unless you want to subset the dataset granule right? Correct... or a slide of several granules e.g. do an aggregation. > how long does a granule subset job submitted to PO.DAAC takes to complete? I am going to say that the time will increase in a linear fashion depending on the number if granules submitted within the JSON request. The PO.DAAC granule_subset webservice is using OPenDaP's aggregation functionality now so we are one of the first primary consumers of data in this fashion. I suppose we will find out once we use the service more... this is also information we can provide back to the PO.DAAC team. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #410: CLIMATE-852 [OCW Documentation] Theme not found error
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/410 Please go ahead and merge with master branch. On Tuesday, September 27, 2016, Omkar <notificati...@github.com> wrote: > LGTM. I have checked it in my local machine. +1 > > â > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <https://github.com/apache/climate/pull/410#issuecomment-249887129>, or mute > the thread > <https://github.com/notifications/unsubscribe-auth/ABHJlyvvqGIXlpggn5kxmEN33YCILyMwks5quS5CgaJpZM4KHNUo> > . > -- *Lewis* Dr. Lewis J. McGibbney Ph.D, B.Sc Director Phone: +1(626)498-3090 Skype: lewis.john.mcgibbney Email: lewis.mcgibb...@gmail.com --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #411: CLIMATE-875 Upgrade to Podaacpy 1.3.0
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/411 CLIMATE-875 Upgrade to Podaacpy 1.3.0 Hi @Omkar20895 this is the beginning of the upgrade. I've upgraded the pip stuff and also the conda recipe which is important however we need to think about how to upgrade the API and how much of the Podaacpy API to bring in here. I am thinking to replace use of podaac.extract_l4_granule with podaac.granule_subset(input_json) this will provide much more flexibility. I also propose to utilize podaac.subset_status such that we can update the user view console output how long their request is going to take... something like what anaconda and pip do. What do you think? Is there anything I am overlooking which we might want to bring in here? You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate CLIMATE-875 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/411.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #411 commit e56d0216b990933ef0330e0193e66841f591d084 Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2016-09-27T03:15:52Z CLIMATE-875 Upgrade to Podaacpy 1.3.0 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #410: CLIMATE-852 [OCW Documentation] Theme not found e...
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/410 CLIMATE-852 [OCW Documentation] Theme not found error Hi @Omkar20895 this fixes the documentation warning on my end and also runs autopep8 over the entire Python source to ensure that it is PEP8-compliant. Can you please check it all. You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate CLIMATE-852 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/410.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #410 commit 731419f8a2b31b8f3466d78efe7710957c44f813 Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2016-09-27T02:59:22Z CLIMATE-852 [OCW Documentation] Theme not found error --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #408: CLIMATE-873 easy-ocw install-ubun.sh should use m...
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/408 CLIMATE-873 easy-ocw install-ubun.sh should use miniconda3 Addresses https://issues.apache.org/jira/browse/CLIMATE-873 You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate CLIMATE-873 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/408.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #408 commit 98020c9fe7654267d03f8553f35af4ca9ed55952 Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2016-09-15T04:27:02Z CLIMATE-873 easy-ocw install-ubun.sh should use miniconda3 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #407: CLIMATE-854 Ensure that ocw runs with Python 3.X
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/407 CLIMATE-854 Ensure that ocw runs with Python 3.X Updates https://issues.apache.org/jira/browse/CLIMATE-854 to address syntax issues. You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate CLIMATE-854v2 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/407.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #407 commit e050ad55f1e23bed65752faddfb99ed485306ebd Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2016-09-15T04:20:24Z 2nd pass at CLIMATE-854 Ensure that ocw runs with Python 3.X commit bff23ffe3a167ac7f6ff1e5c87fea74a841bd86e Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2016-09-15T04:21:30Z Merge branch 'master' into CLIMATE-854v2 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #379: Adding PO.DAAC data source
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/379 @huikyole can you link the issue you created in Jira here so we can address it? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #400: CLIMATE-854 Ensure that ocw runs with Python 3.X
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/400 I will look at what deprecation is highlted within the travis-ci build. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #400: CLIMATE-854 Ensure that ocw runs with Python 3.X
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/400 CLIMATE-854 Ensure that ocw runs with Python 3.X This PR addresses https://issues.apache.org/jira/browse/CLIMATE-854 You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate CLIMATE-854 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/400.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #400 commit 82856a686578c0f754f4a2960789a348c5842550 Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2016-08-28T17:06:10Z CLIMATE-854 Ensure that ocw runs with Python 3.X --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #383: Adding PO.DAAC data source along with the tests
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/383 @agoodm please reach out to us for any issues. @Omkar20895 excellent work. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #383: Adding PO.DAAC data source along with the tests
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/383 Hi @Omkar20895 yes please. Go ahead and merge. We can work on Podaacpy 1.1.0 and then update here when necessary. Thanks very much. Excellent work and very well done over the summer. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #383: Adding PO.DAAC data source along with the tests
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/383 Cool, if travis-ci build is stable then please merge with master. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #383: Adding PO.DAAC data source along with the tests
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/383 +1 please do. BTW @Omkar20895 did you check out https://github.com/lewismc/podaacpy/issues/64 LOL We should still commit and improve the podaac_datasource based upon what does and does not work within the PO.DAAC WS API --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #383: Adding PO.DAAC data source along with the tests
Github user lewismc commented on a diff in the pull request: https://github.com/apache/climate/pull/383#discussion_r76349961 --- Diff: easy-ocw/ocw-pip-dependencies.txt --- @@ -9,3 +9,4 @@ esgf-pyclient>=0.1.6 python-dateutil>=2.5.3 mock>=2.0.0 myproxyclient>=1.4.3 +podaacpy>=1.0.1 --- End diff -- Once this is upgraded to 1.0.2 we are good to merge this patch. Thanks @Omkar20895 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #399: CLIMATE-861 - An example to plot aerosol optical ...
Github user lewismc commented on a diff in the pull request: https://github.com/apache/climate/pull/399#discussion_r76193927 --- Diff: examples/draw_climatology_map_MISR_AOD.py --- @@ -0,0 +1,34 @@ +import ocw.dataset as ds +import ocw.data_source.local as local +import ocw.dataset_processor as dsp +import ocw.plotter as plotter + +import numpy as np +import numpy.ma as ma + + +''' data source: https://dx.doi.org/10.6084/m9.figshare.3753321.v1''' +dataset = local.load_file('/home/huikyole/climate/examples/AOD_monthly_2000-MAR_2016-FEB_from_MISR_L3_JOINT.nc', --- End diff -- This relative path needs to be replaced ideally by a remote, publicly available one @huikyole --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #399: CLIMATE-861 - An example to plot aerosol optical ...
Github user lewismc commented on a diff in the pull request: https://github.com/apache/climate/pull/399#discussion_r76193809 --- Diff: examples/draw_climatology_map_MISR_AOD.py --- @@ -0,0 +1,34 @@ +import ocw.dataset as ds --- End diff -- License header 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 feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #383: Adding PO.DAAC data source along with the tests
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/383 @Omkar20895 here you go https://pypi.python.org/pypi/podaacpy --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #383: Adding PO.DAAC data source along with the tests
Github user lewismc commented on a diff in the pull request: https://github.com/apache/climate/pull/383#discussion_r76080409 --- Diff: ocw/dataset_loader.py --- @@ -24,6 +24,7 @@ import ocw.data_source.esgf as esgf import ocw.data_source.rcmed as rcmed import ocw.data_source.dap as dap +import ocw.data_source.podaac as podaac --- End diff -- Should be ``` +import ocw.data_source.podaac_datasource as podaac --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #383: Adding PO.DAAC data source along with the tests
Github user lewismc commented on a diff in the pull request: https://github.com/apache/climate/pull/383#discussion_r76080544 --- Diff: ocw/tests/test_podaac.py --- @@ -0,0 +1,67 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# with the License. You may obtain a copy of the License at +# +#http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, +# software distributed under the License is distributed on an +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +# KIND, either express or implied. See the License for the +# specific language governing permissions and limitations +# under the License. + + +import ocw.data_source.podaac as podaac --- End diff -- Should be ``` +import ocw.data_source.podaac_datasource as podaac ``` --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #383: Adding PO.DAAC data source along with the tests
Github user lewismc commented on a diff in the pull request: https://github.com/apache/climate/pull/383#discussion_r76080210 --- Diff: examples/podaac_integration_example.py --- @@ -0,0 +1,75 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# with the License. You may obtain a copy of the License at +# +#http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, +# software distributed under the License is distributed on an +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +# KIND, either express or implied. See the License for the +# specific language governing permissions and limitations +# under the License. + +import ocw.data_source.podaac as podaac --- End diff -- Should be ``` +import ocw.data_source.podaac_datasource as podaac --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate issue #383: Adding PO.DAAC data source along with the tests
Github user lewismc commented on the issue: https://github.com/apache/climate/pull/383 Thanks @agoodm good man --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] climate pull request #397: CLIMATE-859 Enable restructured text markdown for...
GitHub user lewismc opened a pull request: https://github.com/apache/climate/pull/397 CLIMATE-859 Enable restructured text markdown for Pypi uploads This issue addresses https://issues.apache.org/jira/browse/CLIMATE-859 You can merge this pull request into a Git repository by running: $ git pull https://github.com/lewismc/climate CLIMATE-859 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/397.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #397 commit 818784edebcbaedd4786a27b08b1f3af5498c140 Author: Lewis John McGibbney <lewis.mcgibb...@gmail.com> Date: 2016-08-19T20:43:43Z CLIMATE-859 Enable restructured text markdown for Pypi uploads --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---