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

Adding some lines of code #95

Open
wants to merge 48 commits into
base: merge-update
Choose a base branch
from

Conversation

Shalinis19137
Copy link

Description
This update adds additional functionality to the user authentication process by implementing a password strength checker. This change addresses issue #80, where users were able to set weak passwords, potentially compromising security. The motivation behind this update is to enhance the overall security of the application by ensuring users create strong passwords that meet certain criteria. No new dependencies are required for this change.

Fixes #80

Type of change
Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update
How Has This Been Tested?
I ran multiple tests to verify the effectiveness of the password strength checker. Tests included various password inputs, such as weak passwords (e.g., "12345," "password"), medium passwords (e.g., "Password1"), and strong passwords (e.g., "Str0ngP@ssw0rd!"). The system responded correctly by accepting only strong passwords.

Validated password strength for different input types
Checked user experience with feedback on password strength
Checklist:
My code follows the style guidelines of this project
I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes
Any dependent changes have been merged and published in downstream modules

I am a beginner in this field if you like my work please provide points under gssocs-ext 2024

Devasy23 and others added 30 commits March 8, 2024 14:06
Add API documentation for create_new_faceEntry, get_employees, read_employee,...

See merge request Devasy23/FaceRec!2
Preliminary code for vector search
Add Face Recognition System Proposal and Project Plan
* Update sonarcloud.yml

* Update sonarcloud.yml

* Update sonarcloud.yml

* Update sonarcloud.yml

* Update sonarcloud.yml via gpt4

* Update sonarcloud.yml

* Update sonarcloud.yml

* Update workflows

* Update build.yml

* Update metadata for sonarcloud

* Update build.yml

* chore: Update SonarCloud configuration

* chore: Update SonarCloud configuration

* chore: Update SonarCloud configuration
* Add vector_search function for pipeline aggregation

* Added Vector search

* chore: Add error handling for no match found in face recognition

* Refactor face recognition code to use Facenet512 model for better accuracy

* chore: Add tests for recognizing faces with no match found

* feat: Add test for vector search functionality
* Add pre-commit hooks and update requirements.txt

* Exclude testing directory from flake8 hook

* Fix formatting issues and update dependencies

* Add wily hook to pre-commit configuration

* fix(.pre-commit-config.yaml): Add commitizen hook to pre-commit config

* Updated Requirements

* Added input sanitation

* Converted logging.info() to logging.debug()

* Removed tensorflow-intel==2.15.0

---------

Co-authored-by: Devansh Shah <[email protected]>
Devasy23 and others added 18 commits June 4, 2024 23:08
* chore: Add Changelog CI workflow

* chore: Add Changelog CI workflow

* chore: Update Changelog CI workflow and dependencies

* chore: Update Changelog CI workflow and dependencies

* chore: Update Changelog CI workflow and dependencies

* chore: Update Changelog CI workflow and dependencies

* [Changelog CI] Add Changelog for Version v0.1.6

* chore: Update Changelog CI workflow and dependencies

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>, @devansh-shah-11
…e recognition (Devasy23#43)

* Add vector_search function for pipeline aggregation

* Added Vector Search

* chore: Add error handling for no match found in face recognition

* Refactor face recognition code to use the Facenet512 model for better accuracy

* Code added for recognize_face API

* temp commit to resolve merge

* Updated gitignore

---------

Co-authored-by: Devansh Shah <[email protected]>
…are them between original and fine-tuned models (Devasy23#49)

* feat: Add functionality to calculate intra-cluster distances and compare them between original and fine-tuned models

* Created using Colab

* Updated code to load finetuned model

* made fixing changes

* feat: Add functionality to calculate intra-cluster distances and compare them between original and fine-tuned models

* updated gitignore

* Updated code to load finetuned model

* Now everything works

* Created using Colab

* Kaggle Notebook | face rec_trail on indian faces | Version 2

* Added finetuned model

* Updated endpoints to calculate embeddings via finetuned model

* Added code to extract face, working Recognize_Face endpoint

* updated embedding function

* updated code to recalculate embeddings

* Updated vector search and recognize face endpoint

* removed "mtcnn" from detector backend to check embedding accuracy

---------

Co-authored-by: Devansh Shah <[email protected]>
* Restructured code

* chore: made important changes

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* chore: removed buggy unicode from precommit bot

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* chore: removed buggy unicode from precommit bot

---------

Co-authored-by: Devasy Patel <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* made important changes related to requirements.txt

* bumped version of matplotlib

* humped version of matplotlib

* added httpx to get some test back up passnig

* added sphinx to get read the docs passing

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* added myst-parser to get read the docs passing

* Update requirements.txt

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

4 participants