Diffusion dagster 5e290d6dd62d

(more-config-work-5) Allow bare dict in lieu of Field(Dict(...))
5e290d6dd62d
Actions

Description

(more-config-work-5) Allow bare dict in lieu of Field(Dict(...))

Summary:
Proposed change to implement #1908

Wanted to dicuss before doing work of adding full test coverage

https://github.com/dagster-io/dagster/issues/1908

Depends on D1622

Test Plan: BK

Reviewers: alangenfeld, max, sashank, themissinghlink

Reviewed By: alangenfeld

Differential Revision: https://dagster.phacility.com/D1614

Details

Provenance

schrockn

Authored on Dec 9 2019, 10:57 PM

Reviewer

alangenfeld

Differential Revision

D1614: (more-config-work-5) Allow bare dict in lieu of Field(Dict(...))

Parents

R1:879c5d00af8b: (more-config-work-4) Rename Dict Api object to DagsterDictApi

Branches

Unknown

Tags

Unknown

Event Timeline

schrockn committed R1:5e290d6dd62d: (more-config-work-5) Allow bare dict in lieu of Field(Dict(...)) (authored by schrockn).Dec 10 2019, 10:16 PM

schrockn added an edge: D1614: (more-config-work-5) Allow bare dict in lieu of Field(Dict(...)).

Changes (17)

Path

Size

examples/

dagster_examples/

airline_demo/

solids.py

toys/

error_monster.py

python_modules/

dagster-graphql/

dagster_graphql_tests/

graphql/

setup.py

dagster/

core/

definitions/

config.py

environment_configs.py

types/

config/

field_utils.py

dagster_tests/

cli_tests/

test_config_scaffolder.py

core_tests/

config_types_tests/

evaluator_tests/

test_evaluator.py

test_config_type_system.py

test_invalid_config_arguments.py

test_type_printer.py

definitions_tests/

test_definition_errors.py

test_preset_definition.py

system_config_tests/

test_system_config.py

py3_tests/

test_type_examples_py3.py

libraries/

dagster-gcp/

dagster_gcp/

bigquery/

configs.py

dagster-pandas/

dagster_pandas/

data_frame.py

R1:5e290d6dd62d

View Options

examples/dagster_examples/airline_demo/solids.py

View Options

examples/dagster_examples/toys/error_monster.py

View Options

python_modules/dagster-graphql/dagster_graphql_tests/graphql/setup.py

View Options

python_modules/dagster/dagster/core/definitions/config.py

View Options

python_modules/dagster/dagster/core/definitions/environment_configs.py

View Options

python_modules/dagster/dagster/core/types/config/field_utils.py

View Options

python_modules/dagster/dagster_tests/cli_tests/test_config_scaffolder.py

View Options

python_modules/dagster/dagster_tests/core_tests/config_types_tests/evaluator_tests/test_evaluator.py

View Options

python_modules/dagster/dagster_tests/core_tests/config_types_tests/test_config_type_system.py

View Options

python_modules/dagster/dagster_tests/core_tests/config_types_tests/test_invalid_config_arguments.py

View Options

python_modules/dagster/dagster_tests/core_tests/config_types_tests/test_type_printer.py

View Options

python_modules/dagster/dagster_tests/core_tests/definitions_tests/test_definition_errors.py

View Options

python_modules/dagster/dagster_tests/core_tests/definitions_tests/test_preset_definition.py

View Options

python_modules/dagster/dagster_tests/core_tests/system_config_tests/test_system_config.py

View Options

python_modules/dagster/dagster_tests/py3_tests/test_type_examples_py3.py

View Options

python_modules/libraries/dagster-gcp/dagster_gcp/bigquery/configs.py

View Options

python_modules/libraries/dagster-pandas/dagster_pandas/data_frame.py

(more-config-work-5) Allow bare dict in lieu of Field(Dict(...))5e290d6dd62dActions

Description

Details

Event Timeline

Changes (17)

R1:5e290d6dd62d

examples/dagster_examples/airline_demo/solids.py

examples/dagster_examples/toys/error_monster.py

python_modules/dagster-graphql/dagster_graphql_tests/graphql/setup.py

python_modules/dagster/dagster/core/definitions/config.py

python_modules/dagster/dagster/core/definitions/environment_configs.py

python_modules/dagster/dagster/core/types/config/field_utils.py

python_modules/dagster/dagster_tests/cli_tests/test_config_scaffolder.py

python_modules/dagster/dagster_tests/core_tests/config_types_tests/evaluator_tests/test_evaluator.py

python_modules/dagster/dagster_tests/core_tests/config_types_tests/test_config_type_system.py

python_modules/dagster/dagster_tests/core_tests/config_types_tests/test_invalid_config_arguments.py

python_modules/dagster/dagster_tests/core_tests/config_types_tests/test_type_printer.py

python_modules/dagster/dagster_tests/core_tests/definitions_tests/test_definition_errors.py

python_modules/dagster/dagster_tests/core_tests/definitions_tests/test_preset_definition.py

python_modules/dagster/dagster_tests/core_tests/system_config_tests/test_system_config.py

python_modules/dagster/dagster_tests/py3_tests/test_type_examples_py3.py

python_modules/libraries/dagster-gcp/dagster_gcp/bigquery/configs.py

python_modules/libraries/dagster-pandas/dagster_pandas/data_frame.py

(more-config-work-5) Allow bare dict in lieu of Field(Dict(...))
5e290d6dd62d
Actions