Skip to content

Conversation

@daveey
Copy link
Contributor

@daveey daveey commented Dec 8, 2025

Move SkyDeck from projects/ to packages/ directory

TL;DR

Relocated the SkyDeck dashboard from projects/skydeck to packages/skydeck to better reflect its status as a package.

What changed?

  • Moved all SkyDeck files from projects/skydeck/ to packages/skydeck/
  • Updated all file paths in documentation and code references
  • Updated workspace configuration in root pyproject.toml
  • Updated dependency references in uv.lock

How to test?

  1. Verify SkyDeck installation works with the new path:

    cd packages/skydeck
    uv pip install -e .
  2. Run the dashboard and confirm it works:

    python -m skydeck
  3. Run tests to ensure everything passes:

    cd packages/skydeck
    uv run pytest

Why make this change?

This change better organizes the repository structure by moving SkyDeck from the projects/ directory to the packages/ directory, which more accurately reflects its status as a reusable package rather than a standalone project. This improves discoverability and aligns with the repository's organizational conventions.

Copy link
Contributor Author

daveey commented Dec 8, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@daveey daveey changed the title cp Move skydeck from projects/ to packages/ directory Dec 8, 2025
@datadog-official
Copy link

datadog-official bot commented Dec 8, 2025

⚠️ Tests

⚠️ Warnings

🧪 117 Tests failed

    test_mission_describe[evals.diagnostic_agile] from test_all_games_describe.py

test_mission_describe[evals.diagnostic_agile_hard] from test_all_games_describe.py (Datadog)
assert 'Mission Configuration:' in "1 validation error for AgentConfig\nvibe_transfers\n  Extra inputs are not permitted \n    For further information visit \nhttps://errors.pydantic.dev/2.12/v/extra_forbidden\n\n╭───────────────────┬──────┬───────────┬───────────────────────────────────────╮\n│ Mission           │ Cogs │ Map Size  │ Description                           │\n├───────────────────┼──────┼───────────┼───────────────────────────────────────┤\n│ training_facility │ 1-4  │   13x13   │ COG Training Facility. Basic training │\n│                   │      │           │ facility with open spaces and no      │\n│                   │      │           │ obstacles.                            │\n├───────────────────┼──────┼───────────┼───────────────────────────────────────┤\n│ hello_world       │ 1-20 │  100x100  │ Welcome to space.                     │\n├───────────────────┼──────┼───────────┼───────────────────────────────────────┤\n│ machina_1         │ 1-20 │  200x200  │ Your first mission. Collect resources │\n│                   │      │           │ and assemble HEARTs.                  │\n├───────────────────┼──────┼───────────┼───────────────────────────────────────┤\n│ evals             │ 1-8  │ NonexNone │ Diagnostic evaluation arenas.         │\n╰───────────────────┴──────┴───────────┴───────────────────────────────────────╯\n\nTo set -m:\n  • Use <site>.<mission> (e.g., training_facility.harvest)\n  • Or pass a mission config file path\n  • List a site's missions: cogames missions training_facility\n\nVariants:\n  • Repeat --variant <name> (e.g., --variant solar_flare)\n\nCogs:\n  • --cogs N or -c N\n\nExamples:\n  cogames missions\n  cogames missions training_facility\n  cogames play --mission training_facility.harvest\n  cogames play --mission machina_1.open_world --variant solar_flare --variant \nrough_terrain --cogs 8\n  cogames train --mission <site>.<mission> --cogs 4\n\nUsage: root missions [OPTIONS] [SITE]\n\n\n"
 +  where "1 validation error for AgentConfig\nvibe_transfers\n  Extra inputs are not permitted \n    For further information visit \nhttps://errors.pydantic.dev/2.12/v/extra_forbidden\n\n╭───────────────────┬──────┬───────────┬───────────────────────────────────────╮\n│ Mission           │ Cogs │ Map Size  │ Description                           │\n├───────────────────┼──────┼───────────┼───────────────────────────────────────┤\n│ training_facility │ 1-4  │   13x13   │ COG Training Facility. Basic training │\n│                   │      │           │ facility with open spaces and no      │\n│                   │      │           │ obstacles.                            │\n├───────────────────┼──────┼───────────┼───────────────────────────────────────┤\n│ hello_world       │ 1-20 │  100x100  │ Welcome to space.                     │\n├───────────────────┼──────┼───────────┼───────────────────────────────────────┤\n│ machina_1         │ 1-20 │  200x200  │ Your first mission. Collect resources │\n│                   │      │           │ and assemble HEARTs.                  │\n├───────────────────┼──────┼───────────┼────────────────────...

    test_mission_describe[evals.diagnostic_assemble_seeded_near] from test_all_games_describe.py

View all

ℹ️ Info

❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 287d989 | Docs | Was this helpful? Give us feedback!

@daveey daveey force-pushed the daveey-skydeck branch 4 times, most recently from 884faf0 to 986d61a Compare December 9, 2025 07:41
@daveey daveey changed the base branch from daveey-skytower to graphite-base/4247 December 9, 2025 07:55
@daveey daveey force-pushed the graphite-base/4247 branch from 1438c6c to df04914 Compare December 9, 2025 07:55
@daveey daveey changed the base branch from graphite-base/4247 to daveey-cvc-arena December 9, 2025 07:55
@daveey daveey force-pushed the daveey-skydeck branch 2 times, most recently from eb9d382 to a55fbed Compare December 9, 2025 08:04
@daveey daveey force-pushed the daveey-cvc-arena branch 2 times, most recently from 99d4d0f to 30cdd89 Compare December 9, 2025 20:57
@daveey daveey force-pushed the daveey-skydeck branch 2 times, most recently from a00514c to 18efc34 Compare December 10, 2025 07:15
@daveey daveey changed the base branch from daveey-cvc-arena to graphite-base/4247 December 10, 2025 20:58
@openhands-ai
Copy link

openhands-ai bot commented Dec 10, 2025

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • Validate pyproject.toml
    • Test and Benchmark
    • Validate pyproject.toml

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #4247 at branch `daveey-skydeck`

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

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