eliminate middle function create_assets_def_from_fn_and_decorator_args

[schrockn]

Summary & Motivation

How I Tested These Changes

[schrockn]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• Make optional handling in make_asset_deps codepaths more strict #22497
• elim unnecessary local vars #22496
• Eliminate middle args object #22495
• eliminate middle function create_assets_def_from_fn_and_decorator_args #22494 👈
• use better pattern for decorator compute_fn is Nomne #22493
• Push backfill policy check into generic builder path #22429
• Have multi_asset_check call builder.create_asset_checks_definition( #22417 : 1 other dependent PR (#22428 )
• Make AssetChecksDefinition.create call trivially stacked on builder properties in multi_asset_check #22416
• Use builder.create_op_definition in @multi_asset_check #22415
• Use builder to construct required_resource_keys in multi_asset_check #22414
• Move AssetChecksDefinition.create into the builder #22413
• In preparation for moving AssetChecksDefinition.create into the builder, make it trivially use builder properties. #22407
• add typing information to list_param variants #22412
• Use create_op_definition instead of manually creating _Op #22408
• Move check spec validation into asset-decorator-only creation pathways #22406 : 1 other dependent PR (#22403 )
• Begin stack to use DecoratorAssetsDefinitionBuilder for asset-check-related decorators  #22402
• Rename the factory methods to DecoratorAssetsDefinitionBuilder. #22400 : 1 other dependent PR (#22380 )
• Add type checks to test_asset_decorator_with_check_specs.py #22404
• Add is_iterable and use in test_asset_check_decorator #22401
• Make is_list typed #22399
• Type test_asset_check_decorator.py #22398
• Sort attributes on DecoratorAssetsDefinitionBuilderArgs #22383 : 1 other dependent PR (#22384 )
• Use explicitly passed in arguments instead of args object in builder creation codepaths #22379
• Rename build_asset_outs to build_named_outs #22382
• Renamed build_asset_ins to build_named_ins #22381
• Use compute required resources help in multi_asset #22279
• Use compute_required_resource_keys_for_underlying_op in @asset_check #22258
• Clarifying rename of AssetsDefinitionFactory-related classes and properties #22251
• Factor out strange resource-related logic into helper class RelatedResourceState #22243
• Use compute_required_resource_keys_for_underlying_op in @asset construction #22236 : 1 other dependent PR (#22241 )
• Eliminate layer of callables by making _Asset a non-callable NamedTuple #22235
• Make _Asset have no logic in prep for its elimination #22234
• Have @asset use AssetsDefinitionBuilder #22232
• InOutMapping --> AssetsDefinitionBuilder #22230
• Move spec resolution into mapper #22229
• Extract compute_required_resource_keys #22228
• Extract logic to create partition_mapping into the factory #22227
• Factor out keys_by_input_name and keys_by_output_name into the factory area #22225 : 1 other dependent PR (#22226 )
• Move outs parameter contruction to InOutMapper #22224
• Move check_specs_by_output_name into InOutMapper #22223
• Extract InOutMapper to begin refactoring AssetsDefinition construction process #22222
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @schrockn and the rest of your teammates on Graphite