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

Passing arguments to activities as dictionaries instead of as list of lists. #42

Open
GabrieleMeoni opened this issue Nov 15, 2022 · 0 comments
Labels
enhancement New feature or request user-facing Anything that users can interact with

Comments

@GabrieleMeoni
Copy link
Collaborator

GabrieleMeoni commented Nov 15, 2022

Feature

Desired Behavior / Functionality

At the moment, when you register an activity in the Sentinel 2 notebook or in the activity tests, if you want to return an argument or modify data, you need to pass it as input as a list or a list of lists. It would be nice to edit the code to take as input dictionaries, making the code much more readable and understandable.

Furthermore, it would be nice to have a dedicated mechanism to access values return from activities.

What Needs to Be Done

  • Change args for activities to be kwargs
  • Process return values of activity functions in some way
  • Update Sentinel 2 example notebook from Sentinel2 examples #38 to use this

How Can It Be Tested

The outputs should be the same as now.

@GabrieleMeoni GabrieleMeoni added enhancement New feature or request user-facing Anything that users can interact with labels Nov 15, 2022
@gomezzz gomezzz mentioned this issue Nov 17, 2022
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request user-facing Anything that users can interact with
Projects
None yet
Development

No branches or pull requests

1 participant