#34191: Combine multiple analysis files into single data set -------------------------------+-------------------------------- Reporter: karsten | Owner: acute Type: enhancement | Status: needs_revision Priority: Medium | Milestone: Component: Metrics/Onionperf | Version: Severity: Normal | Resolution: Keywords: | Actual Points: 0.8 Parent ID: #33321 | Points: 0.5 Reviewer: karsten | Sponsor: Sponsor59-must -------------------------------+--------------------------------
Comment (by acute): Replying to [comment:8 karsten]: > Quick thought: How about we accept a single `PATH` argument (followed by the `LABEL` argument) but accept a file or directory or (quoted) path pattern expanded by OnionPerf rather than the shell? The command in your example would then be: `onionperf visualize -d "onionperf-2019-11/*/*nl*" nov2019 -d ...` (note the quotes). Untested! After all, if the user wants to use spaces in their labels, they'll have to quote the label, too. This could work - the only possible issues would be shell compatibility, and, of course, the user would still have to create directories if wanting to aggregate measurements over say, different instances or files without similar names. I pushed fixes to address the label and usage concerns you had previously: https://github.com/ana-cc/onionperf/commits/34191 However, if there are still objections to accepting multiple paths, I can explore the option you suggest here. -- Ticket URL: <https://trac.torproject.org/projects/tor/ticket/34191#comment:10> Tor Bug Tracker & Wiki <https://trac.torproject.org/> The Tor Project: anonymity online
_______________________________________________ tor-bugs mailing list tor-bugs@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs