abstracting plotting

[K20shores]

When checking out differences in TS1, I noticed that plotting no longer worked after we (correctly) added data headers in #273

This PR

• moves all plotting to its own file and adds tests, also makes it easier to plot
• corrects the readme examples
• corrects some data output functionality so that we can output to a file and print to the terminal if the output type is csv

[adityakiran1423]

Hello @K20shores. Just one thing :
In src/acom_music_box/main.py (before changes where made to it) the plot_with_gnuplot function runs a finally block where data_file_path is deleted if generated after successfully plotting using gnu_plot.

However

In src/acom_music_box/plot_output.py the plot_with_gnuplot function doesn't seem to have a finally block for deleting data_file_path. Is this done for a specific reason?

I've reviewed all the other files changed in the PR, looks good to me!

[adityakiran1423]

Looks good to me!

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 90.27778% with 7 lines in your changes missing coverage. Please review.

Project coverage is 81.91%. Comparing base (383e938) to head (90380ca).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
src/acom_music_box/plot_output.py	88.70%	7 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #278      +/-   ##
==========================================
+ Coverage   79.90%   81.91%   +2.01%     
==========================================
  Files          12       13       +1     
  Lines         821      846      +25     
==========================================
+ Hits          656      693      +37     
+ Misses        165      153      -12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[montythind]

is there reason for a Deep copy?

[K20shores]

Yes, without the deep copy, the changes made on the dataframe inside this class are reflected elsewhere, and that's not what we want

[montythind]

Looks good to me.

[boulderdaze]

Looks good to me! I just have a minor suggestion