<div dir="ltr"><div dir="ltr"><div>Hi,</div><div>Done. [1]</div><div>A.<br></div><div><br></div><div>[1]: <a href="https://github.com/geopython/pywps/pull/432">https://github.com/geopython/pywps/pull/432</a><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">pá 14. 12. 2018 v 21:29 odesílatel Carsten Ehbrecht <<a href="mailto:ehbrecht@dkrz.de">ehbrecht@dkrz.de</a>> napsal:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Adam,<br>
<br>
If you have an improved version of the bugfix could please open a PR?<br>
<br>
Here is the one from David:<br>
<br>
<a href="https://github.com/geopython/pywps/pull/431" rel="noreferrer" target="_blank">https://github.com/geopython/pywps/pull/431</a><br>
<br>
Cheers,<br>
Carsten<br>
<br>
> On 14 Dec 2018, at 09:10, Adam Laža <<a href="mailto:ad.laza32@gmail.com" target="_blank">ad.laza32@gmail.com</a>> wrote:<br>
> <br>
> Hi David,<br>
> <br>
> Just cosmetics, I slightly edited the template. CDATA tag is not present if we know that mimeType is XML-based. Response XML looks better then.<br>
> <br>
> A.<br>
> <br>
> [1]: <a href="https://github.com/lazaa32/pywps/commit/f8b66e3f43f027ec3f20eacf9a82f9ac9c0e8499" rel="noreferrer" target="_blank">https://github.com/lazaa32/pywps/commit/f8b66e3f43f027ec3f20eacf9a82f9ac9c0e8499</a><br>
> <br>
> čt 13. 12. 2018 v 8:00 odesílatel David Huard <<a href="mailto:huard.david@ouranos.ca" target="_blank">huard.david@ouranos.ca</a>> napsal:<br>
> Hi Adam, I've created a PR for your bug fix. Does it work for you ? It seems to work on my use cases. <br>
> <br>
> On Wed, Dec 12, 2018 at 9:31 AM David Huard <<a href="mailto:huard.david@ouranos.ca" target="_blank">huard.david@ouranos.ca</a>> wrote:<br>
> Looks good, trying to write a few tests to exercise it. <br>
> Good catch !<br>
> <br>
> On Wed, Dec 12, 2018 at 1:02 AM Adam Laža <<a href="mailto:ad.laza32@gmail.com" target="_blank">ad.laza32@gmail.com</a>> wrote:<br>
> Hi David,<br>
> <br>
> check my bug fix [1]. It works for me but I would like to know if it is right approach.<br>
> <br>
> Cheers,<br>
> Adam<br>
> <br>
> [1]: <a href="https://github.com/lazaa32/pywps/commit/5cccc38c997965d4566b21df0badf3244021bdeb" rel="noreferrer" target="_blank">https://github.com/lazaa32/pywps/commit/5cccc38c997965d4566b21df0badf3244021bdeb</a><br>
> <br>
> st 12. 12. 2018 v 2:32 odesílatel David Huard <<a href="mailto:huard.david@ouranos.ca" target="_blank">huard.david@ouranos.ca</a>> napsal:<br>
> Hi Adam, <br>
> <br>
> I think we have the same issue : <a href="https://github.com/geopython/pywps/issues/412" rel="noreferrer" target="_blank">https://github.com/geopython/pywps/issues/412</a> <br>
> <br>
> My guess is that it's either due to my refactoring or to the new templates or a combination of both. <br>
> <br>
> I noticed that if you do <br>
> ComplexOutput.data = <file content><br>
> ... then it works<br>
> but if you do <br>
> ComplexOutput.file =  <file path> <br>
> it doesn't. Hopefully this is a hint as to what is going wrong.  <br>
> <br>
> I think the offending bit is is inout.outputs.ComplexOutput. If the ComplexOutput is a file and as_reference is False, _json_data is called which I think might fail to fill the json data field. It's not obvious why though. Do you have a test case you could add to the test suite ?<br>
> <br>
> HTH, <br>
> <br>
> David<br>
>  <br>
> <br>
> <br>
> <br>
> On Tue, Dec 11, 2018 at 4:06 AM Adam Laža <<a href="mailto:ad.laza32@gmail.com" target="_blank">ad.laza32@gmail.com</a>> wrote:<br>
> Hi Carsten,<br>
> <br>
> thanks for reply. Good to know. However since I switched for master branch I can get output in ExecuteResponse only as reference. Even if I force attribute asReference=false I still get only reference to output, not the output directly.<br>
> I use pywps-flask demo server and Buffer process for testing. Any idea what could be wrong?<br>
> <br>
> Cheers,<br>
> Adam<br>
> <br>
> út 11. 12. 2018 v 0:16 odesílatel Carsten Ehbrecht <<a href="mailto:ehbrecht@dkrz.de" target="_blank">ehbrecht@dkrz.de</a>> napsal:<br>
> Hi Adam,<br>
> <br>
> yes, please use master branch to rebase. We switched awhile ago (Code sprint Bonn in March?).<br>
> <br>
> Cheers,<br>
> Carsten<br>
> <br>
> > On 8 Dec 2018, at 04:16, Adam Laža <<a href="mailto:ad.laza32@gmail.com" target="_blank">ad.laza32@gmail.com</a>> wrote:<br>
> > <br>
> > Hi devs,<br>
> > <br>
> > I'd like to make a new PR for my Docker extension in few days. Now I'm just bit confused because I wanted to rebase my branch onto develop branch to avoid conflicts but it seems to me that develop branch is no more used and that you use master instead (develop has last commit July 16, master 26 days ago).<br>
> > <br>
> > Could anyone confirm that right approach is to rebase onto master and make a PR against master?<br>
> > <br>
> > Thanks,<br>
> > Adam<br>
> > _______________________________________________<br>
> > pywps-dev mailing list<br>
> > <a href="mailto:pywps-dev@lists.osgeo.org" target="_blank">pywps-dev@lists.osgeo.org</a><br>
> > <a href="https://lists.osgeo.org/mailman/listinfo/pywps-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/pywps-dev</a><br>
> <br>
> _______________________________________________<br>
> pywps-dev mailing list<br>
> <a href="mailto:pywps-dev@lists.osgeo.org" target="_blank">pywps-dev@lists.osgeo.org</a><br>
> <a href="https://lists.osgeo.org/mailman/listinfo/pywps-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/pywps-dev</a><br>
> _______________________________________________<br>
> pywps-dev mailing list<br>
> <a href="mailto:pywps-dev@lists.osgeo.org" target="_blank">pywps-dev@lists.osgeo.org</a><br>
> <a href="https://lists.osgeo.org/mailman/listinfo/pywps-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/pywps-dev</a><br>
> _______________________________________________<br>
> pywps-dev mailing list<br>
> <a href="mailto:pywps-dev@lists.osgeo.org" target="_blank">pywps-dev@lists.osgeo.org</a><br>
> <a href="https://lists.osgeo.org/mailman/listinfo/pywps-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/pywps-dev</a><br>
<br>
</blockquote></div>