Skip to content
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

Added cli functionality #106

Merged
merged 3 commits into from
Sep 10, 2023
Merged

Added cli functionality #106

merged 3 commits into from
Sep 10, 2023

Conversation

Exterminator11
Copy link
Contributor

Description:
This PR strives to add new command line utility to the chat-miner project, this cli supports the conversion of chats to data frames that are eventually converted to csv files.

Changes Made:

  • Added the cli.py file that contains the main functionality for the CLI
  • Added setup.py file to use the CLI functionality
  • Added help to the CLI functionality

Screenshots:
Screenshot 2023-09-10 at 8 40 03 PM
Screenshot 2023-09-10 at 8 40 15 PM

Related issues:
#105 : CLI to parse into csv files

Additional notes:
I'm open to feedback and further improvements. Please let me know if any changes are needed.

@codecov-commenter
Copy link

codecov-commenter commented Sep 10, 2023

Codecov Report

Patch coverage has no change and project coverage change: -10.36% ⚠️

Comparison is base (8d0d7ad) 71.58% compared to head (1d599e7) 61.23%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #106       +/-   ##
===========================================
- Coverage   71.58%   61.23%   -10.36%     
===========================================
  Files           1        2        +1     
  Lines         278      325       +47     
===========================================
  Hits          199      199               
- Misses         79      126       +47     
Flag Coverage Δ
unittests 61.23% <0.00%> (-10.36%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
chatminer/cli.py 0.00% <0.00%> (ø)

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

@joweich joweich merged commit fc38cd1 into joweich:main Sep 10, 2023
13 checks passed
@joweich
Copy link
Owner

joweich commented Sep 10, 2023

@Exterminator11 looks great, thank you! 🚀
I quickly fixed the formatting to align with black/isort and removed the comments that are self-explanatory.
Congrats on your very first open source contribution! 🎉

@Exterminator11
Copy link
Contributor Author

Thank you for the kind words! 🚀 I'm glad I could help. Excited to be part of the open source community! 🎉.

@Exterminator11 Exterminator11 deleted the cli-chat-csv branch September 11, 2023 00:44
@joweich joweich mentioned this pull request Sep 16, 2023
4 tasks
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.

None yet

3 participants