Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/522
Thank you @markschn thanks for following up here.
---
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 user lewismc commented on the issue:
https://github.com/apache/climate/pull/513
+1, @prateekiiest thank you
---
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
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 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 user lewismc commented on the issue:
https://github.com/apache/climate/pull/503
@Peter-Gibson ping
---
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 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
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/513
@prateekiiest can you please provide a Unit test?
---
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/512
Excellent @prateekiiest I will merge
---
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 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 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 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 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
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 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 exi
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/494
+1 @agoodm
---
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 user lewismc commented on the issue:
https://github.com/apache/climate/pull/488
Can you please update to master branch @MichaelArthurAnderson
---
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 user lewismc commented on the issue:
https://github.com/apache/climate/pull/490
Can you please update to master branch @MichaelArthurAnderson thank you
---
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/493
Hi @MichaelArthurAnderson I just checked our private@ mailing list archives
and this was because in the past you chose not to join the project as a
committer and PMC member. Has your situation
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/483
+1 @MichaelArthurAnderson please push to remote master
---
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/485
+1 @MichaelArthurAnderson please push to remote master
---
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/479
+1 @MichaelArthurAnderson please push to remote master
---
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/489
+1 @MichaelArthurAnderson please push to remote master
---
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/490
+1 @MichaelArthurAnderson please push to remote master
---
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/491
+1 @MichaelArthurAnderson please push to remote master
---
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/492
+1 @MichaelArthurAnderson please push to remote master
---
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/493
+1 @MichaelArthurAnderson please push to remote master
---
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 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 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 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
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
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
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
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
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
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
GitHub user lewismc opened a pull request:
https://github.com/apache/climate/pull/447
CLIMATE-920 Make examples Python 3 compatible
This issue addresses https://issues.apache.org/jira/browse/CLIMATE-920
You can merge this pull request into a Git repository by running:
$ git
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 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
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
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
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
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
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
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
GitHub user lewismc opened a pull request:
https://github.com/apache/climate/pull/438
CLIMATE-885 Upgrade to Podaacpy 1.9.0
Hi folks, this is the attempt to upgrade to Podaacpy 1.9.0, @Omkar20895 can
you try this out? You can also try the example and tests which should give you
a
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 wrote:
> @lewismc <https://github.com/lewismc> I have replicated the README in the
> root
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
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
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 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
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
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
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
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 appe
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 ha
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 GitH
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
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
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
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
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
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/416
+1 from me
On Monday, October 24, 2016, Alex Goodman wrote:
> Fixed. Once again CI build failure seems to be due to connectivity issues,
> so I will merge later today
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
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):
'''
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
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
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 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
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
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
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
Github user lewismc commented on the issue:
https://github.com/apache/climate/pull/411
> shall we update the PR for now
Update it with what?
> and add the downloading the dataset part as soon as they reply?
I'll go and speak with someone face
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
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 wrote
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
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?
-
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
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 wrote:
> LGTM. I have checked it in my local machine. +1
>
> â
> You a
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
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
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
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
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
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
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
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
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
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
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
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
myproxyclie
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
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
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
1 - 100 of 269 matches
Mail list logo
