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

Port docker utils to swekit #282

Merged
merged 1 commit into from
Jul 10, 2024
Merged

Port docker utils to swekit #282

merged 1 commit into from
Jul 10, 2024

Conversation

angrybayblade
Copy link
Collaborator

No description provided.

Copy link
Contributor

ellipsis-dev bot commented Jul 10, 2024

Your free trial has expired. To keep using Ellipsis, sign up at https://app.ellipsis.dev for $20/seat/month or reach us at help@ellipsis.dev

@kaavee315
Copy link
Contributor

Changes Summary:

  • Objective: This PR ports the Docker utilities to swekit and involves scanning for package data during setup.
  • Type: Refactor
  • Major Changes:
    1. File Modifications:
      • setup.py: Added a function to scan for package data and modified the package_data parameter to utilize this function.
      • utils.py: Adjusted imports to reflect the new structure.
    2. File Renames (moved files to the swekit directory):
      • docker/THIRD-PARTY-LICENSE
      • docker/__init__.py
      • docker/docker/Dockerfile.swe_agent
      • docker_file_generator/const.py
      • docker_file_generator/context_manager.py
      • docker_file_generator/docker_file_generator.py
      • docker_file_generator/evaluate_instance.py
      • docker_file_generator/run_docker.py
      • docker_file_generator/utils.py
      • docker/evaulate_on_docker.py
      • docker/scripts/build_docker_images.sh
      • docker/scripts/docker_push.sh
      • docker/templates/Dockerfile.base
      • docker/templates/Dockerfile.pyenv_instance
      • docker/templates/Dockerfile.swe
      • docker/templates/getconda.sh
      • docker/templates/install_composio.sh

The changes involve refactoring the directory structure for better organization and maintainability by moving docker-related scripts and configuration files under the swekit path. The setup.py adjustments allow dynamic inclusion of package data.

Important Change Files:

  • python/swe/setup.py
  • python/swe/swekit/benchmark/docker/docker_file_generator/*
  • python/swe/swekit/benchmark/utils.py

@angrybayblade angrybayblade merged commit 6d334ff into master Jul 10, 2024
5 checks passed
@angrybayblade angrybayblade deleted the fix/swe-bench-imports branch July 10, 2024 07:11
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.

2 participants