What if we gave PartitionSelectors a name attribute, registered them globally, and used them here?
... --selector LastPartitionSelector ... --selector BackfillSelector ... --selector FailedRunsSelector # possible when we thread instance to the selector
You can use:
To take the pipeline_name as an argument and get the pipeline from the handle.
This is awesome. The interactive part is really nice to use.
One thing I noticed was that the tags are not being added properly, but I'm not sure why, since they are being passed into PipelineRun
This must be click's fault, but if I type a single character and then hit backspace, then the entire prompt disappears.
This is great
Maybe truncate the middle if len(partitions) > big_number?
I think this needs to be environment_dict_fn_for_partition (missing fn)
Same here (tags_fn_for_partition)
will look into the tags issue
Ah, yeah... I am using pipeline_target_command already (see line 401)...
I had create_pipeline_from_cli_args at first, but when we needed to load the partitions off the repo, I switched to using handle_for_repo_cli_args and had to pop things off the kwargs, because handle_for_repo_cli_args throws if it finds other args.
Ultimately, I ended up building up the repo_args manually so we could use both handle_for_repo_cli_args and create_pipeline_from_cli_args, but it is a little confusing :/
yeah, this is still WIP, to be honest....
i really wanted to start measuring window size, so i could tabular columns like ls, but haven't gotten around to it
ah, bad merge... i had that fixed in some version...