openpi_liu - #1
Open
Limo-d wants to merge 1 commit into
Open
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
This PR appears to add a pre-built Conda environment snapshot under .docker-build/openpi_liu (package metadata, history, and compiler compatibility stubs), despite the description stating “add own_policy”.
Changes:
- Added many
conda-meta/*.jsonpackage metadata files for theopenpi_liuenvironment. - Added
conda-meta/history/conda-meta/created_atenvironment provenance files. - Added
compiler_compat/*compatibility artifacts inside the build output directory.
Reviewed changes
Copilot reviewed 44 out of 7781 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| .docker-build/openpi_liu/conda-meta/libgl-1.7.0-ha4b6fd6_2.json | Adds generated Conda package metadata (libgl) for the docker build environment snapshot |
| .docker-build/openpi_liu/conda-meta/libgfortran5-15.2.0-h68bc16d_18.json | Adds generated Conda package metadata (libgfortran5) |
| .docker-build/openpi_liu/conda-meta/libgfortran-15.2.0-h69a702a_18.json | Adds generated Conda package metadata (libgfortran) |
| .docker-build/openpi_liu/conda-meta/libgcc-ng-15.2.0-h69a702a_18.json | Adds generated Conda package metadata (libgcc-ng) |
| .docker-build/openpi_liu/conda-meta/libgcc-15.2.0-he0feb66_18.json | Adds generated Conda package metadata (libgcc) |
| .docker-build/openpi_liu/conda-meta/libfreetype6-2.14.1-h73754d4_0.json | Adds generated Conda package metadata (libfreetype6) |
| .docker-build/openpi_liu/conda-meta/libfreetype-2.14.1-ha770c72_0.json | Adds generated Conda package metadata (libfreetype) |
| .docker-build/openpi_liu/conda-meta/libffi-3.5.2-h3435931_0.json | Adds generated Conda package metadata (libffi) |
| .docker-build/openpi_liu/conda-meta/libexpat-2.7.4-hecca717_0.json | Adds generated Conda package metadata (libexpat) |
| .docker-build/openpi_liu/conda-meta/libegl-1.7.0-ha4b6fd6_2.json | Adds generated Conda package metadata (libegl) |
| .docker-build/openpi_liu/conda-meta/libdeflate-1.25-h17f619e_0.json | Adds generated Conda package metadata (libdeflate) |
| .docker-build/openpi_liu/conda-meta/libcblas-3.11.0-5_h0358290_openblas.json | Adds generated Conda package metadata (libcblas) |
| .docker-build/openpi_liu/conda-meta/libblas-3.11.0-5_h4a7cf45_openblas.json | Adds generated Conda package metadata (libblas) |
| .docker-build/openpi_liu/conda-meta/libass-0.17.3-hba53ac1_1.json | Adds generated Conda package metadata (libass) |
| .docker-build/openpi_liu/conda-meta/lerc-4.0.0-h0aef613_1.json | Adds generated Conda package metadata (lerc) |
| .docker-build/openpi_liu/conda-meta/ld_impl_linux-64-2.45.1-default_hbd61a6d_101.json | Adds generated Conda package metadata (linker implementation) |
| .docker-build/openpi_liu/conda-meta/lcms2-2.18-h0c24ade_0.json | Adds generated Conda package metadata (lcms2) |
| .docker-build/openpi_liu/conda-meta/lame-3.100-h166bdaf_1003.json | Adds generated Conda package metadata (lame) |
| .docker-build/openpi_liu/conda-meta/history | Adds generated Conda environment history (commands + resolved packages) |
| .docker-build/openpi_liu/conda-meta/graphite2-1.3.14-hecca717_2.json | Adds generated Conda package metadata (graphite2) |
| .docker-build/openpi_liu/conda-meta/gmp-6.3.0-hac33072_2.json | Adds generated Conda package metadata (gmp) |
| .docker-build/openpi_liu/conda-meta/fribidi-1.0.16-hb03c661_0.json | Adds generated Conda package metadata (fribidi) |
| .docker-build/openpi_liu/conda-meta/freetype-2.14.1-ha770c72_0.json | Adds generated Conda package metadata (freetype) |
| .docker-build/openpi_liu/conda-meta/fonts-conda-forge-1-hc364b38_1.json | Adds generated Conda package metadata (fonts-conda-forge) |
| .docker-build/openpi_liu/conda-meta/fonts-conda-ecosystem-1-0.json | Adds generated Conda package metadata (fonts-conda-ecosystem) |
| .docker-build/openpi_liu/conda-meta/font-ttf-ubuntu-0.83-h77eed37_3.json | Adds generated Conda package metadata (Ubuntu fonts) |
| .docker-build/openpi_liu/conda-meta/font-ttf-source-code-pro-2.038-h77eed37_0.json | Adds generated Conda package metadata (Source Code Pro fonts) |
| .docker-build/openpi_liu/conda-meta/font-ttf-inconsolata-3.000-h77eed37_0.json | Adds generated Conda package metadata (Inconsolata fonts) |
| .docker-build/openpi_liu/conda-meta/font-ttf-dejavu-sans-mono-2.37-hab24e00_0.json | Adds generated Conda package metadata (DejaVu fonts) |
| .docker-build/openpi_liu/conda-meta/dav1d-1.2.1-hd590300_0.json | Adds generated Conda package metadata (dav1d) |
| .docker-build/openpi_liu/conda-meta/created_at | Adds generated environment creation timestamp marker |
| .docker-build/openpi_liu/conda-meta/cairo-1.18.4-h3394656_0.json | Adds generated Conda package metadata (cairo) |
| .docker-build/openpi_liu/conda-meta/ca-certificates-2026.2.25-hbd8a1cb_0.json | Adds generated Conda package metadata (CA cert bundle) |
| .docker-build/openpi_liu/conda-meta/bzip2-1.0.8-hda65f42_9.json | Adds generated Conda package metadata (bzip2) |
| .docker-build/openpi_liu/conda-meta/aom-3.9.1-hac33072_0.json | Adds generated Conda package metadata (aom) |
| .docker-build/openpi_liu/conda-meta/_openmp_mutex-4.5-20_gnu.json | Adds generated Conda package metadata (_openmp_mutex) |
| .docker-build/openpi_liu/compiler_compat/ld | Adds generated compiler compatibility shim file |
| .docker-build/openpi_liu/compiler_compat/README | Adds generated compiler compatibility readme |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+1
to
+3
| ==> 2026-03-03 13:21:02 <== | ||
| # cmd: /new_world/cockatiel/miniconda3/bin/conda create -n openpi_liu python=3.11 ffmpeg av -c conda-forge | ||
| # conda version: 25.11.1 |
There was a problem hiding this comment.
The PR description says “add own_policy”, but the diff only adds a generated Conda environment snapshot (.docker-build/openpi_liu/*). Please either update the PR description to reflect the actual change (vendoring build artifacts) or include the intended own_policy change in this PR.
Comment on lines
+1
to
+30
| ==> 2026-03-03 13:21:02 <== | ||
| # cmd: /new_world/cockatiel/miniconda3/bin/conda create -n openpi_liu python=3.11 ffmpeg av -c conda-forge | ||
| # conda version: 25.11.1 | ||
| +conda-forge/linux-64::_openmp_mutex-4.5-20_gnu | ||
| +conda-forge/linux-64::alsa-lib-1.2.15.3-hb03c661_0 | ||
| +conda-forge/linux-64::aom-3.9.1-hac33072_0 | ||
| +conda-forge/linux-64::attr-2.5.2-h39aace5_0 | ||
| +conda-forge/linux-64::av-16.1.0-py311hf24c605_0 | ||
| +conda-forge/linux-64::bzip2-1.0.8-hda65f42_9 | ||
| +conda-forge/linux-64::cairo-1.18.4-he90730b_1 | ||
| +conda-forge/linux-64::dav1d-1.2.1-hd590300_0 | ||
| +conda-forge/linux-64::dbus-1.16.2-h24cb091_1 | ||
| +conda-forge/linux-64::ffmpeg-8.0.1-gpl_h43fde53_912 | ||
| +conda-forge/linux-64::fontconfig-2.17.1-h27c8c51_0 | ||
| +conda-forge/linux-64::fribidi-1.0.16-hb03c661_0 | ||
| +conda-forge/linux-64::gdk-pixbuf-2.44.5-h2b0a6b4_1 | ||
| +conda-forge/linux-64::glslang-16.2.0-h96af755_1 | ||
| +conda-forge/linux-64::gmp-6.3.0-hac33072_2 | ||
| +conda-forge/linux-64::graphite2-1.3.14-hecca717_2 | ||
| +conda-forge/linux-64::harfbuzz-12.3.2-h6083320_0 | ||
| +conda-forge/linux-64::icu-78.2-h33c6efd_0 | ||
| +conda-forge/linux-64::intel-gmmlib-22.9.0-hb700be7_0 | ||
| +conda-forge/linux-64::intel-media-driver-25.3.4-hecca717_0 | ||
| +conda-forge/linux-64::lame-3.100-h166bdaf_1003 | ||
| +conda-forge/linux-64::lcms2-2.18-h0c24ade_0 | ||
| +conda-forge/linux-64::ld_impl_linux-64-2.45.1-default_hbd61a6d_101 | ||
| +conda-forge/linux-64::lerc-4.0.0-h0aef613_1 | ||
| +conda-forge/linux-64::level-zero-1.28.2-hb700be7_0 | ||
| +conda-forge/linux-64::libabseil-20260107.1-cxx17_h7b12aa8_0 | ||
| +conda-forge/linux-64::libass-0.17.4-h96ad9f0_0 |
There was a problem hiding this comment.
These files look like generated build artifacts (Conda env history + resolved package list). Committing them will create frequent noisy diffs (timestamps, solver output) and makes builds less reproducible across machines/CI. Prefer committing a declarative environment spec (e.g., environment.yml / conda-lock lockfile) and generating the .docker-build/openpi_liu directory during the Docker build; also add .docker-build/openpi_liu/ (or at least conda-meta/ and compiler_compat/) to .gitignore.
Suggested change
| ==> 2026-03-03 13:21:02 <== | |
| # cmd: /new_world/cockatiel/miniconda3/bin/conda create -n openpi_liu python=3.11 ffmpeg av -c conda-forge | |
| # conda version: 25.11.1 | |
| +conda-forge/linux-64::_openmp_mutex-4.5-20_gnu | |
| +conda-forge/linux-64::alsa-lib-1.2.15.3-hb03c661_0 | |
| +conda-forge/linux-64::aom-3.9.1-hac33072_0 | |
| +conda-forge/linux-64::attr-2.5.2-h39aace5_0 | |
| +conda-forge/linux-64::av-16.1.0-py311hf24c605_0 | |
| +conda-forge/linux-64::bzip2-1.0.8-hda65f42_9 | |
| +conda-forge/linux-64::cairo-1.18.4-he90730b_1 | |
| +conda-forge/linux-64::dav1d-1.2.1-hd590300_0 | |
| +conda-forge/linux-64::dbus-1.16.2-h24cb091_1 | |
| +conda-forge/linux-64::ffmpeg-8.0.1-gpl_h43fde53_912 | |
| +conda-forge/linux-64::fontconfig-2.17.1-h27c8c51_0 | |
| +conda-forge/linux-64::fribidi-1.0.16-hb03c661_0 | |
| +conda-forge/linux-64::gdk-pixbuf-2.44.5-h2b0a6b4_1 | |
| +conda-forge/linux-64::glslang-16.2.0-h96af755_1 | |
| +conda-forge/linux-64::gmp-6.3.0-hac33072_2 | |
| +conda-forge/linux-64::graphite2-1.3.14-hecca717_2 | |
| +conda-forge/linux-64::harfbuzz-12.3.2-h6083320_0 | |
| +conda-forge/linux-64::icu-78.2-h33c6efd_0 | |
| +conda-forge/linux-64::intel-gmmlib-22.9.0-hb700be7_0 | |
| +conda-forge/linux-64::intel-media-driver-25.3.4-hecca717_0 | |
| +conda-forge/linux-64::lame-3.100-h166bdaf_1003 | |
| +conda-forge/linux-64::lcms2-2.18-h0c24ade_0 | |
| +conda-forge/linux-64::ld_impl_linux-64-2.45.1-default_hbd61a6d_101 | |
| +conda-forge/linux-64::lerc-4.0.0-h0aef613_1 | |
| +conda-forge/linux-64::level-zero-1.28.2-hb700be7_0 | |
| +conda-forge/linux-64::libabseil-20260107.1-cxx17_h7b12aa8_0 | |
| +conda-forge/linux-64::libass-0.17.4-h96ad9f0_0 | |
| # Generated Conda environment history is intentionally not tracked. | |
| # Recreate this environment during the Docker/build process from the | |
| # project's declarative environment specification instead of committing | |
| # machine-specific timestamps, local command paths, or resolved package logs. |
Comment on lines
+1
to
+35
| ==> 2026-03-03 13:21:02 <== | ||
| # cmd: /new_world/cockatiel/miniconda3/bin/conda create -n openpi_liu python=3.11 ffmpeg av -c conda-forge | ||
| # conda version: 25.11.1 | ||
| +conda-forge/linux-64::_openmp_mutex-4.5-20_gnu | ||
| +conda-forge/linux-64::alsa-lib-1.2.15.3-hb03c661_0 | ||
| +conda-forge/linux-64::aom-3.9.1-hac33072_0 | ||
| +conda-forge/linux-64::attr-2.5.2-h39aace5_0 | ||
| +conda-forge/linux-64::av-16.1.0-py311hf24c605_0 | ||
| +conda-forge/linux-64::bzip2-1.0.8-hda65f42_9 | ||
| +conda-forge/linux-64::cairo-1.18.4-he90730b_1 | ||
| +conda-forge/linux-64::dav1d-1.2.1-hd590300_0 | ||
| +conda-forge/linux-64::dbus-1.16.2-h24cb091_1 | ||
| +conda-forge/linux-64::ffmpeg-8.0.1-gpl_h43fde53_912 | ||
| +conda-forge/linux-64::fontconfig-2.17.1-h27c8c51_0 | ||
| +conda-forge/linux-64::fribidi-1.0.16-hb03c661_0 | ||
| +conda-forge/linux-64::gdk-pixbuf-2.44.5-h2b0a6b4_1 | ||
| +conda-forge/linux-64::glslang-16.2.0-h96af755_1 | ||
| +conda-forge/linux-64::gmp-6.3.0-hac33072_2 | ||
| +conda-forge/linux-64::graphite2-1.3.14-hecca717_2 | ||
| +conda-forge/linux-64::harfbuzz-12.3.2-h6083320_0 | ||
| +conda-forge/linux-64::icu-78.2-h33c6efd_0 | ||
| +conda-forge/linux-64::intel-gmmlib-22.9.0-hb700be7_0 | ||
| +conda-forge/linux-64::intel-media-driver-25.3.4-hecca717_0 | ||
| +conda-forge/linux-64::lame-3.100-h166bdaf_1003 | ||
| +conda-forge/linux-64::lcms2-2.18-h0c24ade_0 | ||
| +conda-forge/linux-64::ld_impl_linux-64-2.45.1-default_hbd61a6d_101 | ||
| +conda-forge/linux-64::lerc-4.0.0-h0aef613_1 | ||
| +conda-forge/linux-64::level-zero-1.28.2-hb700be7_0 | ||
| +conda-forge/linux-64::libabseil-20260107.1-cxx17_h7b12aa8_0 | ||
| +conda-forge/linux-64::libass-0.17.4-h96ad9f0_0 | ||
| +conda-forge/linux-64::libblas-3.11.0-5_h4a7cf45_openblas | ||
| +conda-forge/linux-64::libbrotlicommon-1.2.0-hb03c661_1 | ||
| +conda-forge/linux-64::libbrotlidec-1.2.0-hb03c661_1 | ||
| +conda-forge/linux-64::libbrotlienc-1.2.0-hb03c661_1 | ||
| +conda-forge/linux-64::libcap-2.77-h3ff7636_0 |
There was a problem hiding this comment.
These files look like generated build artifacts (Conda env history + resolved package list). Committing them will create frequent noisy diffs (timestamps, solver output) and makes builds less reproducible across machines/CI. Prefer committing a declarative environment spec (e.g., environment.yml / conda-lock lockfile) and generating the .docker-build/openpi_liu directory during the Docker build; also add .docker-build/openpi_liu/ (or at least conda-meta/ and compiler_compat/) to .gitignore.
Suggested change
| ==> 2026-03-03 13:21:02 <== | |
| # cmd: /new_world/cockatiel/miniconda3/bin/conda create -n openpi_liu python=3.11 ffmpeg av -c conda-forge | |
| # conda version: 25.11.1 | |
| +conda-forge/linux-64::_openmp_mutex-4.5-20_gnu | |
| +conda-forge/linux-64::alsa-lib-1.2.15.3-hb03c661_0 | |
| +conda-forge/linux-64::aom-3.9.1-hac33072_0 | |
| +conda-forge/linux-64::attr-2.5.2-h39aace5_0 | |
| +conda-forge/linux-64::av-16.1.0-py311hf24c605_0 | |
| +conda-forge/linux-64::bzip2-1.0.8-hda65f42_9 | |
| +conda-forge/linux-64::cairo-1.18.4-he90730b_1 | |
| +conda-forge/linux-64::dav1d-1.2.1-hd590300_0 | |
| +conda-forge/linux-64::dbus-1.16.2-h24cb091_1 | |
| +conda-forge/linux-64::ffmpeg-8.0.1-gpl_h43fde53_912 | |
| +conda-forge/linux-64::fontconfig-2.17.1-h27c8c51_0 | |
| +conda-forge/linux-64::fribidi-1.0.16-hb03c661_0 | |
| +conda-forge/linux-64::gdk-pixbuf-2.44.5-h2b0a6b4_1 | |
| +conda-forge/linux-64::glslang-16.2.0-h96af755_1 | |
| +conda-forge/linux-64::gmp-6.3.0-hac33072_2 | |
| +conda-forge/linux-64::graphite2-1.3.14-hecca717_2 | |
| +conda-forge/linux-64::harfbuzz-12.3.2-h6083320_0 | |
| +conda-forge/linux-64::icu-78.2-h33c6efd_0 | |
| +conda-forge/linux-64::intel-gmmlib-22.9.0-hb700be7_0 | |
| +conda-forge/linux-64::intel-media-driver-25.3.4-hecca717_0 | |
| +conda-forge/linux-64::lame-3.100-h166bdaf_1003 | |
| +conda-forge/linux-64::lcms2-2.18-h0c24ade_0 | |
| +conda-forge/linux-64::ld_impl_linux-64-2.45.1-default_hbd61a6d_101 | |
| +conda-forge/linux-64::lerc-4.0.0-h0aef613_1 | |
| +conda-forge/linux-64::level-zero-1.28.2-hb700be7_0 | |
| +conda-forge/linux-64::libabseil-20260107.1-cxx17_h7b12aa8_0 | |
| +conda-forge/linux-64::libass-0.17.4-h96ad9f0_0 | |
| +conda-forge/linux-64::libblas-3.11.0-5_h4a7cf45_openblas | |
| +conda-forge/linux-64::libbrotlicommon-1.2.0-hb03c661_1 | |
| +conda-forge/linux-64::libbrotlidec-1.2.0-hb03c661_1 | |
| +conda-forge/linux-64::libbrotlienc-1.2.0-hb03c661_1 | |
| +conda-forge/linux-64::libcap-2.77-h3ff7636_0 | |
| # This file is a generated Conda build artifact and should not store | |
| # committed environment history, timestamps, or solver-resolved package output. | |
| # Regenerate `.docker-build/openpi_liu/conda-meta/history` during the Docker/build step | |
| # from a declarative environment specification instead of tracking it in source control. |
Comment on lines
+21
to
+23
| "requested_spec": "None", | ||
| "package_tarball_full_path": "/new_world/cockatiel/miniconda3/pkgs/libgl-1.7.0-ha4b6fd6_2.conda", | ||
| "extracted_package_dir": "/new_world/cockatiel/miniconda3/pkgs/libgl-1.7.0-ha4b6fd6_2", |
There was a problem hiding this comment.
The committed Conda metadata embeds machine-specific absolute paths (e.g., /new_world/cockatiel/miniconda3/...) and other environment-local details. This is likely to break portability and will churn whenever the environment is rebuilt elsewhere. If you need reproducibility, store a platform-appropriate lockfile and rebuild the environment in CI/Docker instead of committing conda-meta outputs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
add own_policy