-
Notifications
You must be signed in to change notification settings - Fork 4
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
Can't get fastmultigather to write to a specific output file #299
Comments
right, see #239 - |
Ahh too bad. Would there be a way to work around this using zip files? as i currently cannot get rockdbs to work |
@ctb are you ok with switching to single-file output for all queries? Or should we aim to support both? |
Two hot takes - I'm kind of against switching things up wilhe-nilhe just because? The branchwater plugin is becoming a bit of a UX nightmare already and changing up what we do in the middle of a point release seems suboptimal. We've done it before, we'll do it again, but I question the necessity here ;). Can't we just give @amterhorst a csvtk command that aggregates all of the CSV files into one or something?? I would be more supportive of making it so that when This is all in the background of a complicated set of design decisions discussed over in sourmash-bio/sourmash#2328, for which I have no answer. So, umm, after all of this - yeah, sure, switch to |
Ok: rocksdb not working was my fault, as i tried to create directly from a sig.gz file, not a list of filepaths. As for a csvtk command, that be great, but my issue is that the output file gets overwritten. Doing this because if using one db, it doesn't finish. Anyway: Rocksdb fixed my problem. |
True! Another workaround is to run it in a separate directory. |
…ather (#320) Fixes #313 Fixes #285 Addresses #239 / #299 by bailing out when an output path is provided but will not be used: ``` RuntimeError: output path specified, but not running fastmultigather against a rocksdb. See issue #239 ``` although we are leaning towards respecting `-o` in the future, see #299 discussion, so this will soon get revisited. --------- Co-authored-by: Tessa Pierce Ward <bluegenes@users.noreply.github.com>
Running fastmultigather on a set of metagenome reads x a set of zip files and i want the output to be named as metagenome.x.zip.csv
Running as:
sourmash scripts fastmultigather ../../sourmash_sketches/sketch_reads/ERR1135178.k21.zip ../../sourmash_sketches/split_sig/ERR11351_1.zip --output ERR1135178xERR11351_1.csv -k 21 -t 1000 -s 100 -c 1
Gives me 2 output files:
ERR1135178.gather.csv
andERR1135178.gather.csv
, which is not what I wantThe text was updated successfully, but these errors were encountered: