Skip to content

Fix: Issue #95 - Remove duplicate routes and simplify Flask routing#96

Open
shubhapramanik07 wants to merge 1 commit intoruxailab:mainfrom
shubhapramanik07:fix-issue-95
Open

Fix: Issue #95 - Remove duplicate routes and simplify Flask routing#96
shubhapramanik07 wants to merge 1 commit intoruxailab:mainfrom
shubhapramanik07:fix-issue-95

Conversation

@shubhapramanik07
Copy link
Copy Markdown

Closes #95

This PR refactors the Flask routing structure to remove duplicate and unused route definitions, ensuring a single clear entry point for the application.

Changes made:

  • Cleaned up app/main.py:

    • Removed commented-out and duplicate route blocks
    • Retained only active and necessary route definitions
    • Simplified route handlers to delegate to session routes
  • Updated app/routes/session.py:

    • Removed redundant Flask app initialization
    • Ensured routing logic is properly handled from app/main.py

Impact:

  • No changes to business logic (calibration or prediction)
  • Improved code clarity and maintainability
  • Reduced redundancy and confusion in routing structure

Validation:

  • Application runs successfully using flask run
  • All existing endpoints are accessible and working as expected

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.

Refactor main.py to remove duplicate Flask initialization and inconsistent routes

1 participant