Skip to content

Conversation

@d-callan
Copy link
Collaborator

changed strategy a bit to try to go around the issue where some workflows may need support for additional pre-populated parameters, but still get some fun things on the site quickly.

this should get single read options for rnaseq and chipseq, add cutandrun, atacseq and some scrnaseq stuff.

also improved display names of some things. perhaps we want to improve those in iwc in future, and if so, probably an issue or two need filed somewhere for that?

feedback welcome

@d-callan d-callan requested a review from mvdbeek February 25, 2025 19:47
@NoopDog NoopDog changed the title adding more workflows feat: adding more workflows Feb 25, 2025
@NoopDog NoopDog changed the title feat: adding more workflows feat: adding more workflows (#322) Feb 25, 2025
@NoopDog NoopDog changed the title feat: adding more workflows (#322) feat: added additional workflow workflows (#322) Feb 25, 2025
@NoopDog NoopDog requested a review from hunterckx February 25, 2025 20:01
@NoopDog NoopDog changed the title feat: added additional workflow workflows (#322) feat: added additional workflows (#322) Feb 25, 2025
@mvdbeek
Copy link
Member

mvdbeek commented Feb 26, 2025

Can you please update this in the IWC? If we diverge here it will be hard to keep this updated.

@NoopDog
Copy link
Collaborator

NoopDog commented Feb 26, 2025

Hi @mvdbeek I was hoping we could move to taking this info directly from IWC. I will create a ticket for that in a bit. I'm hoping we don't need to block this PR on the IWC update.

@NoopDog
Copy link
Collaborator

NoopDog commented Feb 26, 2025

Hi @mvdbeek. Would you be able to review this for technical accuracy so we can merge it? We definitely need a plan to avoid the duplication of the descriptions, but for right now, do these look ok? Thanks!

@mvdbeek
Copy link
Member

mvdbeek commented Feb 26, 2025

You can merge this, but we're going to override this later.

@NoopDog NoopDog requested review from hunterckx and removed request for hunterckx February 26, 2025 20:20
@NoopDog NoopDog merged commit 8cab2ad into main Feb 26, 2025
1 check passed
@NoopDog NoopDog deleted the 322-more-workflows branch February 26, 2025 21:30
mvdbeek pushed a commit to mvdbeek/brc-analytics that referenced this pull request Mar 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants