chore: enhance flampy dependencies.

[k82cn]

No description provided.

[gemini-code-assist]

Summary of Changes

Hello @k82cn, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request focuses on optimizing Python dependency management and caching for 'flamepy' within the system's Docker environments and the 'flmadm' installation process. The changes aim to improve startup performance and reliability by strategically pre-warming 'uv' caches and ensuring correct environment variable precedence, particularly for Python package installations and execution.

Highlights

• Dependency Cache Pre-warming: Implemented pre-warming of the 'uv' cache for 'flamepy' dependencies within Docker images ('Dockerfile.console', 'Dockerfile.fem') to optimize application startup and runtime performance.
• Cross-Filesystem Compatibility: Introduced 'UV_LINK_MODE=copy' to 'uv pip install' commands in Dockerfiles to ensure robust operation across different filesystem types, preventing potential issues with hard links.
• Enhanced 'flmadm' Caching Logic: Refactored 'flmadm's 'flamepy' dependency caching into a two-phase process: initial caching via 'uv pip install' and subsequent pre-warming of the 'uv run' ephemeral environment cache for optimal 'flmrun' startup.
• Environment Variable Precedence: Modified environment variable handling in 'host_shim.rs' to ensure that application-specific environment variables, such as 'UV_CACHE_DIR', take precedence over default settings.

Changelog

• docker/Dockerfile.console
  • Added 'UV_LINK_MODE=copy' to the 'uv pip install' command for cross-filesystem compatibility.
  • Introduced a new 'RUN' command to pre-warm the 'uv' cache for 'flamepy' dependencies in the runtime environment.
• docker/Dockerfile.fem
  • Added a comment to the 'flmadm install' step, noting that Python SDK cache warming is deferred to the runtime stage due to differing Python versions.
  • Inserted a new 'RUN' command in the runtime stage to pre-warm the 'uv' cache for 'flamepy' dependencies, ensuring it matches the actual Python version.
• executor_manager/src/shims/host_shim.rs
  • Updated the logic for extending environment variables, ensuring that existing application-specific variables are not overwritten by defaults from the working directory setup.
• flmadm/src/managers/installation.rs
  • Refactored the 'flamepy' dependency caching into two distinct phases: initial caching via 'uv pip install' and subsequent pre-warming of the 'uv run' ephemeral environment cache.
  • Updated comments to clarify the caching process and added a new print statement for the pre-warming phase.

Activity

• No activity information was provided for this pull request.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

This pull request enhances flamepy dependency management by introducing uv cache warming to improve performance. The changes across the Dockerfiles and flmadm ensure that Python dependencies are cached correctly for different environments. The modification in host_shim.rs correctly prioritizes application-specific environment variables over defaults. My review includes suggestions to optimize the Dockerfile by combining layers and using wheels consistently, and to simplify the cache warming logic in flmadm by avoiding the use of temporary files.