Skip to content

Conversation

@BryonLewis
Copy link
Collaborator

resolves #4

  • Adds an output folder option to pipeline
  • removes the unnecessary printing of the scoring (scoring isn't enabled yet)
  • Creates the directory if the folder doesn't exist for output
  • Still defaults the current directory if nothing is provided.

@BryonLewis BryonLewis requested a review from stoutta6 January 22, 2026 16:13
@codecov
Copy link

codecov bot commented Jan 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.3%. Comparing base (ee1c09b) to head (b93f23e).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main      #7     +/-   ##
=======================================
+ Coverage   71.1%   71.3%   +0.1%     
=======================================
  Files          6       6             
  Lines        964     969      +5     
=======================================
+ Hits         686     691      +5     
  Misses       278     278             
Files with missing lines Coverage Δ
batbot/__init__.py 75.7% <100.0%> (+0.7%) ⬆️
batbot/spectrogram/__init__.py 70.3% <100.0%> (+<0.1%) ⬆️
tests/test_spectrogram.py 100.0% <100.0%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee1c09b...b93f23e. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bluemellophone bluemellophone force-pushed the issue-4-output-folder-option branch from 0aebbf0 to b93f23e Compare January 26, 2026 20:03
Copy link
Collaborator

@bluemellophone bluemellophone left a comment

Choose a reason for hiding this comment

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

LGTM

@bluemellophone bluemellophone merged commit 0fdf8b9 into main Jan 26, 2026
11 checks passed
@bluemellophone bluemellophone deleted the issue-4-output-folder-option branch January 26, 2026 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feat: Add output option to the pipeline function

2 participants