Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Start promoting Falco as a FastQC replacement in key GTN material #5706

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

wm75
Copy link
Contributor

@wm75 wm75 commented Jan 21, 2025

Last year at GCC we discussed that FastQC is among the top tools in terms of total cpu usage on public Galaxy instances and that Falco could help us save quite some resources if people used it instead of the original.
Since then I've been testing Falco and helped fixing bugs upstream and in our wrapper. The tool should now be on par with FastQC and so it's time to have more users discover it by actively promoting it in key GTN tutorials.

@wm75 wm75 requested a review from a team as a code owner January 21, 2025 10:16
@wm75
Copy link
Contributor Author

wm75 commented Jan 21, 2025

Going to adjust more tutorials whenever I have time.


Unfortunately the current version of multiQC (the tool we use to combine reports) does not support list of pairs collections.
We will first need to transform our the list of pairs to a simple list.
Unfortunately the current version of MultiQC (the tool we use to combine reports) does not support list of pairs collections.
Copy link
Member

Choose a reason for hiding this comment

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

Oh is this still the case? If so we need to work on it.

Copy link
Member

@bgruening bgruening left a comment

Choose a reason for hiding this comment

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

LGTM, but it would be nice to have more approvals - as this is a big change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants