[Pywps-dev] PyWPS testing

[Jonas Eberle]

Hi Jachym, 

thanks. I did not have time the last two days to create my own pull request. If you push your changes to master, I will try again the installation and the process execution on the OSGeo Live 10. I will arrive in Bonn/Basecamp later today at around 6. 

Cheers, 
Jonas 

----- Am 21. Aug 2016 um 14:13 schrieb Jachym Cepicky <jachym.cepicky at gmail.com>: 

> Hi Jonas,
> I've implemented your fixes to mine branch [1]

> should be part of the latest pull request [2]

> [1]
> https://github.com/jachym/PyWPS/commit/292231257e234ba3e6b9b17a6e0ae47d135ac6e6
> [2] https://github.com/geopython/pywps/pull/155

> J

> pá 19. 8. 2016 v 9:11 odesílatel Jachym Cepicky < jachym.cepicky at gmail.com >
> napsal:

>> Jonas,
>> big thanks - some of those issues, I've fixed somewhere else, those slipped
>> through my eyes or I did it wrong

>> can you please make pull request and merge?

>> J

>> čt 18. 8. 2016 v 21:46 odesílatel Jonas Eberle < jonas.eberle at gmx.de > napsal:

>>> Hi all,

>>> I discovered some issues when testing the current master repository:

>>> 1) Python error (no such file or directory because working directory has been
>>> cleaned up already before) when storing output data as reference with
>>> synchronous execution. As a solution I added the variable "async" to Process
>>> and extended the if clause in line 244 (app/Process.py):
>>> https://github.com/jonas-eberle/pywps/commit/51338260674b7271246cc617ff2c37482ec86080#diff-bddcbbd3f8d8f4182a33f03a1f9a847dR245

>>> 2) Output url in inout/storage.py is not a combination of "url" and "outputurl",
>>> only "outputurl" from configuration is used:
>>> https://github.com/jonas-eberle/pywps/commit/51338260674b7271246cc617ff2c37482ec86080#diff-12acb511d79bad7767468cd036926743R76

>>> 3) The calculation of available space on disk was wrong, for Linux systems the
>>> method "get_free_space" delivers the number of free blocks (inout/storage.py)
>>> https://github.com/jonas-eberle/pywps/commit/51338260674b7271246cc617ff2c37482ec86080#diff-12acb511d79bad7767468cd036926743R87

>>> 4) The output format class has no method "get_extension" (inout/storage.py),
>>> thus I used just the extension attribute:
>>> https://github.com/jonas-eberle/pywps/commit/51338260674b7271246cc617ff2c37482ec86080#diff-12acb511d79bad7767468cd036926743R98

>>> Best regards,
>>> Jonas
>>> _______________________________________________
>>> pywps-dev mailing list
>>> pywps-dev at lists.osgeo.org
>>> http://lists.osgeo.org/mailman/listinfo/pywps-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/pywps-dev/attachments/20160821/838642cc/attachment.html>