Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Added Phishing fake Url Detector using ML #222

Closed
wants to merge 1 commit into from

Conversation

SimranShaikh20
Copy link

@SimranShaikh20 SimranShaikh20 commented Nov 10, 2024

Description

Added folder inlucing mandatory file and format

Issue Resolved

Issue no - #221

Checklist

  • My code follows the current project structure
  • I have thoroughly reviewed and updated the requirements.txt file to include any new packages
  • The predict.py file includes a properly implemented model_details() function, or the notebook contains this function to print a detailed model report. The model will not be accepted without this function, as it is essential for generating the necessary model details.
  • I have added relevant tests (if necessary).
  • I have added comments in the code where needed.
  • This PR is submitted under Hacktoberfest.
  • This PR is submitted under GirlScript Summer of Code (GSSoC-Extd).

@yashasvini121 pls review my PR and merged it if there is any conflicts then let me know

Copy link
Owner

@yashasvini121 yashasvini121 left a comment

Choose a reason for hiding this comment

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

Missing the model page. Kindly go through the tutorial properly and add the page as well.

@SimranShaikh20
Copy link
Author

hey @yashasvini121 can pls check again i have model
image

@SimranShaikh20
Copy link
Author

models/
├── loan_approval/
│ ├── data/ # Folder for storing the dataset used for training
│ ├── notebooks/ # Jupyter notebooks for training and experimentation
│ ├── saved_models/ # Folder for saving the trained model and any scalers
│ ├── model.py # Script to define and train your model
│ ├── predict.py # Script to load the trained model and make predictions
│ ├── modelEvaluation.py

as per above format i have added

@SimranShaikh20 SimranShaikh20 closed this by deleting the head repository Nov 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants