Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/417
This PR did not affect the master branch in any way, I am going to open a
new one. I did not close this PR, it was merged automatically due to this PR
#418
---
If your project is set up
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/418
@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
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/418
Okay on it, Can you let me know what you wanted to add into the READme? I
did not get that part. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/418
I have a theory that I have learnt from my digging up on this kind of
issues, the modules that we are using in turn have unmet dependencies and their
respective owners need t update them, I am
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/418
@jarifibrahim yes exactly, I got the same errors and I mentioned the same
earlier, according to my understanding the errors are not caused due to our
part of the code, I have updated every
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/418
@lewismc yes I did. Just to be sure node dependencies is the only folder
that npm install generates right?? If that's right then I did delete that
folder and tried the patch.
---
If your
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/418
@agoodm this is not regarding travis build this is regarding the node
dependencies. Please correct me if I am wrong.
---
If your project is set up for it, you can reply to this email and have
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/418
@lewismc I tried this PR out in a clean virtualenv and I still got the same
errors. I think we need to take others opinion if everything is working fine in
your system with this PR
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/418
@lewismc I am on a vacation so I won't be able to do it, I am gonna update
this as soon as I return to my university this sunday. Thanks.
---
If your project is set up for it, you can reply
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/418
I haven't tested the branch yet, I am testing it now. I merged your PR just
now into my branch and hence it appeared here.
---
If your project is set up for it, you can reply to this email
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/418
I don't know how the first commit got in the way here. Anyways just try out
the package.json once and I will close this PR.
---
If your project is set up for it, you can reply to this email
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/418
CLIMATE-886 Updating front-end dependencies.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Omkar20895/climate CLIMATE-886
Alternatively
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/417
Yes sure, I did not notice this issue, I will work on it. Thanks for the
heads up!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/417
CLIMATE-877 Adding PODAAC data source to OCW GUI
- This PR adds PODAAC to OCW GUI.
- This PR will be updated until the community is satisfied with the code
addition.
- Right now
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/411
Update it with the change that I have mentioned above. 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
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/411
PO.DAAC forum is not responding anything regarding this shall we update the
PR for now and add the downloading the dataset part as soon as they reply?
Thanks very much.
---
If your project
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/411
ping @lewismc
---
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 Omkar20895 commented on a diff in the pull request:
https://github.com/apache/climate/pull/411#discussion_r81418144
--- Diff: ocw/data_source/podaac_datasource.py ---
@@ -41,9 +42,39 @@ def convert_times_to_datetime(time):
parsed_time = utime(units
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/411
Yes I have opened an issue in the morning regarding this in PO.DAAC forum,
forgot to mention here.
Here is the
[link](https://podaac.jpl.nasa.gov/forum/viewtopic.php?f=53=424
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/411
@lewismc I can do one thing, I can write a script in my local machine and
track the time taken on average for each request sent to podaac and share the
results here. Let me know what you think
Github user Omkar20895 closed the pull request at:
https://github.com/apache/climate/pull/409
---
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
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/409
closing this off, since it is a duplicate of #410
---
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 Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/410
LGTM. I have checked it in my local machine. +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
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/411
Hi @lewismc 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
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/409
@lewismc can you please check if you are able to reproduce the warnings
once again. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/409
CLIMATE-852 Addressing warnings in ocw documentation build
- Addressed the warnings in documenation.
- Made the conf.py code PEP8 compliant.
You can merge this pull request into a Git
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/401
@agoodm No coveralls is not required for the podaacpy code to run, it just
checks the total coverage the tests have on the actual code, so basically it is
just an add on for checking the code
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
Pheww! The PR is merged and with this my GSOC would have come to an end,
but obviously PO.DAAC JPL doesn't want that(:P). Anyways it has been a really
good learning experience and I would
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
To conclude the PR:
- I have integrated PO.DAAC data source to OCW.
- All tests run 'OK' and tests can be added further.
- I have also added an example for using podaac
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
@lewismc you are right, the search_granule has been compromised ð
, they
have removed the shortName option from it, this PR is gonna take a while to get
merged now. we need to update podaacpy
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
@lewismc I think the new version luckily does not affect the present
behaviour of podaac_datasource since podaac_datasource depends solely on
search_granule and dap which are intact in the new
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
Finally!! The build passed OK! ð @lewismc I will add in a commit and
merge it, Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
@lewismc roger that! I will change them. I haven't worked on this branch
because the podaacpy installation is broken, please look at the latest build on
this branch that has failed(look
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
@agoodm yes definitely! As soon as this PR is merged let's open another PR
and work on setting up a conda recipe for podaac. Thanks.
---
If your project is set up for it, you can reply
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
@lewismc I will push a trivial update as soon as the podaacpy installation
becomes stable and validate the conda installation after integrating this.
Thanks.
---
If your project is set up
Github user Omkar20895 commented on a diff in the pull request:
https://github.com/apache/climate/pull/383#discussion_r75575915
--- Diff: ocw/data_source/podaac.py ---
@@ -0,0 +1,111 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
@lewismc Please take a look at the above change.
This [cheat sheet](http://python-future.org/compatible_idioms.html) could
come in handy for in future for checking the differences between
Github user Omkar20895 commented on a diff in the pull request:
https://github.com/apache/climate/pull/383#discussion_r75501372
--- Diff: ocw/data_source/podaac.py ---
@@ -0,0 +1,113 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/395
@lewismc this PR was opened way before @jarifibrahim fixed the tests and
hence travis-ci build failed then, I have asked @jarifibrahim to cross check it
in his local machine, after his 'GO' I
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/396
+1 for merge. 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
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/395
CLIMATE-853 Fixing broken documentation in dap.py and making it PEP8 â¦
â¦compliant
- This PR addresses issue CLIMATE-853.
- I have removed '_' in the function
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/390
CLIMATE-830 Reverting the broken test case in test_dap.py
- This PR comments the failing test case in test_dap.py as suggested by
@lewismc.
- We can comment it out once we find a good fix
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
@lewismc I think the failure of the tests is due to some other module but
not podaac, I have run the tests cases in my local machine all the tests
related to podaac data source are running OK
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/376
@jarifibrahim I did not merge this PR and I think the PR has been merged to
address this [issue](https://issues.apache.org/jira/browse/CLIMATE-830).
---
If your project is set up for it, you
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
@lewismc I got that earlier, i had some merge conflicts while I was
squashing and it was late at night, so I figured I would solve them in the
morning, I will work on it now, sorry for the delay
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/383
@lewismc sure thing!
---
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 Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/383
Adding PO.DAAC data source along with the tests
- This PR adds PO.DAAC data source to ocw.
- This PR also addresses the required tests.
Any suggestions are welcome, Thanks.
You
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/379
I am closing this PR and sending a new one which works with all test cases
running OK. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user Omkar20895 commented on a diff in the pull request:
https://github.com/apache/climate/pull/379#discussion_r72910817
--- Diff: ocw/data_source/PODAAC.py ---
@@ -0,0 +1,104 @@
+# Licensed to the Apache Software Foundation (ASF) under one
--- End diff
Github user Omkar20895 commented on a diff in the pull request:
https://github.com/apache/climate/pull/379#discussion_r72908642
--- Diff: ocw/data_source/PODAAC.py ---
@@ -0,0 +1,104 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/379
@lewismc
---
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 Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/376
CLIMATE-830 Broken tests in dap.py
- The tests fail because the dataset that I have used for testing does not
contain time variable in it.
- The dap.py runs on a presumption
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/359
Addressing CLIMATE-810 Demo tab doesn't appear in ocw ui
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Omkar20895/climate master
Github user Omkar20895 commented on the issue:
https://github.com/apache/climate/pull/356
@lewismc @huikyole @MJJoyce Can you please review this?? 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
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/318#issuecomment-207793570
So I was just wondering if all the conflicts in the dependencies are
solved? If they are, just let me know. The VM has been delayed for too long and
we haven't
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/324#issuecomment-207032739
Done :)
---
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 Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/324#issuecomment-207031449
Yeah! I will do it in an hour.
---
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 user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/324#issuecomment-204896114
With this addressed we can close #318 and get going with the VM, the work
was stalled because of conflicting dependencies during the installation on the
VM
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/323#issuecomment-204766408
I have tested the code in my local machine and the code does have the same
import errors in multiple lines and in multiple files. @jarifibrahim Thanks for
pointing
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/324#issuecomment-204765483
Hello @jarifibrahim, to my knowledge we have numpy 1.11.0 and h5py 2.6.0
available and are pretty stable versions. If we are going to update the
dependencies why
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/317
CLIMATE-572 Address deprecation and WARN's in ocw-ui/frontend npm insâ¦
â¦tall
This pull requests adds a new package 'grunt-ng-annotate' instead of
'grunt-ngmin' which is deprecated
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/311#issuecomment-188782167
@lewismc yeah I am sure that I can test this patch locally by pulling it.
But, I am not sure of a way to update THIS pull request with my changes. Pardon
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/311#issuecomment-188627089
@lewismc can you commit this pull request so that I can pull it and add my
changes accordingly??
Thanks
---
If your project is set up for it, you can reply
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/311#issuecomment-188430566
Hey @lewismc
I will send the snippet for
>>The script does not install dependencies via anaconda. That means modules
'h5py', 'netcdf4
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/286#issuecomment-179964688
@lewismc just out of curiosity I went ahead and tried this patch. It worked
successfully and I was able to import to all the packages mentioned in this
commit
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/297#issuecomment-177595354
@MichaelArthurAnderson Please put all the small code changes in one pull
request rather than multiple requests unless you feel the necessity for another
pull
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/295#issuecomment-177593936
Thanks for the patch @MichaelArthurAnderson :)
---
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 Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/281
CLIMATE-715 Adding a new demo tab along with the evaluate and result.
- Added a new demo tab.
- The tab includes a demo video of how the web-app works.
- Added a new footer
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/276#issuecomment-175762787
I haven't been granted the write access to the repository yet. right??
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/276#issuecomment-173590721
hey @lewismc @MJJoyce, please have a look at this patch.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/276
CLIMATE-379 - Allows dataset customisation
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Omkar20895/climate master
Alternatively you can
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/271
Resolve CLIMATE-407 Dataset select does not queue empty datasets.
Addresses the issue CLIMATE-407. Please have a look at it.
You can merge this pull request into a Git repository by running
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/270
Resolve CLIMATE-561 : wraping up the error in install-ubuntu.sh
We haven't removed the code discrepancy in the install-ubuntu.sh. This pull
request solves it. @lewismc please have a look
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/270#issuecomment-166128255
I have updated the pull request which solves the issue CLIMATE-560. Please
verify it.
---
If your project is set up for it, you can reply to this email and have
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/269#issuecomment-165959942
Thank you guys @lewismc and @MJJoyce for bearing with me though I am a
novice to the open source development :) I also wanted to ask one more thing, I
am on my
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/269#issuecomment-165420823
No, this pull request is active. The previous pull request with ticket #255
got clumsy with unnecessary commits and got corrupted in my local fork so I
closed
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/260#issuecomment-158738608
Hello @MJJoyce can you please cancel my pull request?? I have messed up my
fork repository of ocw and I am unable to revert it. I will fork the repo
again, apply
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/260
CLIMATE-561 Prompts the user for a destination directory to install the ocw
virtaulenv.
Hello @lewismc I have tried to squash the commits as you mentioned in the
previous pull request but I
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/260#issuecomment-156460989
Sorry, my mistake. @MJJoyce
---
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 user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/255#issuecomment-156275041
Thank you. @lewismc
---
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 Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/255#issuecomment-155437421
hello @lewismc can you merge the snippet now??
@MJJoyce thanks for the code review. :)
---
If your project is set up for it, you can reply to this email
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/255#issuecomment-153823906
@lewismc that would be great! :)
---
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 Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/255#issuecomment-153478791
Hi @lewismc can you specify your os details because I have run the script
so many times on OS X and it shows me the output log with all the executing
commands. I am
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/255#issuecomment-152341615
Hi @lewismc
According to my new version of the patch when you enter ./install-osx.sh -e
it should not prompt for a yes or no.
If you enter ./install-osx.sh
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/255#issuecomment-151474710
Hello lewis, I have updated the pull request and sorry there were so many
errors in the patch. I just hope there aren't any now. I am also attaching the
screenshot
GitHub user Omkar20895 opened a pull request:
https://github.com/apache/climate/pull/255
CLIMATE-556 - Changes in easy_install scripts : creates new virtualenv by
prompting the user for permission.
Added a patch which enables the user to grant permission for creating a new
Github user Omkar20895 commented on the pull request:
https://github.com/apache/climate/pull/255#issuecomment-151231660
I have updated the pull request, run it and let me know if there is any
trouble.
---
If your project is set up for it, you can reply to this email and have your
87 matches
Mail list logo
