-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Enhancements to CI Workflows and Python Module Initialization with Minor Fixes #1061
base: master
Are you sure you want to change the base?
Commits on Oct 24, 2024
-
Fix GitHub Actions workflow issues
- Add fail-fast: false to prevent cancellations - Fix typos in artifact names and make them unique - Add proper error handling and verbosity to Python tests - Fix gather-digests job name and make it more resilient - Add proper ARM64 skip configuration
Configuration menu - View commit details
-
Copy full SHA for 70afaf3 - Browse repository at this point
Copy the full SHA 70afaf3View commit details -
Configuration menu - View commit details
-
Copy full SHA for f48894c - Browse repository at this point
Copy the full SHA f48894cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 1c9d5ba - Browse repository at this point
Copy the full SHA 1c9d5baView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8734e76 - Browse repository at this point
Copy the full SHA 8734e76View commit details -
Configuration menu - View commit details
-
Copy full SHA for 69fe26f - Browse repository at this point
Copy the full SHA 69fe26fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8b6f03f - Browse repository at this point
Copy the full SHA 8b6f03fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8009ff6 - Browse repository at this point
Copy the full SHA 8009ff6View commit details -
Configuration menu - View commit details
-
Copy full SHA for 9e82cc2 - Browse repository at this point
Copy the full SHA 9e82cc2View commit details -
fix: Improve Python wrapper build setup in cmake workflow
- Add pip, setuptools, and wheel upgrades - Install build and pytest dependencies explicitly - Set PYTHONPATH and LD_LIBRARY_PATH for proper library discovery - Ensure proper environment setup before building Python wrapper
Configuration menu - View commit details
-
Copy full SHA for 9861fed - Browse repository at this point
Copy the full SHA 9861fedView commit details -
fix: Update job-level permissions in cmake workflow
- Add comprehensive permissions at job level to match workflow level - Include write permissions for pull-requests, actions, and checks - Ensure proper execution of pull_request_target events
Configuration menu - View commit details
-
Copy full SHA for 20a26ed - Browse repository at this point
Copy the full SHA 20a26edView commit details -
fix: Split Python wrapper build into platform-specific steps with pro…
…per environment setup
Configuration menu - View commit details
-
Copy full SHA for c34d059 - Browse repository at this point
Copy the full SHA c34d059View commit details -
Configuration menu - View commit details
-
Copy full SHA for 13dac41 - Browse repository at this point
Copy the full SHA 13dac41View commit details -
Configuration menu - View commit details
-
Copy full SHA for e1062eb - Browse repository at this point
Copy the full SHA e1062ebView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6dc9aea - Browse repository at this point
Copy the full SHA 6dc9aeaView commit details -
Configuration menu - View commit details
-
Copy full SHA for bdd7253 - Browse repository at this point
Copy the full SHA bdd7253View commit details -
Configuration menu - View commit details
-
Copy full SHA for accc605 - Browse repository at this point
Copy the full SHA accc605View commit details -
ci: Improve workflow configuration to prevent cancellations
- Remove push triggers to focus on pull_request_target - Add workflow_dispatch for manual triggering - Improve concurrency settings with SHA-based grouping - Add explicit GITHUB_TOKEN environment variable - Simplify permissions inheritance - Add conditional to only run on forks and manual triggers
Configuration menu - View commit details
-
Copy full SHA for 6b29d4d - Browse repository at this point
Copy the full SHA 6b29d4dView commit details -
fix: Add _init.py to handle proper module initialization and prevent …
…circular imports
Configuration menu - View commit details
-
Copy full SHA for 3b47b7a - Browse repository at this point
Copy the full SHA 3b47b7aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 48067e9 - Browse repository at this point
Copy the full SHA 48067e9View commit details -
fix: Move __version__ import to beginning of pythoncode block to prev…
…ent circular imports
Configuration menu - View commit details
-
Copy full SHA for cbf7919 - Browse repository at this point
Copy the full SHA cbf7919View commit details -
Configuration menu - View commit details
-
Copy full SHA for 286bf5b - Browse repository at this point
Copy the full SHA 286bf5bView commit details -
fix: Update workflow triggers from pull_request_target to pull_reques…
…t and simplify checkout configuration
Configuration menu - View commit details
-
Copy full SHA for 13bb730 - Browse repository at this point
Copy the full SHA 13bb730View commit details -
Configuration menu - View commit details
-
Copy full SHA for be8e57a - Browse repository at this point
Copy the full SHA be8e57aView commit details -
fix: Improve environment variable handling in cmake workflow for Pyth…
…on wrapper build
Configuration menu - View commit details
-
Copy full SHA for 80c8adf - Browse repository at this point
Copy the full SHA 80c8adfView commit details -
Configuration menu - View commit details
-
Copy full SHA for 90f9aaa - Browse repository at this point
Copy the full SHA 90f9aaaView commit details -
Configuration menu - View commit details
-
Copy full SHA for fd0467a - Browse repository at this point
Copy the full SHA fd0467aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 84a0994 - Browse repository at this point
Copy the full SHA 84a0994View commit details -
ci: Simplify cmake workflow to focus on Ubuntu builds
- Remove Windows-specific configurations - Add environment variables at workflow level - Add verbose output for debugging - Add explicit dependency installation
Configuration menu - View commit details
-
Copy full SHA for dd9b790 - Browse repository at this point
Copy the full SHA dd9b790View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8473a92 - Browse repository at this point
Copy the full SHA 8473a92View commit details -
Configuration menu - View commit details
-
Copy full SHA for 47b0d34 - Browse repository at this point
Copy the full SHA 47b0d34View commit details -
fix: Improve module initialization to prevent circular imports
- Add error handling in _load_sentencepiece() - Create separate registration functions - Move registrations to end of file - Add centralized initialization function
Configuration menu - View commit details
-
Copy full SHA for 8aa9829 - Browse repository at this point
Copy the full SHA 8aa9829View commit details -
fix: Add proper SWIG registration order and improve error handling
- Register immutable classes before processor classes - Add explicit error message for module load failures - Remove silent error handling in _load_sentencepiece
Configuration menu - View commit details
-
Copy full SHA for 9b71d86 - Browse repository at this point
Copy the full SHA 9b71d86View commit details -
fix: Implement lazy loading and proper registration sequence
- Add lazy initialization for immutable classes - Defer SWIG registrations until after class definitions - Improve error handling for module loading and registration - Fix circular import issues in module initialization
Configuration menu - View commit details
-
Copy full SHA for 279a981 - Browse repository at this point
Copy the full SHA 279a981View commit details -
Merge pull request #1 from kasinadhsarma/devin/fix-workflow-issues/2745
Fix GitHub Actions workflow issues
Configuration menu - View commit details
-
Copy full SHA for 529f314 - Browse repository at this point
Copy the full SHA 529f314View commit details -
fix: Improve module initialization and registration sequence
- Add global initialization flag to prevent double registration - Remove premature class registration - Improve error handling with specific error messages - Ensure proper registration order for immutable classes
Configuration menu - View commit details
-
Copy full SHA for e0aa610 - Browse repository at this point
Copy the full SHA e0aa610View commit details -
fix: Improve module initialization and registration sequence
- Add proper state tracking for module loading - Prevent circular imports during initialization - Add dependency checks for SWIG registrations - Improve error handling for module loading - Ensure proper registration order for all classes
Configuration menu - View commit details
-
Copy full SHA for 513b766 - Browse repository at this point
Copy the full SHA 513b766View commit details -
fix: Implement proper lazy loading and initialization for SWIG classes
- Add state tracking with _initialized flag - Implement safe lazy loading with _ensure_initialized - Add SWIG module availability checks - Remove direct SWIG calls from __init__ - Improve error handling for module loading - Prevent circular imports during initialization
Configuration menu - View commit details
-
Copy full SHA for c160d10 - Browse repository at this point
Copy the full SHA c160d10View commit details -
fix: Improve module initialization and registration sequence
- Add registration state tracking to prevent circular imports - Modify _load_sentencepiece to handle registration phase - Add checks for required SWIG registration functions - Improve error handling in SentencePieceNormalizer - Restructure registration sequence for proper dependency order
Configuration menu - View commit details
-
Copy full SHA for f65c814 - Browse repository at this point
Copy the full SHA f65c814View commit details -
fix: Improve module initialization and registration handling
- Add module load attempt tracking to prevent circular imports - Improve registration phase state management - Add proper cleanup of state flags on errors - Ensure proper initialization sequence for SWIG classes
Configuration menu - View commit details
-
Copy full SHA for e836866 - Browse repository at this point
Copy the full SHA e836866View commit details -
fix: Improve module initialization and import mechanism
- Use absolute imports to prevent path-related issues - Add proper path resolution for src directory - Ensure module initialization happens after path setup
Configuration menu - View commit details
-
Copy full SHA for f696c4f - Browse repository at this point
Copy the full SHA f696c4fView commit details -
fix: Add SWIG registration function verification
- Add early verification of required SWIG registration functions - Improve error messages for missing registration functions - Prevent silent failures during module initialization
Configuration menu - View commit details
-
Copy full SHA for 1d800a8 - Browse repository at this point
Copy the full SHA 1d800a8View commit details -
fix: Improve module initialization and registration sequence
- Add registration lock to prevent circular imports - Move SWIG registrations to dedicated function - Improve error handling and state management - Remove redundant function verification
Configuration menu - View commit details
-
Copy full SHA for 42cf801 - Browse repository at this point
Copy the full SHA 42cf801View commit details -
fix: Improve module initialization to prevent circular imports
- Separate module loading from registration phase - Add two-phase initialization in _register_all_classes - Remove registration lock from _load_sentencepiece - Improve error handling for module loading
Configuration menu - View commit details
-
Copy full SHA for 58eb50e - Browse repository at this point
Copy the full SHA 58eb50eView commit details -
Merge pull request #2 from kasinadhsarma/devin/fix-workflow-issues/2745
fix workflow issues/2745
Configuration menu - View commit details
-
Copy full SHA for 8aac6ba - Browse repository at this point
Copy the full SHA 8aac6baView commit details -
fix: Implement robust module loading and registration sequence
- Add loading lock to prevent circular imports - Improve error handling with proper cleanup - Add retry mechanism for SWIG registrations - Ensure proper initialization order for all classes
Configuration menu - View commit details
-
Copy full SHA for 2cfb0ff - Browse repository at this point
Copy the full SHA 2cfb0ffView commit details -
Merge pull request #3 from kasinadhsarma/devin/fix-workflow-issues/2745
fix: Implement robust module loading and registration sequence
Configuration menu - View commit details
-
Copy full SHA for 3a525a2 - Browse repository at this point
Copy the full SHA 3a525a2View commit details