search

VizierDB / web-api-async

Vizier Asynchronous Web API
Apache License 2.0
2 stars 2 forks source link

Allow tasks to update their arguments #92

Closed okennedy closed 4 years ago

okennedy commented 4 years ago

The main goal of this PR is to allow tasks to update their arguments. This is not intended to be used in general, but some cells have the ability to "infer" or "guess" default arguments from context. Through this PR, the cell can fill in these arguments with the ones it picked,

  1. ... allowing the user to override those arguments more easily
  2. ... freezing the arguments in place for posterity, in case the "guessing" algorithm changes

As a secondary goal, this PR also does a bunch of cleanup. flake8 is added as tools/lint, and as of right now the system passes with defaults provided in the .flake8 config file. All unit tests pass, and tools/typecheck has gotten a little more pedantic.

okennedy commented 4 years ago

progress towards https://github.com/VizierDB/web-ui/issues/55