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

Enhancements to CI Workflows and Python Module Initialization with Minor Fixes #1061

Open
wants to merge 46 commits into
base: master
Choose a base branch
from

Commits on Oct 24, 2024

  1. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    70afaf3 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    f48894c View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    1c9d5ba View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    8734e76 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    69fe26f View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    8b6f03f View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    8009ff6 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    9e82cc2 View commit details
    Browse the repository at this point in the history
  9. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    9861fed View commit details
    Browse the repository at this point in the history
  10. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    20a26ed View commit details
    Browse the repository at this point in the history
  11. Configuration menu
    Copy the full SHA
    c34d059 View commit details
    Browse the repository at this point in the history
  12. Configuration menu
    Copy the full SHA
    13dac41 View commit details
    Browse the repository at this point in the history
  13. Configuration menu
    Copy the full SHA
    e1062eb View commit details
    Browse the repository at this point in the history
  14. Configuration menu
    Copy the full SHA
    6dc9aea View commit details
    Browse the repository at this point in the history
  15. Configuration menu
    Copy the full SHA
    bdd7253 View commit details
    Browse the repository at this point in the history
  16. Configuration menu
    Copy the full SHA
    accc605 View commit details
    Browse the repository at this point in the history
  17. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    6b29d4d View commit details
    Browse the repository at this point in the history
  18. Configuration menu
    Copy the full SHA
    3b47b7a View commit details
    Browse the repository at this point in the history
  19. Configuration menu
    Copy the full SHA
    48067e9 View commit details
    Browse the repository at this point in the history
  20. Configuration menu
    Copy the full SHA
    cbf7919 View commit details
    Browse the repository at this point in the history
  21. Configuration menu
    Copy the full SHA
    286bf5b View commit details
    Browse the repository at this point in the history
  22. fix: Update workflow triggers from pull_request_target to pull_reques…

    …t and simplify checkout configuration
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    13bb730 View commit details
    Browse the repository at this point in the history
  23. Configuration menu
    Copy the full SHA
    be8e57a View commit details
    Browse the repository at this point in the history
  24. Configuration menu
    Copy the full SHA
    80c8adf View commit details
    Browse the repository at this point in the history
  25. Configuration menu
    Copy the full SHA
    90f9aaa View commit details
    Browse the repository at this point in the history
  26. Configuration menu
    Copy the full SHA
    fd0467a View commit details
    Browse the repository at this point in the history
  27. Configuration menu
    Copy the full SHA
    84a0994 View commit details
    Browse the repository at this point in the history
  28. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    dd9b790 View commit details
    Browse the repository at this point in the history
  29. Configuration menu
    Copy the full SHA
    8473a92 View commit details
    Browse the repository at this point in the history
  30. Configuration menu
    Copy the full SHA
    47b0d34 View commit details
    Browse the repository at this point in the history
  31. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    8aa9829 View commit details
    Browse the repository at this point in the history
  32. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    9b71d86 View commit details
    Browse the repository at this point in the history
  33. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    279a981 View commit details
    Browse the repository at this point in the history
  34. Merge pull request #1 from kasinadhsarma/devin/fix-workflow-issues/2745

    Fix GitHub Actions workflow issues
    kasinadhsarma authored Oct 24, 2024
    Configuration menu
    Copy the full SHA
    529f314 View commit details
    Browse the repository at this point in the history
  35. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    e0aa610 View commit details
    Browse the repository at this point in the history
  36. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    513b766 View commit details
    Browse the repository at this point in the history
  37. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    c160d10 View commit details
    Browse the repository at this point in the history
  38. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    f65c814 View commit details
    Browse the repository at this point in the history
  39. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    e836866 View commit details
    Browse the repository at this point in the history
  40. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    f696c4f View commit details
    Browse the repository at this point in the history
  41. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    1d800a8 View commit details
    Browse the repository at this point in the history
  42. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    42cf801 View commit details
    Browse the repository at this point in the history
  43. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    58eb50e View commit details
    Browse the repository at this point in the history
  44. Configuration menu
    Copy the full SHA
    8aac6ba View commit details
    Browse the repository at this point in the history
  45. 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
    devin-ai-integration[bot] committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    2cfb0ff View commit details
    Browse the repository at this point in the history
  46. Merge pull request #3 from kasinadhsarma/devin/fix-workflow-issues/2745

    fix: Implement robust module loading and registration sequence
    kasinadhsarma authored Oct 24, 2024
    Configuration menu
    Copy the full SHA
    3a525a2 View commit details
    Browse the repository at this point in the history