Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

native client: how to return multiple values? #64

Closed
cehbrecht opened this issue Nov 23, 2018 · 5 comments · Fixed by #66, #74 or #75
Closed

native client: how to return multiple values? #64

cehbrecht opened this issue Nov 23, 2018 · 5 comments · Fixed by #66, #74 or #75

Comments

@cehbrecht
Copy link
Member

Description

In the current native client implementation we either return a single value or a list of values. I would find it more descriptive to return a dictionary with:

key=parameter identifier
value=parameter value

@huard @davidcaron
What do you think?

Here a link from a quick google search:
https://stackoverflow.com/questions/354883/how-do-you-return-multiple-values-in-python

Environment

Steps to Reproduce

Additional Information

@huard
Copy link
Contributor

huard commented Nov 23, 2018

I'm not a fan of the dictionary solution. What about the NamedTuple suggested on stack ?

@tomLandry
Copy link

There is a somewhat similar discussion at OGC on how to manage multiple outputs for WPS. It's documented as a recommendation in an Engineering Report... I'll identify later. Please tell me if you want investigation at OGC-standard level.

@huard
Copy link
Contributor

huard commented Nov 23, 2018

Yes, I think it would be useful to develop a solution that will work with the OGC standard and slowly move birdhouse processes to that standard.

@tomLandry
Copy link

Ok, sending the ER to both Carsten and you.

cehbrecht added a commit that referenced this issue Nov 23, 2018
* renamed BirdyClient to WPSClient (#64)

* update import
@cehbrecht
Copy link
Member Author

accidentally closed due wrong reference.

@cehbrecht cehbrecht reopened this Nov 26, 2018
@huard huard mentioned this issue Nov 27, 2018
cehbrecht added a commit that referenced this issue Nov 28, 2018
huard added a commit that referenced this issue Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants