Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/420
The travis build is failing due to some random issue. See this build
https://travis-ci.org/jarifibrahim/climate/builds/174863520
---
If your project is set up for it, you can reply to this
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/418
@Omkar20895 `npm install` would create a folder `node_modules` and install
all the dependencies in it. If you do `npm install -g` then the dependencies
are installed globally.
I tried
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/419
@agoodm Done via
https://github.com/apache/climate/commit/6156eff08b11f92acbfee0ca5be5d78e473483c6
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/420
@lewismc 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
enabled and wishes so
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/419
@agoodm @lewismc I've added some comments to the exception handling. Should
I merge it now?
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/419
Right. That would happen. Maybe we could temporarily skip these tests.
Once opendap servers are working we could add these tests back.
---
If your project is set up for it, you can reply
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/419
The bug/exceptions are not in ocw but in its dependencies. The downloading
of the dataset is handled by the respective packages. So if something goes
wrong while downloading the datasets that
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/420
Climate 863
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-863
Alternatively you can review and apply
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/419
CLIMATE-889 - Fix Travis Build
- Skip test if unable to download the dataset
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim
Github user jarifibrahim commented on a diff in the pull request:
https://github.com/apache/climate/pull/383#discussion_r75575893
--- Diff: ocw/data_source/podaac.py ---
@@ -0,0 +1,111 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
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
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
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
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/389
Climate-851 - Have nosetests log to STDOUT
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-851
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/388
CLIMATE-844 - Attribute error in test_smoke.py
This PR fixes the Attribute error in test_smoke.py file. The tests are
still failing because of broken test cases.
You can merge this pull
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/commit/02dfdec65adb81cdc659764008045e99f61bbf4d#commitcomment-18626164
In ocw/dataset.py:
In ocw/dataset.py on line 314:
@huikyole Can you please take a look at these tests[0] and fix
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/commit/02dfdec65adb81cdc659764008045e99f61bbf4d#commitcomment-18620602
In ocw/dataset.py:
In ocw/dataset.py on line 314:
@huikyole Shouldn't we check this in some kind of setter m
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/376
@Omkar20895 This merge breaks the test instead of fixing it. We might have
to undo the changes made by this PR to get the tests working again.
---
If your project is set up for it, you can
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/383
@lewismc The exception can be fixed by renaming `time_range` to
`temporal_boundaries`.
But there are many more exceptions in the unit tests. A lot of new code has
been added lately but the
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/376
@Omkar20895 @huikyole This branch wasn't supposed to be merged, right? It
breaks the dap tests
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/374
@agoodm I'm not sure why the build is stuck. I tried starting a new build
but it also got stuck at `waiting`. The issue seems to be with landscape.
Please go ahead and merge this PR.
-
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/378
Climate-833 - Update travis.yml to use smoke tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-833
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/377
Climate-832 - Add smoke tests
- Update requirements
- Add smoke tests
Sample output of smoke tests
```
Checking installed dependencies
numpy>=1.10.4 â
sc
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/374
@agoodm can you please fix landscape.io issues? The build failed because
there were multiple PEP8 violations. Please take a look at this ->
[Link](https://landscape.io/diff/413629)
---
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/375
Climate-486 - Normalize DSP function parameter ordering
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-486
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/373
CLIMATE-488 - Change Dataset.time_range to Dataset.temporal_boundaries
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/367
CLIMATE-820 - Update pip requirements
- Remove unused requirements (nose-exclude, pylint)
- Update all packages to latest version
You can merge this pull request into a Git repository by
Github user jarifibrahim commented on a diff in the pull request:
https://github.com/apache/climate/pull/360#discussion_r68692972
--- Diff: .landscape.yml ---
@@ -0,0 +1,18 @@
+python-targets:
--- End diff --
Done.
---
If your project is set up for it, you can
Github user jarifibrahim closed the pull request at:
https://github.com/apache/climate/pull/358
---
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 jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/358
@huikyole Thank you so much. Closing PR because of **Invalid Issue**.
---
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 jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/358
@huikyole The link to the image in your comment is broken. Please fix 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 jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/358
@huikyole 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
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/364
CLIMATE-816 - Add requires.io badge to README.me
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-816
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/363
CLIMATE-814 - Fix PEP8 violations in dataset
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-814
Github user jarifibrahim commented on a diff in the pull request:
https://github.com/apache/climate/pull/363#discussion_r67599325
--- Diff: ocw/dataset.py ---
@@ -28,10 +28,9 @@
import ocw.utils as utils
-from mpl_toolkits.basemap import shiftgrid
--- End
Github user jarifibrahim commented on a diff in the pull request:
https://github.com/apache/climate/pull/362#discussion_r67599123
--- Diff: ocw/utils.py ---
@@ -326,97 +339,107 @@ def calc_climatology_monthly(dataset):
raise ValueError(error)
else
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/362
CLIMATE-813 - Fix PEP8 violations in utils module
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-813
Github user jarifibrahim commented on a diff in the pull request:
https://github.com/apache/climate/pull/361#discussion_r67598748
--- Diff: ocw/dataset_processor.py ---
@@ -202,130 +214,158 @@ def spatial_regrid(target_dataset, new_latitudes,
new_longitudes, boundary_check
Github user jarifibrahim commented on a diff in the pull request:
https://github.com/apache/climate/pull/361#discussion_r67598737
--- Diff: ocw/dataset_processor.py ---
@@ -51,107 +53,114 @@ def temporal_subset(month_start, month_end,
target_dataset, average_each_year=Fa
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/361
CLIMATE-812 - Fix PEP8 violations in dataset processor
- Fix PEP8 violations
- Remove unused variables (also a PEP8 violation)
You can merge this pull request into a Git repository by
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/360
CLIMATE-811 - Add landscape.io integration
- Add .landscape.yml file
- Remove .pylintrc file
- Add landscape.io badge
You can merge this pull request into a Git repository by running
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/358
Climate-798 - Redundant code in evaluation.py
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-798
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/357
CLIMATE-809 - Fix coveragerc file
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-809
Alternatively you can
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/355
CLIMATE-808 - Add tests for plotter module
- Add tests to verify computations in plotter module
- Update .coveragerc file to ignore image generation code
You can merge this pull request
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/354
CLIMATE-807 - Add test coverage badge to README
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-807
Github user jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/348
@lewismc conflict resolved.
---
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 jarifibrahim commented on the issue:
https://github.com/apache/climate/pull/347
@lewismc Yes. I'll update the JIRA ticket to explain the issue. 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
Github user jarifibrahim closed the pull request at:
https://github.com/apache/climate/pull/341
---
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 jarifibrahim commented on a diff in the pull request:
https://github.com/apache/climate/pull/341#discussion_r66836127
--- Diff: ocw/dataset_processor.py ---
@@ -68,8 +68,6 @@ def temporal_subset(month_start, month_end,
target_dataset, average_each_year=Fa
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/353
Climate-806 - Add tests for local module
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-806
Alternatively
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/352
Climate-805 - Add tests for Utils module
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-805
Alternatively
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/351
CLIMATE-804 - normalize_lat_lon_values does not work
- Fix trivial bug in utils module
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/350
Climate-801
Add new tests for dataset processor module.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-795
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/349
Climate-801
Add new tests for dataset module.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-801
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/348
Climate 802
Add new tests for evaluation module.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-802
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/347
CLIMATE-800 - TypeError in _rcmes_calc_average_on_new_time_unit
- Fix TypeError in dataset processor
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/346
CLIMATE-799 - TypeError in subset function
- Fix TypeError in subset function
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/345
CLIMATE-796 - TypeError in temporal_slice function
Fix TypeError in temporal_slice
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/343#issuecomment-222323332
* conda_environment.txt file is useless.
* install-conda.sh can be merged to install-ubuntu.sh to improve the
process of installation. The user has to run only
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/344#issuecomment-222318186
The builds failed because the travis is using the old version of the
install-ubuntu.sh script. Please merge the CLIMATE-793 to fix the build.
Please check
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/343#issuecomment-222318099
The Jenkins build is failing due to some permission issues with the conda
installation.
The script works fine on a bare machine.
Please check this
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/344
CLIMATE-794 - Update travis.yml
- Use easy-ocw script for build
- Track coverage
- Upload code coverage report to coveralls.io
You can merge this pull request into a Git repository by
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/343
CLIMATE-793 - Update install-ubuntu.sh script
- Merge install-ubuntu.sh and install-conda.sh
- Delete install-conda.sh
- Delete conda_environment.txt
You can merge this pull request
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/337#issuecomment-221347963
The commit has been merged. Hence closing #337
---
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 jarifibrahim closed the pull request at:
https://github.com/apache/climate/pull/337
---
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 jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/334#issuecomment-219536362
[INFRA-11783 Add Github integration to
jenkins](https://issues.apache.org/jira/browse/INFRA-11783)
---
If your project is set up for it, you can reply to this
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/334#issuecomment-219529546
@lewismc The builds for the PR's aren't working right now. The code is
built only after it is merged to master branch. I've raised a JIRA ticket an
Github user jarifibrahim closed the pull request at:
https://github.com/apache/climate/pull/335
---
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 jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/335#issuecomment-219528399
Yeah.
---
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 jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/342#issuecomment-219524436
ok to test
---
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 jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/342
Fix PEP8 violations
Following PEP8 violations are fixed
- Remove trailing whitespaces
- Fix lines with length > 79
- Add two blank lines after every function
You can me
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/341
Fix temporal_subset() function
- Add tests for temporal_subset()
Fixes https://issues.apache.org/jira/browse/CLIMATE-791
You can merge this pull request into a Git repository by
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/337
Remove redundant code in temporal_resolution function
Fixes https://issues.apache.org/jira/browse/CLIMATE-789
You can merge this pull request into a Git repository by running:
$ git pull
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/336
CLIMATE-786 - Update rcmed.py and test_rcmed.py
- Update rcmed.py file
- Fix test_rcmed.py file
Function `test_function_parameter_dataset_lats_monthly` is added to improve
test
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/335#issuecomment-215851989
Please do no close the PR. I'm waiting for OCWJenkins to comment on this PR
with the test results.
---
If your project is set up for it, you can reply to
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/335
test 1
Jenkins test branch. Do Not Merge this branch.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate test
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/334
CLIMATE-785 - RCMED.py: optimize code
Fixes https://issues.apache.org/jira/browse/CLIMATE-785
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/329
CLIMATE-780 - Add Travis-CI build status to README.md
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-780
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/328#issuecomment-209411521
@lewismc I'm not getting any such errors. I'm using sphinx 1.3.6
```
docs[CLIMATE-760] % make html
sphinx-build -b html -d build/doctrees so
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/328#issuecomment-208852444
@lewismc I was unable to reproduce all the errors in the Jira issue. I just
pulled from the master branch and fixed all the issues that the documentation
had
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/318#issuecomment-207817574
Yes Lewis. Exciting days ahead :dancers:
---
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 jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/328
CLIMATE-760 - Address documentation warnings
- Fix configuration_writer import error
- Update table of contents tree
You can merge this pull request into a Git repository by running
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/318#issuecomment-207815116
Here's the latest
[build](https://travis-ci.org/jarifibrahim/climate/builds/121927729).
Btw the Travis-CI does not use the install-ubuntu.sh scri
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/318#issuecomment-207812290
@Omkar20895 All dependency conflicts have been resolved.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/318#issuecomment-207066640
@lewismc No. This issue has not been merged.
---
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 jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/324#issuecomment-207040203
Thank you @lewismc @Omkar20895.
Please take a look at the PR for
[ubuntu-install.sh](https://github.com/apache/climate/pull/318) and
[utils_tests](https
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/324#issuecomment-206982581
@Omkar20895 @lewismc Please merge this branch ASAP. The travis-ci build is
failing because of conflicting dependencies.
---
If your project is set up for it, you
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/324#issuecomment-206457535
@Omkar20895 Basemap does not work with numpy 1.11.0. It requires numpy
1.10* . I'll update h5py to 2.6.0.
---
If your project is set up for it, you can rep
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/324
CLIMATE-776 - Update conda dependencies
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-776
Alternatively
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/323#issuecomment-203018094
The tests failed on executing `nosetests` from the root climate folder. The
tests had some improper imports.
---
If your project is set up for it, you can reply
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/322#issuecomment-203016653
The tests failed when they were executed from the root folder of climate
project. Try executing `nosetests` from the root climate folder to reproduce
these errors
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/318#issuecomment-203015997
@lewismc The script works fine. But there's a dependency issue. Is there a
wiki page that lists the dependencies ? The existing `ocw-conda-dependency`
fil
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/321#issuecomment-202998033
@lewismc Yes. The Travis build fails because of the bad tests but Travis
works as expected. I've created a Travis-test branch which works fine after
fixin
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/323
CLIMATE-775 - Fix failing tests in ocw_config_runner
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-775
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/322
CLIMATE-774 - Fix failing tests
- Add 'mock' to pip-dependencies
- Fix file path in test_utils.py
You can merge this pull request into a Git repository by running:
$ git
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/321
add Travis-CI config
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jarifibrahim/climate CLIMATE-773
Alternatively you can review and
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/318#issuecomment-192293306
@MichaelArthurAnderson @lewismc Could you please test this script?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/318
CLIMATE-766 Easy-ocw/install-ubuntu.sh script is broken
Can anyone please test this script and post the results.
I have not tested it. I didn't want to mess with my installation.
Yo
GitHub user jarifibrahim opened a pull request:
https://github.com/apache/climate/pull/315
CLIMATE-759 - Fix failing test in test_dataset_processor.py
- fix tests in TestTemporalRebin class
`TemporalRebin()` function passes the received parameters to
Github user jarifibrahim commented on the pull request:
https://github.com/apache/climate/pull/312#issuecomment-189380356
Yeah. I had fixed the issue but the tests were still returning false. I'll
fix the tests and then push a PR.
---
If your project is set up for it, you can
1 - 100 of 102 matches
Mail list logo