Skip to content

Conversation

@LeiGuo622
Copy link
Contributor

  • Added Dockerfile based on python:3.10-slim with necessary dependencies (git, ctags, docker-cli, build-essential, zlib).
  • Added .dockerignore to optimize build context.
  • Updated README.md and README.zh-CN.md with Docker build and run instructions.

- Added Dockerfile based on python:3.10-slim with necessary dependencies (git, ctags, docker-cli, build-essential, zlib).
- Added .dockerignore to optimize build context.
- Updated README.md with Docker build and run instructions.
@mudongliang mudongliang requested a review from hai119 January 27, 2026 15:16
```shell
# Ensure you mount the necessary directories (code, config, datasets)
# Example: assuming config.yml is in current dir and datasets are in /data
docker run --rm -v $(pwd):/app/src -v /path/to/dataset:/path/to/dataset patch-backporting python backporting.py --config config.yml
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could also use docker run -it patch-backporting /bin/bash and execute scripts in docker.
Please add it to README.

```shell
# 请确保挂载了必要的目录(项目代码、配置文件、数据集)
# 示例:假设当前目录下有 config.yml,数据集位于 /path/to/dataset
docker run --rm -v $(pwd):/app/src -v /path/to/dataset:/path/to/dataset patch-backporting python backporting.py --config config.yml
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same to README.en

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

README has been updated with the interactive mode instructions. Thanks!

@LeiGuo622 LeiGuo622 requested a review from hai119 January 29, 2026 09:17
@hai119
Copy link
Collaborator

hai119 commented Jan 29, 2026

Merged. Also closed #5 .

@hai119 hai119 merged commit 1ac97fb into OS3Lab:master Jan 29, 2026
1 check passed
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