-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add example and CLI to docs #60
Conversation
for more information, see https://pre-commit.ci
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #60 +/- ##
==========================================
+ Coverage 86.91% 89.97% +3.05%
==========================================
Files 14 15 +1
Lines 688 708 +20
==========================================
+ Hits 598 637 +39
+ Misses 90 71 -19 ☔ View full report in Codecov by Sentry. |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Note to self: I broke some stuff, so I'll need to fix it before I can ask for a review. |
for more information, see https://pre-commit.ci
…ports into examples-and-cli
report.generate(subject=participant_label) | ||
counter = report.generate(subject=participant_label) | ||
else: | ||
counter = report.generate() | ||
|
||
common_patterns = counter.most_common() | ||
if not common_patterns: | ||
LOGGER.warning("No common patterns found.") | ||
else: | ||
report.generate() | ||
with open(output_dir / "report.txt", "w") as f: | ||
f.write(str(counter.most_common()[0][0])) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will write out the most common report to a text file, but only if there's a description to write out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
couple of small things to address here or later
good with me merge at will thanks @tsalo |
Closes #34.
Changes proposed:
report.txt
with the most common description, when available.