Remove old processed files - Githubissues

srbhr / Resume-Matcher

Resume Matcher is an open source, free tool to improve your resume. It works by using language models to compare and rank resumes with job descriptions.

https://www.resumematcher.fyi/

Apache License 2.0

4.76k stars 1.93k forks source link

Remove old processed files #86

Closed ILB-96 closed 11 months ago

ILB-96 commented 12 months ago

Pull Request Title

Improved Data handling

Related Issue

issue #74

Description

Json files in the Processed folder would stack on top of each other if not deleted manually but now they are removed automatically.
If you had 1-2 files the slider would cause an error because you can't have a slide with 1 item and the default value of the sliders was 2.

Type

[x] Bug Fix
[x] Feature Enhancement
[ ] Documentation Update
[ ] Code Refactoring
[ ] Other (please specify):

Proposed Changes

added files remover function
changed the slider default value to 0
added a condition to show a slider only if you have more than one file in the folder.

Screenshots / Code Snippets (if applicable)

How to Test

python run_first.py would remove the existing files
streamlite run streamlite_app.py would work even if you have only 1 resume/job description json file

Checklist

[x] The code compiles successfully without any errors or warnings
[x] The changes have been tested and verified
[ ] The documentation has been updated (if applicable)
[ ] The changes follow the project's coding guidelines and best practices
[x] The commit messages are descriptive and follow the project's guidelines
[x] All tests (if applicable) pass successfully
[x] This pull request has been linked to the related issue (if applicable)

Additional Information

srbhr commented 12 months ago

Thanks @Hazannovich for this PR. I will test it and then merge it.

Thanks for the PR.