-
Notifications
You must be signed in to change notification settings - Fork 8
Phase 1 of my planned edits #9
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
Open
TataSatyaPratheek
wants to merge
64
commits into
ERPdotAI:main
Choose a base branch
from
TataSatyaPratheek:phase1
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… progress tracking during training and evaluation
… and dependency management
…; remove unused histogram plotting in process mining
…alization options
…cess maps analysis
…g fallback logic for enhanced utilities
…sure correct tensor handling and avoid broadcasting issues
…sion and improving metric calculations
…n_neighbors values
… license in README
…with new modes and parameters
…ance configuration
…ndling and batch processing
…mized functions for node and edge processing
…ce error handling mechanisms
…iveGATConv for improved performance
…anagement utilities
…graph utility functions
…methods, and improve visualization memory efficiency
…ion for improved performance
…sing for tuple handling
… mapping in data loader, and update README for DGL integration and ablation study features
…h newer NetworkX versions
… float conversion for statistics
…and improve error handling
…hancing label extraction and improving fallback mechanism for random splits.
…nd improve compatibility with NumPy input data
…mprove validation for various model types
…pes and ensure continuous labels for classification
… consistent padding
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Process Mining Enhancements with Memory Efficiency and DGL Integration
Overview
Greetings ERP.AI team,
I'm thrilled to submit this pull request that transforms the original GNN-based process mining framework into a more comprehensive, memory-efficient toolkit with Deep Graph Library (DGL) integration. I've had an absolute blast working on this project and exploring the fascinating intersection of process mining and graph neural networks!
Key Enhancements
Background
As a mathematician from Hyderabad with a Masters in Mathematics from India and another Masters in Data Analytics and AI from France, I'm passionate about applying advanced mathematical modeling to improve manufacturing and business processes in India. While I'm not an engineer by training, I believe my mathematical background offers a unique perspective on process optimization problems.
Implementation Notes
I want to be transparent that I used Claude 3.7 Sonnet to help me with code generation, as I was eager to contribute but had limited time available. The implementation is currently incomplete and requires further testing and refinement from a competent team. I've focused primarily on the architecture and API design, with placeholder implementations for some of the more complex components.
Future Work
If merged, I'd love to continue contributing to this project by:
I'm eager to learn from your team and improve my contributions based on your feedback. I believe this work represents a good starting point for discussion about the direction of the project.
Thank you for considering my contribution. The childlike excitement I feel about improving processes and manufacturing in India through this work is difficult to contain!
Warm regards,
Satya Pratheek Tata