Skip to content

feat: support custom CUR label aliases - #143

Merged
FloWuenne merged 4 commits into
mainfrom
cursor/issue-83-cur-label-docs-7425
Apr 23, 2026
Merged

feat: support custom CUR label aliases#143
FloWuenne merged 4 commits into
mainfrom
cursor/issue-83-cur-label-docs-7425

Conversation

@FloWuenne

@FloWuenne FloWuenne commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool, parameter, or workflow path, update the relevant docs.
  • If plugin declarations changed, update CITATIONS.md, README.md, and agent/context guidance in the same PR.
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
  • README.md is updated (including new tool citations and authors/contributors).

Description

This PR expands benchmark cost enrichment so users can provide custom CUR resource label aliases instead of being limited to the built-in user_unique_run_id / user_nf_unique_run_id, user_pipeline_process, and user_task_hash names.

New feature

Add optional benchmark_aws_cur_label_map YAML config support. The mapping file declares aliases for the logical benchmark cost fields:

  • run_id
  • process
  • task_hash

The normalizer now resolves those aliases against both supported CUR layouts:

  • flattened resource_tags_<alias> columns
  • map-style resource_tags entries

If no mapping file is supplied, the current default aliases still apply.

Included updates

  • thread benchmark_aws_cur_label_map through config, schema, workflow, and normalize CLI
  • resolve custom aliases during CUR normalization
  • update scripts/build_filtered_cost_sidecar.py to honor the same run-id alias mapping
  • document the new parameter and show example YAML config in README.md
  • sync agent/context/design docs with the new public parameter
  • add focused normalize tests covering custom flat aliases, custom map aliases, precedence, and config validation

Validation

  • uv run --with typer --with pyyaml --with jinja2 --with pyarrow --with pytest --with httpx pytest -v modules/local/normalize_benchmark_jsonl/tests/test_normalize.py
  • pre-commit run --all-files
Open in Web Open in Cursor 

cursoragent and others added 2 commits April 22, 2026 20:27
Co-authored-by: Florian Wuennemann <flowuenne@gmail.com>
Co-authored-by: Florian Wuennemann <flowuenne@gmail.com>
@cursor cursor Bot changed the title docs: document CUR resource labels feat: support custom CUR label aliases Apr 22, 2026
Co-authored-by: Florian Wuennemann <flowuenne@gmail.com>
@FloWuenne
FloWuenne requested a review from edmundmiller April 22, 2026 21:50
@FloWuenne
FloWuenne marked this pull request as ready for review April 23, 2026 13:42
@FloWuenne
FloWuenne merged commit 9080f6b into main Apr 23, 2026
46 checks passed
@FloWuenne
FloWuenne deleted the cursor/issue-83-cur-label-docs-7425 branch April 23, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants