Skip to content

feat: make building the docker image configurable.#94

Merged
hiverge-robot merged 1 commit into
mainfrom
1uc/docker-config
Feb 23, 2026
Merged

feat: make building the docker image configurable.#94
hiverge-robot merged 1 commit into
mainfrom
1uc/docker-config

Conversation

@luc-hiverge

Copy link
Copy Markdown
Contributor

This commit adds a new top-level section to the experiment config:

docker:
  dockerfile: exp1/Dockerfile
  verbose: true

which enables the user to pick the Dockerfile for the baseimage of the sandbox. It also allows making the build-process verbose, i.e. --process=plain.

This commit adds a new top-level section to the experiment config:

    docker:
      dockerfile: exp1/Dockerfile
      verbose: true

which enables the user to pick the Dockerfile for the baseimage of the
sandbox. It also allows making the build-process verbose, i.e.
`--process=plain`.
@hiverge-robot hiverge-robot added needs-triage Indicates an issue or PR lacks a label and requires one. needs-priority Indicates a PR lacks a label and requires one. do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Feb 19, 2026
@luc-hiverge

Copy link
Copy Markdown
Contributor Author

This allows me to create an "overlay" style setup:

$ ls
checkpoints  experiments  snapshots  src   tools

where the source code to be optimized lives in src (it's "almost" unmodified upstream fairchem).

Then I can create different experiments like this:

$ tree experiments
experiments
├── uma
│   ├── Dockerfile
│   ├── evaluate.py
│   ├── hive.yaml
│   ├── README.md
│   ├── run_benchmark.sh
│   └── uma-speed.yaml
└── wigner
    ├── Dockerfile
    ├── evaluate.py
    ├── hive.yaml
    └── README.md

Then the dockerfiles do:

COPY src /app/repo
COPY experiment/uma/... /app/repo

(the overlay system).

Then I launch experiments as follows:

$ cat experiements/wigner/hive.yaml
project_name: wigner
coordinator_config_name: lucg-uma
docker:
  dockerfile: experiments/wigner/Dockerfile
runtime:
  num_agents: 5
  ...
$ hive create exp uma-wigner- -f experiments/wigner/hive.yaml 

@brp-hiverge

Copy link
Copy Markdown
Contributor

/lgtm

@hiverge-robot hiverge-robot added the lgtm Looks good to me, indicates that a PR is ready to be merged. label Feb 19, 2026
@kerry-hiverge

Copy link
Copy Markdown
Contributor

/kind feature

@hiverge-robot hiverge-robot added feature Categorizes issue or PR as related to a new feature. and removed do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Feb 23, 2026
@kerry-hiverge

Copy link
Copy Markdown
Contributor

/approve

@hiverge-robot hiverge-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 23, 2026
@hiverge-robot hiverge-robot merged commit adfcac8 into main Feb 23, 2026
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. feature Categorizes issue or PR as related to a new feature. lgtm Looks good to me, indicates that a PR is ready to be merged. needs-priority Indicates a PR lacks a label and requires one. needs-triage Indicates an issue or PR lacks a label and requires one.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants