Skip to content

use description as band name#802

Merged
vincentsarago merged 4 commits into
mainfrom
feature/band-name-from-description
Jul 22, 2025
Merged

use description as band name#802
vincentsarago merged 4 commits into
mainfrom
feature/band-name-from-description

Conversation

@vincentsarago

@vincentsarago vincentsarago commented May 21, 2025

Copy link
Copy Markdown
Member

This is to also match what we do in xarray #807

# before
with Reader("tests/fixtures/cog_tags.tif") as src:
    info = src.info()
    img = src.preview()
    stats = src.statistics()

    print(info.band_descriptions)
    >> [('b1', 'Green')]

    print(img.band_names)
    >> ['b1']

    print(list(stats))
    >> ['b1']

# now
with Reader("tests/fixtures/cog_tags.tif") as src:
    info = src.info()
    img = src.preview()
    stats = src.statistics()

    print(info.band_descriptions)
    >> [('b1', 'Green')]

    print(img.band_names)
    >> ['Green']

    print(list(stats))
    >> ['Green']

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'rio-tiler Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.50.

Benchmark suite Current: d26d63c Previous: a7afa55 Ratio
dateline-int32-none 81.05817093625392 iter/sec (stddev: 0.0003640069041421796) 129.02721795604276 iter/sec (stddev: 0.00016375607088560558) 1.59

This comment was automatically generated by workflow using github-action-benchmark.

@vincentsarago vincentsarago merged commit faa9211 into main Jul 22, 2025
8 checks passed
@vincentsarago vincentsarago deleted the feature/band-name-from-description branch July 22, 2025 08:09
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.

1 participant