Fix DSL UQ ergonomics and harden quickstart CI apt install step#1354
Fix DSL UQ ergonomics and harden quickstart CI apt install step#1354
Conversation
Agent-Logs-Url: https://github.com/SkBlaz/py3plex/sessions/e9718885-fa09-4e0b-b9f5-6951ed6cbead Co-authored-by: SkBlaz <10035780+SkBlaz@users.noreply.github.com>
Agent-Logs-Url: https://github.com/SkBlaz/py3plex/sessions/e9718885-fa09-4e0b-b9f5-6951ed6cbead Co-authored-by: SkBlaz <10035780+SkBlaz@users.noreply.github.com>
Agent-Logs-Url: https://github.com/SkBlaz/py3plex/sessions/e9718885-fa09-4e0b-b9f5-6951ed6cbead Co-authored-by: SkBlaz <10035780+SkBlaz@users.noreply.github.com>
Agent-Logs-Url: https://github.com/SkBlaz/py3plex/sessions/e9718885-fa09-4e0b-b9f5-6951ed6cbead Co-authored-by: SkBlaz <10035780+SkBlaz@users.noreply.github.com>
Agent-Logs-Url: https://github.com/SkBlaz/py3plex/sessions/e9718885-fa09-4e0b-b9f5-6951ed6cbead Co-authored-by: SkBlaz <10035780+SkBlaz@users.noreply.github.com>
|
@copilot The failure is in “Install system dependencies” and is caused by
Recommended fixMake apt non-interactive, add retries/timeouts, and increase that step timeout slightly. - name: Install system dependencies
run: |
sudo mkdir -p /etc/apt/apt.conf.d
echo 'Acquire::Retries "3";' | sudo tee /etc/apt/apt.conf.d/80-retries
echo 'Acquire::http::Timeout "30";' | sudo tee /etc/apt/apt.conf.d/80-timeout
echo 'Acquire::https::Timeout "30";' | sudo tee -a /etc/apt/apt.conf.d/80-timeout
sudo DEBIAN_FRONTEND=noninteractive apt-get update -y
sudo DEBIAN_FRONTEND=noninteractive apt-get install -y --no-install-recommends \
gcc g++ build-essential
timeout-minutes: 10Optional optimization (best for this workflow)You likely don’t need this step at all for quickstart validation (manylinux wheels usually cover So the two practical paths are:
|
Agent-Logs-Url: https://github.com/SkBlaz/py3plex/sessions/0b62a529-65d2-484a-84db-8f32eadc19da Co-authored-by: SkBlaz <10035780+SkBlaz@users.noreply.github.com>
Applied the workflow hardening in |
uncertainty=False over query-level .uq()
Query-level
.uq(...)was leaking into metrics that were explicitly configured as deterministic viacompute(..., uncertainty=False). This broke expected per-metric override behavior in mixed deterministic/uncertainty queries.Per-metric UQ override semantics
None: not specified at metric levelTrue: explicitly enabledFalse: explicitly disabledUQ resolution precedence fix
uncertainty_explicit=Falseas an explicit metric-level opt-out from query-level.uq(...)when no metric-level UQ params are provided.method,n_samples) still take precedence.Code-level clarity
CI hardening (from PR feedback)
.github/workflows/tutorial-validation.ymlto reduce apt mirror flakiness inInstall system dependencies:Acquire::Retries "3")Acquire::http::Timeout "30",Acquire::https::Timeout "30")--no-install-recommends