Skip to content

ADD: property resample#60

Draft
cherman2 wants to merge 2 commits intoqiime2:mainfrom
cherman2:resample_property
Draft

ADD: property resample#60
cherman2 wants to merge 2 commits intoqiime2:mainfrom
cherman2:resample_property

Conversation

@cherman2
Copy link
Copy Markdown
Collaborator

@cherman2 cherman2 commented Dec 9, 2025

closes #41

@cherman2 cherman2 self-assigned this Dec 10, 2025
@cherman2 cherman2 moved this to In Development in 2026.1 ❄️ Dec 10, 2025
@Oddant1
Copy link
Copy Markdown
Member

Oddant1 commented Jan 7, 2026

I should probably give a status update here. @ebolyen and I did some sleuthing. This line of code is causing silly things to happen if there are 20+ elements in the collection https://github.com/qiime2/qiime2/blob/dev/qiime2/core/type/grammar.py#L603-L606. Resolving may be non-trivial.

@cherman2
Copy link
Copy Markdown
Collaborator Author

cherman2 commented Jan 7, 2026

Hi @Oddant1 and @ebolyen, looks like it's not worth the squeeze then? Should we draft this in case we want to circle back? If I recall correctly, this was going to be helpful for adagio (spelling?).

@Oddant1
Copy link
Copy Markdown
Member

Oddant1 commented Jan 7, 2026

I'm poking it a bit, but there's no free lunch here.

@cherman2
Copy link
Copy Markdown
Collaborator Author

cherman2 commented Jan 8, 2026

Drafting for now because this is a can of worms

@cherman2 cherman2 marked this pull request as draft January 8, 2026 17:38
@cherman2 cherman2 removed the status in 2026.1 ❄️ Jan 8, 2026
@github-project-automation github-project-automation bot moved this to Backlog in 2026.4 🌱 Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

add property tag to alpha-collections and beta-collections output-collections.

4 participants