Skip to content
This repository was archived by the owner on Apr 20, 2025. It is now read-only.

Pull Request: Add Predictive Model & Interactive Analytics with Linting Improvements#10

Open
khiyami wants to merge 4 commits intomainfrom
analytics
Open

Pull Request: Add Predictive Model & Interactive Analytics with Linting Improvements#10
khiyami wants to merge 4 commits intomainfrom
analytics

Conversation

@khiyami
Copy link
Copy Markdown
Collaborator

@khiyami khiyami commented Mar 15, 2025

Overview

This pull request includes two sets of changes:

  1. Initial Implementation:

    • Adds a predictive model that processes course and requirement data.
    • Integrates a Plotly bar graph visualization for interactive analytics based on the processed data.
    • Handles merging data from Excel files stored in the correct directories and extracts key fields (e.g., major codes, short requirement labels).
  2. Linting Improvements:

    • Refactors the code to meet PEP8 guidelines.
    • Includes changes such as improved spacing, consistent quoting, and proper formatting.
    • Adds a main() function with a main guard (if __name__ == "__main__":) to enhance modularity and reusability.

Changes Made

  • Initial Commit:

    • Reads Excel files from the data/audit and data/course directories.
    • Merges datasets and processes data to extract and map major codes.
    • Generates an interactive bar chart using Plotly with a dropdown for major selection.
  • Linting Commit:

    • Reformats the code for consistency and readability.
    • Applies PEP8 styling to functions, variable names, and overall file structure.
    • Implements a main guard to ensure the code only runs when executed directly.

Testing

  • The predictive model and visualizations have been tested locally.

@khiyami khiyami added the enhancement New feature or request label Mar 15, 2025
@khiyami khiyami requested review from bbendou and jullia02 March 15, 2025 03:22
@khiyami khiyami self-assigned this Mar 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant