[GitHub] climate issue #383: Adding PO.DAAC data source along with the tests

[Omkar20895]

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 python 2 and 
3. 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.
---

[jira] [Commented] (CLIMATE-858) Fix Travis Build

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-858?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428648#comment-15428648
 ] 

ASF GitHub Bot commented on CLIMATE-858:


Github user asfgit closed the pull request at:

https://github.com/apache/climate/pull/396


> Fix Travis Build
> 
>
> Key: CLIMATE-858
> URL: https://issues.apache.org/jira/browse/CLIMATE-858
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Reporter: Ibrahim Jarif
>Assignee: Ibrahim Jarif
>Priority: Critical
> Fix For: 1.2.0
>
>
> The travis build is broken due to lack of display variable on travis. The 
> travis.yml file needs to be updated to resolve this issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate issue #396: CLIMATE-858 - Fix Travis Build

[lewismc]

Github user lewismc commented on the issue:

https://github.com/apache/climate/pull/396
  
+1

On Aug 19, 2016 7:21 AM, "Ibrahim Jarif"  wrote:

> @lewismc  Please take a look at this.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> , or 
mute
> the thread
> 

> .
>



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

[jira] [Commented] (CLIMATE-851) Have nosetests log to STDOUT

[lewis john mcgibbney (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-851?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428494#comment-15428494
 ] 

lewis john mcgibbney commented on CLIMATE-851:
--

Yes i gathered that. Btw, can you please update README with exactoy that.




> Have nosetests log to STDOUT
> 
>
> Key: CLIMATE-851
> URL: https://issues.apache.org/jira/browse/CLIMATE-851
> Project: Apache Open Climate Workbench
>  Issue Type: Improvement
>  Components: tests
>Affects Versions: 1.1.0
>Reporter: Lewis John McGibbney
>Assignee: Ibrahim Jarif
> Fix For: 1.2.0
>
>
> Right now it is not good to have no insight or logging for output of the test 
> suite. An example of the logging to STDOUT is provided below. It is kinda 
> useless.
> By default we should have verbose logging.
> {code}
> lmcgibbn@LMC-032857 /usr/local/climate(master) $ nosetests
> EFF..EE.
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (CLIMATE-858) Fix Travis Build

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-858?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428499#comment-15428499
 ] 

ASF GitHub Bot commented on CLIMATE-858:


Github user lewismc commented on the issue:

https://github.com/apache/climate/pull/396
  
+1

On Aug 19, 2016 7:21 AM, "Ibrahim Jarif"  wrote:

> @lewismc  Please take a look at this.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> , or 
mute
> the thread
> 

> .
>



> Fix Travis Build
> 
>
> Key: CLIMATE-858
> URL: https://issues.apache.org/jira/browse/CLIMATE-858
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Reporter: Ibrahim Jarif
>Assignee: Ibrahim Jarif
>Priority: Critical
> Fix For: 1.2.0
>
>
> The travis build is broken due to lack of display variable on travis. The 
> travis.yml file needs to be updated to resolve this issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate pull request #396: CLIMATE-858 - Fix Travis Build

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/climate/pull/396


---
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 #395: CLIMATE-853 Fixing broken documentation in dap.py and ma...

[lewismc]

Github user lewismc commented on the issue:

https://github.com/apache/climate/pull/395
  
This PR is broken @Omkar20895 please fix. If this is part of GSoC (which it 
is) then please clean it up.
The fixes are simple.


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

[jira] [Commented] (CLIMATE-853) Broken documentation in dap.py

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-853?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15427827#comment-15427827
 ] 

ASF GitHub Bot commented on CLIMATE-853:


Github user lewismc commented on the issue:

https://github.com/apache/climate/pull/395
  
This PR is broken @Omkar20895 please fix. If this is part of GSoC (which it 
is) then please clean it up.
The fixes are simple.


> Broken documentation in dap.py
> --
>
> Key: CLIMATE-853
> URL: https://issues.apache.org/jira/browse/CLIMATE-853
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Affects Versions: 1.1.0
>Reporter: Omkar Reddy
>Priority: Minor
> Fix For: 1.2.0
>
>
> When we build the documentation we can observe that sphinx drops the function 
> _convert_times_to_datetime() while building the documentation and this 
> function does not appear in the documentation. 
> The reason for the failure according to me could be that sphinx does not load 
> any function that starts with underscore in its automodule docstrings. I will 
> send a patch for this.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate pull request #383: Adding PO.DAAC data source along with the tests

[lewismc]

Github user lewismc commented on a diff in the pull request:

https://github.com/apache/climate/pull/383#discussion_r75452120
  
--- Diff: ocw/tests/test_podaac.py ---
@@ -0,0 +1,66 @@
+# 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
+import unittest
+import os
+from ocw.dataset import Dataset
+
+
+class TestPodaacDataSource(unittest.TestCase):
--- End diff --

Where is the documentation?


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

[lewismc]

Github user lewismc commented on a diff in the pull request:

https://github.com/apache/climate/pull/383#discussion_r75452231
  
--- Diff: ocw/tests/test_podaac.py ---
@@ -0,0 +1,66 @@
+# 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
+import unittest
+import os
+from ocw.dataset import Dataset
+
+
+class TestPodaacDataSource(unittest.TestCase):
--- End diff --

Does the function do cleanup? Yes or no? If not then it need to clean up. 
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 #396: CLIMATE-858 - Fix Travis Build

[jarifibrahim]

GitHub user jarifibrahim opened a pull request:

https://github.com/apache/climate/pull/396

CLIMATE-858 - Fix Travis Build



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jarifibrahim/climate CLIMATE-858

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/climate/pull/396.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 #396


commit 6e4ae04771eb43f2a4aa86b569897734c56b6ac4
Author: Ibrahim Jarif 
Date:   2016-08-19T13:27:28Z

CLIMATE-858 - Fix Travis 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 issue #396: CLIMATE-858 - Fix Travis Build

[OCWJenkins]

Github user OCWJenkins commented on the issue:

https://github.com/apache/climate/pull/396
  
Can one of the admins verify this patch?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your 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.
---

[jira] [Commented] (CLIMATE-853) Broken documentation in dap.py

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-853?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428226#comment-15428226
 ] 

ASF GitHub Bot commented on CLIMATE-853:


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 will merge this PR. Thanks.


> Broken documentation in dap.py
> --
>
> Key: CLIMATE-853
> URL: https://issues.apache.org/jira/browse/CLIMATE-853
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Affects Versions: 1.1.0
>Reporter: Omkar Reddy
>Priority: Minor
> Fix For: 1.2.0
>
>
> When we build the documentation we can observe that sphinx drops the function 
> _convert_times_to_datetime() while building the documentation and this 
> function does not appear in the documentation. 
> The reason for the failure according to me could be that sphinx does not load 
> any function that starts with underscore in its automodule docstrings. I will 
> send a patch for this.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate issue #395: CLIMATE-853 Fixing broken documentation in dap.py and ma...

[Omkar20895]

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 will merge this PR. 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.
---

[jira] [Commented] (CLIMATE-858) Fix Travis Build

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-858?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428251#comment-15428251
 ] 

ASF GitHub Bot commented on CLIMATE-858:


Github user coveralls commented on the issue:

https://github.com/apache/climate/pull/396
  

[![Coverage 
Status](https://coveralls.io/builds/7523886/badge)](https://coveralls.io/builds/7523886)

Changes Unknown when pulling **6e4ae04771eb43f2a4aa86b569897734c56b6ac4 on 
jarifibrahim:CLIMATE-858** into ** on apache:master**.



> Fix Travis Build
> 
>
> Key: CLIMATE-858
> URL: https://issues.apache.org/jira/browse/CLIMATE-858
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Reporter: Ibrahim Jarif
>Assignee: Ibrahim Jarif
>Priority: Critical
> Fix For: 1.2.0
>
>
> The travis build is broken due to lack of display variable on travis. The 
> travis.yml file needs to be updated to resolve this issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate issue #396: CLIMATE-858 - Fix Travis Build

[coveralls]

Github user coveralls commented on the issue:

https://github.com/apache/climate/pull/396
  

[![Coverage 
Status](https://coveralls.io/builds/7523886/badge)](https://coveralls.io/builds/7523886)

Changes Unknown when pulling **6e4ae04771eb43f2a4aa86b569897734c56b6ac4 on 
jarifibrahim:CLIMATE-858** into ** on apache: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.
---

[jira] [Created] (CLIMATE-858) Fix Travis Build

[Ibrahim Jarif (JIRA)]

Ibrahim Jarif created CLIMATE-858:
-

 Summary: Fix Travis Build
 Key: CLIMATE-858
 URL: https://issues.apache.org/jira/browse/CLIMATE-858
 Project: Apache Open Climate Workbench
  Issue Type: Bug
Reporter: Ibrahim Jarif
Assignee: Ibrahim Jarif
Priority: Critical
 Fix For: 1.2.0


The travis build is broken due to lack of display variable on travis. The 
travis.yml file needs to be updated to resolve this issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (CLIMATE-858) Fix Travis Build

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-858?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428202#comment-15428202
 ] 

ASF GitHub Bot commented on CLIMATE-858:


GitHub user jarifibrahim opened a pull request:

https://github.com/apache/climate/pull/396

CLIMATE-858 - Fix Travis Build



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jarifibrahim/climate CLIMATE-858

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/climate/pull/396.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 #396


commit 6e4ae04771eb43f2a4aa86b569897734c56b6ac4
Author: Ibrahim Jarif 
Date:   2016-08-19T13:27:28Z

CLIMATE-858 - Fix Travis Build




> Fix Travis Build
> 
>
> Key: CLIMATE-858
> URL: https://issues.apache.org/jira/browse/CLIMATE-858
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Reporter: Ibrahim Jarif
>Assignee: Ibrahim Jarif
>Priority: Critical
> Fix For: 1.2.0
>
>
> The travis build is broken due to lack of display variable on travis. The 
> travis.yml file needs to be updated to resolve this issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (CLIMATE-858) Fix Travis Build

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-858?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428203#comment-15428203
 ] 

ASF GitHub Bot commented on CLIMATE-858:


Github user OCWJenkins commented on the issue:

https://github.com/apache/climate/pull/396
  
Can one of the admins verify this patch?


> Fix Travis Build
> 
>
> Key: CLIMATE-858
> URL: https://issues.apache.org/jira/browse/CLIMATE-858
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Reporter: Ibrahim Jarif
>Assignee: Ibrahim Jarif
>Priority: Critical
> Fix For: 1.2.0
>
>
> The travis build is broken due to lack of display variable on travis. The 
> travis.yml file needs to be updated to resolve this issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (CLIMATE-858) Fix Travis Build

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-858?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428215#comment-15428215
 ] 

ASF GitHub Bot commented on CLIMATE-858:


Github user Omkar20895 commented on the issue:

https://github.com/apache/climate/pull/396
  
+1 for merge. Thanks


> Fix Travis Build
> 
>
> Key: CLIMATE-858
> URL: https://issues.apache.org/jira/browse/CLIMATE-858
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Reporter: Ibrahim Jarif
>Assignee: Ibrahim Jarif
>Priority: Critical
> Fix For: 1.2.0
>
>
> The travis build is broken due to lack of display variable on travis. The 
> travis.yml file needs to be updated to resolve this issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate issue #396: CLIMATE-858 - Fix Travis Build

[Omkar20895]

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 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 #395: CLIMATE-853 Fixing broken documentation in dap.py and ma...

[jarifibrahim]

Github user jarifibrahim commented on the issue:

https://github.com/apache/climate/pull/395
  
The doc build works fine on my machine. Please rebase and then merge.


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

[jira] [Commented] (CLIMATE-853) Broken documentation in dap.py

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-853?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428244#comment-15428244
 ] 

ASF GitHub Bot commented on CLIMATE-853:


Github user jarifibrahim commented on the issue:

https://github.com/apache/climate/pull/395
  
The doc build works fine on my machine. Please rebase and then merge.


> Broken documentation in dap.py
> --
>
> Key: CLIMATE-853
> URL: https://issues.apache.org/jira/browse/CLIMATE-853
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Affects Versions: 1.1.0
>Reporter: Omkar Reddy
>Priority: Minor
> Fix For: 1.2.0
>
>
> When we build the documentation we can observe that sphinx drops the function 
> _convert_times_to_datetime() while building the documentation and this 
> function does not appear in the documentation. 
> The reason for the failure according to me could be that sphinx does not load 
> any function that starts with underscore in its automodule docstrings. I will 
> send a patch for this.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate pull request #383: Adding PO.DAAC data source along with the tests

[lewismc]

Github user lewismc commented on a diff in the pull request:

https://github.com/apache/climate/pull/383#discussion_r75452084
  
--- Diff: ocw/data_source/podaac.py ---
@@ -0,0 +1,113 @@
+# 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.
+
+from podaac_data_source import Podaac
+import numpy as np
+from ocw.dataset import Dataset
+from netCDF4 import Dataset as netcdf_dataset
+from netcdftime import utime
+import os
+import urllib
+import xml.etree.ElementTree as ET
+
+
+def convert_times_to_datetime(time):
+'''Convert the time object's values to datetime objects
+
+The time values are stored as some unit since an epoch. These need to 
be
+converted into datetime objects for the OCW Dataset object.
+
+:param time: The time object's values to convert
+:type time: pydap.model.BaseType
+
+:returns: list of converted time values as datetime objects
+'''
+units = time.units
+# parse the time units string into a useful object.
+# NOTE: This assumes a 'standard' calendar. It's possible (likely?) 
that
+# users will want to customize this in the future.
+parsed_time = utime(units)
+return [parsed_time.num2date(x) for x in time[:]]
+
+
+def load_dataset(variable, datasetId='', datasetShortName='', name=''):
+'''Loads a Dataset from PODAAC
+
+:param variable: The name of the variable to read from the dataset.
+:type variable: :mod:`string`
+
+:param datasetId: dataset persistent ID. datasetId or \
+shortName is required for a granule search. Example: \
+PODAAC-ASOP2-25X01
+:type datasetId: :mod:`string`
+
+:param shortName: the shorter name for a dataset. \
+Either shortName or datasetId is required for a \
+granule search. Example: ASCATA-L2-25km
+:type shortName: :mod:`string`
+
+:param name: (Optional) A name for the loaded dataset.
+:type name: :mod:`string`
+
+:returns: A :class:`dataset.Dataset` containing the dataset pointed to 
by
+the OpenDAP URL.
+
+:raises: ServerError
+'''
+# Downloading the dataset using podaac toolkit
+podaac = Podaac()
+path = os.path.dirname(os.path.abspath(__file__))
+granuleName = podaac.extract_l4_granule(
+datasetId=datasetId, shortName=datasetShortName, path=path)
+path = path + '/' + granuleName
+d = netcdf_dataset(path, mode='r')
+dataset = d.variables[variable]
+
+# By convention, but not by standard, if the dimensions exist, they 
will be in the order:
+# time (t), altitude (z), latitude (y), longitude (x)
+# but conventions aren't always followed and all dimensions aren't 
always present so
+# see if we can make some educated deductions before defaulting to 
just pulling the first three
+# columns.
+temp_dimensions = map(lambda x: x.lower(), dataset.dimensions)
+dataset_dimensions = dataset.dimensions
+time = dataset_dimensions[temp_dimensions.index(
+'time') if 'time' in temp_dimensions else 0]
+lat = dataset_dimensions[temp_dimensions.index(
+'lat') if 'lat' in temp_dimensions else 1]
+lon = dataset_dimensions[temp_dimensions.index(
+'lon') if 'lon' in temp_dimensions else 2]
+
+# Time is given to us in some units since an epoch. We need to convert
+# these values to datetime objects. Note that we use the main object's
+# time object and not the dataset specific reference to it. We need to
+# grab the 'units' from it and it fails on the dataset specific object.
+times = np.array(convert_times_to_datetime(d[time]))
+lats = np.array(d.variables[lat][:])
+lons = np.array(d.variables[lon][:])
+values = np.array(dataset[:])
+origin = {
+'source': 'PO.DAAC',
+ 

[GitHub] climate pull request #383: Adding PO.DAAC data source along with the tests

[lewismc]

Github user lewismc commented on a diff in the pull request:

https://github.com/apache/climate/pull/383#discussion_r75452072
  
--- Diff: ocw/data_source/podaac.py ---
@@ -0,0 +1,113 @@
+# 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.
+
+from podaac_data_source import Podaac
+import numpy as np
+from ocw.dataset import Dataset
+from netCDF4 import Dataset as netcdf_dataset
+from netcdftime import utime
+import os
+import urllib
+import xml.etree.ElementTree as ET
+
+
+def convert_times_to_datetime(time):
+'''Convert the time object's values to datetime objects
+
+The time values are stored as some unit since an epoch. These need to 
be
+converted into datetime objects for the OCW Dataset object.
+
+:param time: The time object's values to convert
+:type time: pydap.model.BaseType
+
+:returns: list of converted time values as datetime objects
+'''
+units = time.units
+# parse the time units string into a useful object.
+# NOTE: This assumes a 'standard' calendar. It's possible (likely?) 
that
+# users will want to customize this in the future.
+parsed_time = utime(units)
+return [parsed_time.num2date(x) for x in time[:]]
+
+
+def load_dataset(variable, datasetId='', datasetShortName='', name=''):
+'''Loads a Dataset from PODAAC
+
+:param variable: The name of the variable to read from the dataset.
+:type variable: :mod:`string`
+
+:param datasetId: dataset persistent ID. datasetId or \
+shortName is required for a granule search. Example: \
+PODAAC-ASOP2-25X01
+:type datasetId: :mod:`string`
+
+:param shortName: the shorter name for a dataset. \
+Either shortName or datasetId is required for a \
+granule search. Example: ASCATA-L2-25km
+:type shortName: :mod:`string`
+
+:param name: (Optional) A name for the loaded dataset.
+:type name: :mod:`string`
+
+:returns: A :class:`dataset.Dataset` containing the dataset pointed to 
by
+the OpenDAP URL.
+
+:raises: ServerError
+'''
+# Downloading the dataset using podaac toolkit
+podaac = Podaac()
+path = os.path.dirname(os.path.abspath(__file__))
+granuleName = podaac.extract_l4_granule(
+datasetId=datasetId, shortName=datasetShortName, path=path)
+path = path + '/' + granuleName
+d = netcdf_dataset(path, mode='r')
+dataset = d.variables[variable]
+
+# By convention, but not by standard, if the dimensions exist, they 
will be in the order:
+# time (t), altitude (z), latitude (y), longitude (x)
+# but conventions aren't always followed and all dimensions aren't 
always present so
+# see if we can make some educated deductions before defaulting to 
just pulling the first three
+# columns.
+temp_dimensions = map(lambda x: x.lower(), dataset.dimensions)
+dataset_dimensions = dataset.dimensions
+time = dataset_dimensions[temp_dimensions.index(
+'time') if 'time' in temp_dimensions else 0]
+lat = dataset_dimensions[temp_dimensions.index(
+'lat') if 'lat' in temp_dimensions else 1]
+lon = dataset_dimensions[temp_dimensions.index(
+'lon') if 'lon' in temp_dimensions else 2]
+
+# Time is given to us in some units since an epoch. We need to convert
--- End diff --



Make this Python function documentation rather than hidden docs 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 pull request #383: Adding PO.DAAC data source along with the tests

[lewismc]

Github user lewismc commented on a diff in the pull request:

https://github.com/apache/climate/pull/383#discussion_r75452030
  
--- Diff: ocw/data_source/podaac.py ---
@@ -0,0 +1,113 @@
+# 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.
+
+from podaac_data_source import Podaac
+import numpy as np
+from ocw.dataset import Dataset
+from netCDF4 import Dataset as netcdf_dataset
+from netcdftime import utime
+import os
+import urllib
+import xml.etree.ElementTree as ET
+
+
+def convert_times_to_datetime(time):
+'''Convert the time object's values to datetime objects
+
+The time values are stored as some unit since an epoch. These need to 
be
+converted into datetime objects for the OCW Dataset object.
+
+:param time: The time object's values to convert
+:type time: pydap.model.BaseType
+
+:returns: list of converted time values as datetime objects
+'''
+units = time.units
+# parse the time units string into a useful object.
+# NOTE: This assumes a 'standard' calendar. It's possible (likely?) 
that
+# users will want to customize this in the future.
+parsed_time = utime(units)
+return [parsed_time.num2date(x) for x in time[:]]
+
+
+def load_dataset(variable, datasetId='', datasetShortName='', name=''):
+'''Loads a Dataset from PODAAC
+
+:param variable: The name of the variable to read from the dataset.
+:type variable: :mod:`string`
+
+:param datasetId: dataset persistent ID. datasetId or \
+shortName is required for a granule search. Example: \
+PODAAC-ASOP2-25X01
+:type datasetId: :mod:`string`
+
+:param shortName: the shorter name for a dataset. \
+Either shortName or datasetId is required for a \
+granule search. Example: ASCATA-L2-25km
+:type shortName: :mod:`string`
+
+:param name: (Optional) A name for the loaded dataset.
+:type name: :mod:`string`
+
+:returns: A :class:`dataset.Dataset` containing the dataset pointed to 
by
+the OpenDAP URL.
+
+:raises: ServerError
+'''
+# Downloading the dataset using podaac toolkit
+podaac = Podaac()
+path = os.path.dirname(os.path.abspath(__file__))
+granuleName = podaac.extract_l4_granule(
+datasetId=datasetId, shortName=datasetShortName, path=path)
+path = path + '/' + granuleName
+d = netcdf_dataset(path, mode='r')
+dataset = d.variables[variable]
+
+# By convention, but not by standard, if the dimensions exist, they 
will be in the order:
--- End diff --

Make this Python function documentation rather than hidden docs 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 pull request #383: Adding PO.DAAC data source along with the tests

[lewismc]

Github user lewismc commented on a diff in the pull request:

https://github.com/apache/climate/pull/383#discussion_r75452491
  
--- 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
+import ocw.evaluation as evaluation
+import ocw.metrics as metrics
+import ocw.plotter as plotter
+
+datasetId = 'PODAAC-CCF30-01XXX'
+datasetShortName = 'CCMP_MEASURES_ATLAS_L4_OW_L3_0_WIND_VECTORS_FLK'
+variable = 'uwnd'
+name = 'PO.DAAC_test_dataset'
+OUTPUT_PLOT = "cmc_temporal_std"
+""" Step 1: Load Local NetCDF Files into OCW Dataset Objects """
+print("Loading %s dataset into an OCW dataset objects" % datasetShortName)
+cmc_dataset = podaac.load_dataset(
+variable=variable, datasetId=datasetId, 
datasetShortName=datasetShortName, name=name)
+print "CMC_Dataset.values shape: (times, lats, lons) - %s \n" % 
(cmc_dataset.values.shape,)
+
+# Acessing latittudes and longitudes of netCDF file
+lats = cmc_dataset.lats
+lons = cmc_dataset.lons
+
+""" Step 2:  Build a Metric to use for Evaluation - Temporal STD for this 
example """
+# You can build your own metrics, but OCW also ships with some common 
metrics
+print "Setting up a Temporal STD metric to use for evaluation"
+std = metrics.TemporalStdDev()
+
+""" Step 3: Create an Evaluation Object using Datasets and our Metric """
+# The Evaluation Class Signature is:
+# Evaluation(reference, targets, metrics, subregions=None)
+# Evaluation can take in multiple targets and metrics, so we need to 
convert
+# our examples into Python lists.  Evaluation will iterate over the lists
+print "Making the Evaluation definition"
--- End diff --

Please make all function calls Python3 compatible. The call to print is 
not.


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

[jira] [Commented] (CLIMATE-853) Broken documentation in dap.py

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-853?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428287#comment-15428287
 ] 

ASF GitHub Bot commented on CLIMATE-853:


Github user asfgit closed the pull request at:

https://github.com/apache/climate/pull/395


> Broken documentation in dap.py
> --
>
> Key: CLIMATE-853
> URL: https://issues.apache.org/jira/browse/CLIMATE-853
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Affects Versions: 1.1.0
>Reporter: Omkar Reddy
>Priority: Minor
> Fix For: 1.2.0
>
>
> When we build the documentation we can observe that sphinx drops the function 
> _convert_times_to_datetime() while building the documentation and this 
> function does not appear in the documentation. 
> The reason for the failure according to me could be that sphinx does not load 
> any function that starts with underscore in its automodule docstrings. I will 
> send a patch for this.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate pull request #395: CLIMATE-853 Fixing broken documentation in dap.py...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/climate/pull/395


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

[jira] [Commented] (CLIMATE-858) Fix Travis Build

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-858?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428254#comment-15428254
 ] 

ASF GitHub Bot commented on CLIMATE-858:


Github user jarifibrahim commented on the issue:

https://github.com/apache/climate/pull/396
  
@lewismc Please take a look at this.


> Fix Travis Build
> 
>
> Key: CLIMATE-858
> URL: https://issues.apache.org/jira/browse/CLIMATE-858
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Reporter: Ibrahim Jarif
>Assignee: Ibrahim Jarif
>Priority: Critical
> Fix For: 1.2.0
>
>
> The travis build is broken due to lack of display variable on travis. The 
> travis.yml file needs to be updated to resolve this issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate issue #396: CLIMATE-858 - Fix Travis Build

[jarifibrahim]

Github user jarifibrahim commented on the issue:

https://github.com/apache/climate/pull/396
  
@lewismc Please take a look at 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 pull request #383: Adding PO.DAAC data source along with the tests

[Omkar20895]

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 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.
+
+from podaac_data_source import Podaac
+import numpy as np
+from ocw.dataset import Dataset
+from netCDF4 import Dataset as netcdf_dataset
+from netcdftime import utime
+import os
+import urllib
+import xml.etree.ElementTree as ET
+
+
+def convert_times_to_datetime(time):
+'''Convert the time object's values to datetime objects
+
+The time values are stored as some unit since an epoch. These need to 
be
+converted into datetime objects for the OCW Dataset object.
+
+:param time: The time object's values to convert
+:type time: pydap.model.BaseType
+
+:returns: list of converted time values as datetime objects
+'''
+units = time.units
+# parse the time units string into a useful object.
+# NOTE: This assumes a 'standard' calendar. It's possible (likely?) 
that
+# users will want to customize this in the future.
+parsed_time = utime(units)
+return [parsed_time.num2date(x) for x in time[:]]
+
+
+def load_dataset(variable, datasetId='', datasetShortName='', name=''):
+'''Loads a Dataset from PODAAC
+
+:param variable: The name of the variable to read from the dataset.
+:type variable: :mod:`string`
+
+:param datasetId: dataset persistent ID. datasetId or \
+shortName is required for a granule search. Example: \
+PODAAC-ASOP2-25X01
+:type datasetId: :mod:`string`
+
+:param shortName: the shorter name for a dataset. \
+Either shortName or datasetId is required for a \
+granule search. Example: ASCATA-L2-25km
+:type shortName: :mod:`string`
+
+:param name: (Optional) A name for the loaded dataset.
+:type name: :mod:`string`
+
+:returns: A :class:`dataset.Dataset` containing the dataset pointed to 
by
+the OpenDAP URL.
+
+:raises: ServerError
+'''
+# Downloading the dataset using podaac toolkit
+podaac = Podaac()
+path = os.path.dirname(os.path.abspath(__file__))
+granuleName = podaac.extract_l4_granule(
+datasetId=datasetId, shortName=datasetShortName, path=path)
+path = path + '/' + granuleName
+d = netcdf_dataset(path, mode='r')
+dataset = d.variables[variable]
+
+# By convention, but not by standard, if the dimensions exist, they 
will be in the order:
+# time (t), altitude (z), latitude (y), longitude (x)
+# but conventions aren't always followed and all dimensions aren't 
always present so
+# see if we can make some educated deductions before defaulting to 
just pulling the first three
+# columns.
+temp_dimensions = map(lambda x: x.lower(), dataset.dimensions)
+dataset_dimensions = dataset.dimensions
+time = dataset_dimensions[temp_dimensions.index(
+'time') if 'time' in temp_dimensions else 0]
+lat = dataset_dimensions[temp_dimensions.index(
+'lat') if 'lat' in temp_dimensions else 1]
+lon = dataset_dimensions[temp_dimensions.index(
+'lon') if 'lon' in temp_dimensions else 2]
+
+# Time is given to us in some units since an epoch. We need to convert
+# these values to datetime objects. Note that we use the main object's
+# time object and not the dataset specific reference to it. We need to
+# grab the 'units' from it and it fails on the dataset specific object.
+times = np.array(convert_times_to_datetime(d[time]))
+lats = np.array(d.variables[lat][:])
+lons = np.array(d.variables[lon][:])
+values = np.array(dataset[:])
+origin = {
+'source': 'PO.DAAC',
   

[jira] [Created] (CLIMATE-859) Enable restructured text markdown for Pypi uploads

[Lewis John McGibbney (JIRA)]

Lewis John McGibbney created CLIMATE-859:


 Summary: Enable restructured text markdown for Pypi uploads
 Key: CLIMATE-859
 URL: https://issues.apache.org/jira/browse/CLIMATE-859
 Project: Apache Open Climate Workbench
  Issue Type: Improvement
  Components: documentation
Affects Versions: 1.1.0
Reporter: Lewis John McGibbney
Assignee: Lewis John McGibbney
Priority: Trivial
 Fix For: 1.2.0


Right now the Pypi package page for OCW is a mess.
I'll upload a simple PR to sort this out.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate issue #397: CLIMATE-859 Enable restructured text markdown for Pypi u...

[coveralls]

Github user coveralls commented on the issue:

https://github.com/apache/climate/pull/397
  

[![Coverage 
Status](https://coveralls.io/builds/7529958/badge)](https://coveralls.io/builds/7529958)

Coverage remained the same at 80.271% when pulling 
**818784edebcbaedd4786a27b08b1f3af5498c140 on lewismc:CLIMATE-859** into 
**653bec552b3bdd5eeb15fcbabaa94353d595678a on apache: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.
---

[jira] [Updated] (CLIMATE-855) Fix test_local and test_dataset_processor

[Lewis John McGibbney (JIRA)]

 [ 
https://issues.apache.org/jira/browse/CLIMATE-855?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lewis John McGibbney updated CLIMATE-855:
-
Fix Version/s: 1.2.0

> Fix test_local and test_dataset_processor
> -
>
> Key: CLIMATE-855
> URL: https://issues.apache.org/jira/browse/CLIMATE-855
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Affects Versions: 1.1.0
>Reporter: Huikyo Lee
>Assignee: Huikyo Lee
>Priority: Critical
> Fix For: 1.2.0
>
>
> The recent updates in ocw.data_source.local and ocw.dataset_processor cause 
> errors in nosetests. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Updated] (CLIMATE-856) Removing potential bugs from dataset_processor.spatial_regrid

[Lewis John McGibbney (JIRA)]

 [ 
https://issues.apache.org/jira/browse/CLIMATE-856?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lewis John McGibbney updated CLIMATE-856:
-
Affects Version/s: 1.1.0

> Removing potential bugs from dataset_processor.spatial_regrid
> -
>
> Key: CLIMATE-856
> URL: https://issues.apache.org/jira/browse/CLIMATE-856
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Affects Versions: 1.1.0
>Reporter: Huikyo Lee
>Assignee: Huikyo Lee
> Fix For: 1.2.0
>
>
> The spatial_regrid will be updated to remove the following warning messages.
> - using a boolean instead of an integer will result in an error in the future
> - non integer (and non boolean) array-likes will not be accepted as indices 
> in the future



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Resolved] (CLIMATE-855) Fix test_local and test_dataset_processor

[Lewis John McGibbney (JIRA)]

 [ 
https://issues.apache.org/jira/browse/CLIMATE-855?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lewis John McGibbney resolved CLIMATE-855.
--
Resolution: Fixed

> Fix test_local and test_dataset_processor
> -
>
> Key: CLIMATE-855
> URL: https://issues.apache.org/jira/browse/CLIMATE-855
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Affects Versions: 1.1.0
>Reporter: Huikyo Lee
>Assignee: Huikyo Lee
>Priority: Critical
> Fix For: 1.2.0
>
>
> The recent updates in ocw.data_source.local and ocw.dataset_processor cause 
> errors in nosetests. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate pull request #397: CLIMATE-859 Enable restructured text markdown for...

[lewismc]

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

[jira] [Commented] (CLIMATE-859) Enable restructured text markdown for Pypi uploads

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428776#comment-15428776
 ] 

ASF GitHub Bot commented on CLIMATE-859:


Github user asfgit closed the pull request at:

https://github.com/apache/climate/pull/397


> Enable restructured text markdown for Pypi uploads
> --
>
> Key: CLIMATE-859
> URL: https://issues.apache.org/jira/browse/CLIMATE-859
> Project: Apache Open Climate Workbench
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 1.1.0
>Reporter: Lewis John McGibbney
>Assignee: Lewis John McGibbney
>Priority: Trivial
> Fix For: 1.2.0
>
>
> Right now the Pypi package page for OCW is a mess.
> I'll upload a simple PR to sort this out.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Resolved] (CLIMATE-859) Enable restructured text markdown for Pypi uploads

[Lewis John McGibbney (JIRA)]

 [ 
https://issues.apache.org/jira/browse/CLIMATE-859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lewis John McGibbney resolved CLIMATE-859.
--
Resolution: Fixed

> Enable restructured text markdown for Pypi uploads
> --
>
> Key: CLIMATE-859
> URL: https://issues.apache.org/jira/browse/CLIMATE-859
> Project: Apache Open Climate Workbench
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 1.1.0
>Reporter: Lewis John McGibbney
>Assignee: Lewis John McGibbney
>Priority: Trivial
> Fix For: 1.2.0
>
>
> Right now the Pypi package page for OCW is a mess.
> I'll upload a simple PR to sort this out.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Updated] (CLIMATE-856) Removing potential bugs from dataset_processor.spatial_regrid

[Lewis John McGibbney (JIRA)]

 [ 
https://issues.apache.org/jira/browse/CLIMATE-856?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lewis John McGibbney updated CLIMATE-856:
-
Fix Version/s: 1.2.0

> Removing potential bugs from dataset_processor.spatial_regrid
> -
>
> Key: CLIMATE-856
> URL: https://issues.apache.org/jira/browse/CLIMATE-856
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Affects Versions: 1.1.0
>Reporter: Huikyo Lee
>Assignee: Huikyo Lee
> Fix For: 1.2.0
>
>
> The spatial_regrid will be updated to remove the following warning messages.
> - using a boolean instead of an integer will result in an error in the future
> - non integer (and non boolean) array-likes will not be accepted as indices 
> in the future



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] climate pull request #397: CLIMATE-859 Enable restructured text markdown for...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/climate/pull/397


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

[jira] [Commented] (CLIMATE-859) Enable restructured text markdown for Pypi uploads

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428773#comment-15428773
 ] 

ASF GitHub Bot commented on CLIMATE-859:


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 
Date:   2016-08-19T20:43:43Z

CLIMATE-859 Enable restructured text markdown for Pypi uploads




> Enable restructured text markdown for Pypi uploads
> --
>
> Key: CLIMATE-859
> URL: https://issues.apache.org/jira/browse/CLIMATE-859
> Project: Apache Open Climate Workbench
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 1.1.0
>Reporter: Lewis John McGibbney
>Assignee: Lewis John McGibbney
>Priority: Trivial
> Fix For: 1.2.0
>
>
> Right now the Pypi package page for OCW is a mess.
> I'll upload a simple PR to sort this out.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Resolved] (CLIMATE-858) Fix Travis Build

[Lewis John McGibbney (JIRA)]

 [ 
https://issues.apache.org/jira/browse/CLIMATE-858?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lewis John McGibbney resolved CLIMATE-858.
--
Resolution: Fixed

Nice [~jarifibrahim]

> Fix Travis Build
> 
>
> Key: CLIMATE-858
> URL: https://issues.apache.org/jira/browse/CLIMATE-858
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Reporter: Ibrahim Jarif
>Assignee: Ibrahim Jarif
>Priority: Critical
> Fix For: 1.2.0
>
>
> The travis build is broken due to lack of display variable on travis. The 
> travis.yml file needs to be updated to resolve this issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Updated] (CLIMATE-855) Fix test_local and test_dataset_processor

[Lewis John McGibbney (JIRA)]

 [ 
https://issues.apache.org/jira/browse/CLIMATE-855?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lewis John McGibbney updated CLIMATE-855:
-
Affects Version/s: 1.1.0

> Fix test_local and test_dataset_processor
> -
>
> Key: CLIMATE-855
> URL: https://issues.apache.org/jira/browse/CLIMATE-855
> Project: Apache Open Climate Workbench
>  Issue Type: Bug
>Affects Versions: 1.1.0
>Reporter: Huikyo Lee
>Assignee: Huikyo Lee
>Priority: Critical
> Fix For: 1.2.0
>
>
> The recent updates in ocw.data_source.local and ocw.dataset_processor cause 
> errors in nosetests. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (CLIMATE-859) Enable restructured text markdown for Pypi uploads

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/CLIMATE-859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15428779#comment-15428779
 ] 

ASF GitHub Bot commented on CLIMATE-859:


Github user coveralls commented on the issue:

https://github.com/apache/climate/pull/397
  

[![Coverage 
Status](https://coveralls.io/builds/7529958/badge)](https://coveralls.io/builds/7529958)

Coverage remained the same at 80.271% when pulling 
**818784edebcbaedd4786a27b08b1f3af5498c140 on lewismc:CLIMATE-859** into 
**653bec552b3bdd5eeb15fcbabaa94353d595678a on apache:master**.



> Enable restructured text markdown for Pypi uploads
> --
>
> Key: CLIMATE-859
> URL: https://issues.apache.org/jira/browse/CLIMATE-859
> Project: Apache Open Climate Workbench
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 1.1.0
>Reporter: Lewis John McGibbney
>Assignee: Lewis John McGibbney
>Priority: Trivial
> Fix For: 1.2.0
>
>
> Right now the Pypi package page for OCW is a mess.
> I'll upload a simple PR to sort this out.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)