Skip to content

Conversation

@varsha1224
Copy link

This PR implements the NER-LSTM model for claim appeal documents and the LSTM model for stock prediction. The folder structure has been reorganized to separate the two projects for better clarity and management.

Key updates:

  • The model uses Keras version 2.11.0, a different version than previously mentioned, to pass the pylint checks.
  • Faker library (Version: 37.1.0) has been introduced to generate sample claim appeal documents for testing and training purposes.

@codit04
Copy link
Member

codit04 commented Apr 15, 2025

  • add all your imported packages in requirements.txt
  • refactor your code such that your backend and frontend have a clear distinction

@pranathiir
Copy link
Contributor

updated requirements.txt
rearranged code into distinct backend and frontend files

@codit04
Copy link
Member

codit04 commented Apr 15, 2025

there are few conflicts in your requirements.txt

  • removal of necessary packages pytest and pylint
  • unnecessary inclusions of packages like (fqdn,future) which have never been used

Copy link
Member

@codit04 codit04 left a comment

Choose a reason for hiding this comment

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

same as comment

@pranathiir pranathiir requested a review from codit04 April 15, 2025 16:52
@varsha1224 varsha1224 requested a review from Athu101103 April 15, 2025 18:00
Copy link
Member

@codit04 codit04 left a comment

Choose a reason for hiding this comment

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

please mention the format of the input that has to be given to the stocks_app.py, also add a few datasets for the app.py file and have a single frontend app with 2 pages instead of having 2 seperate pages.

@varsha1224
Copy link
Author

we have added a few sample documents (pdf, txt, docs) under Datasets for appeals and have combined the two separate pages into a single frontend app. The format of the input that has to be given to the stocks_app.py is also mentioned.

@varsha1224 varsha1224 requested a review from codit04 April 16, 2025 08:58
@codit04 codit04 merged commit 42a3850 into main Apr 16, 2025
2 checks passed
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.

3 participants