Skip to content
This repository has been archived by the owner on Oct 18, 2022. It is now read-only.

Save Parameters button does not produce a viable datastack #46

Closed
davemfish opened this issue Sep 29, 2020 · 1 comment · Fixed by #50
Closed

Save Parameters button does not produce a viable datastack #46

davemfish opened this issue Sep 29, 2020 · 1 comment · Fixed by #50
Assignees
@davemfish
Labels
bug Something isn't working in progress This issue is being actively worked on
Milestone
0.1.0-alpha

Comments

@davemfish
Copy link
Collaborator

The args property of the resulting json should only have the key:value pairs for the invest model's execute function. But right now args has all the "spec" data in it as well.
@davemfish davemfish added the bug Something isn't working label Sep 29, 2020
@davemfish davemfish added this to the 0.1.0-alpha milestone Sep 29, 2020
@davemfish davemfish self-assigned this Sep 29, 2020
davemfish added a commit to davemfish/invest-workbench that referenced this issue Sep 30, 2020
@davemfish
fix natcap#46 and do more explicit handling of 'undefined' invest arg…
1588494 
… values during SetupTab initialization.
davemfish added a commit to davemfish/invest-workbench that referenced this issue Sep 30, 2020
@davemfish
related to natcap#46. Better handling of which args are exported in a…
bf9a2ed 
…rgs dicts and datastacks - specifically hidden args and n_workers.
@davemfish davemfish added the in progress This issue is being actively worked on label Oct 1, 2020
@davemfish
Copy link
Collaborator Author

🔢 testing slack integration with a comment 🔢

@davemfish davemfish mentioned this issue Oct 8, 2020
Merged
@davemfish davemfish linked a pull request Oct 8, 2020 that will close this issue
Terminating invest subprocesses | Automated browser tests #50
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@davemfish davemfish

Labels
bug Something isn't working in progress This issue is being actively worked on
Projects
None yet
Milestone
0.1.0-alpha
Development

Successfully merging a pull request may close this issue.

Terminating invest subprocesses | Automated browser tests davemfish/invest-workbench
1 participant
@davemfish