Can we only publish the trimmed fastqs, rather tha...
# demultiplexs
Simon Pearce
09/14/2026, 12:31 PMCan we only publish the trimmed fastqs, rather than the raw ones too? This doubles the long-term storage at the moment
m
Maxime Garcia
09/14/2026, 12:42 PMWe can have an option for that, sure
Maxime Garcia
09/14/2026, 12:47 PMI'll whip out a PR
❤️ 1
s
Simon Pearce
09/14/2026, 12:59 PMPing me here when you need a review
m
Maxime Garcia
09/14/2026, 2:07 PMCan I get one on https://github.com/nf-core/demultiplex/pull/421 before?
lingerrings PublishDir is probably messing things up in my current PRs
📝 1
Maxime Garcia
09/14/2026, 2:27 PMI have replied
Maxime Garcia
09/14/2026, 2:34 PMThanks
🙌 1
Maxime Garcia
09/14/2026, 2:48 PMAnd this is for you https://github.com/nf-core/demultiplex/pull/422
Maxime Garcia
09/14/2026, 3:18 PMlooking at your comment, I have no issues having this default as true instead of false, I am just wondering what was the actual default in the last release, to see if it's a breaking change or not
s
Simon Pearce
09/14/2026, 4:01 PMthe default was to publish both
m
Maxime Garcia
09/14/2026, 4:10 PMThen I'd say let's keep it this way for now
booo 1
Maxime Garcia
09/14/2026, 4:27 PMThen I need to do a 1.8.0 release instead
Maxime Garcia
09/14/2026, 4:27 PMOnly because it's you
🤣 1
Maxime Garcia
09/14/2026, 4:28 PMI'll work on this tomorrow
s
Simon Pearce
09/14/2026, 4:30 PMits just find + replace for 1.8.0
m
Maxime Garcia
09/15/2026, 7:54 AMyeah, but a tiny bit more for the current PR, but no worries, I agree that we should have default to produce less data.
Data footprint is important
Maxime Garcia
09/15/2026, 9:13 AMPR has been updated: https://github.com/nf-core/demultiplex/pull/422
✅ 1
s
Simon Pearce
09/15/2026, 9:14 AMyou sure you've pushed Maxime?
Simon Pearce
09/15/2026, 9:14 AMah, I see it now 😉
Simon Pearce
09/15/2026, 9:15 AMThanks 🙂
m
Maxime Garcia
09/15/2026, 9:17 AMsorry, bad connection
3 Views