Add observer_agent_id parameter to obs_features for agent-specific observations

[daveey]

TL;DR

Added an observer_agent_id parameter to the obs_features() method to support agent-specific observations.

What changed?

• Modified the GridObject::obs_features() virtual method to accept an optional observer_agent_id parameter (defaulting to UINT_MAX to indicate no specific observer)
• Added documentation explaining that this parameter is used by Assembler to report agent-specific cooldowns
• Updated all derived class implementations of obs_features() to include the new parameter
• Added (void)observer_agent_id to implementations that don't use the parameter to prevent unused variable warnings
• Included <climits> header to support the UINT_MAX constant

How to test?

• Verify that all existing code using obs_features() continues to work without changes
• Create a test case where an Assembler reports different cooldown values to different agents
• Ensure that agent-specific observations are correctly generated when the parameter is used

Why make this change?

This change enables objects to provide different observation features depending on which agent is observing them. This is particularly useful for Assemblers that need to report agent-specific cooldown information, allowing for more personalized and accurate observations in the grid environment.

[daveey]

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

• Add observer_agent_id parameter to obs_features for agent-specific observations #4539 👈 (View in Graphite)
• Add Area of Effect (AOE) resource effects to grid objects #4538
• Add CommonsChest for shared inventory access across agents #4537
• Add Commons system for shared inventories between grid objects #4536
• Rename cog_arena.py to cogsguard.py and remove unused imports #4477
• Enhance attack system with armor, weapons, and configurable loot #4313 : 1 other dependent PR (#4321 )
• Make Transfer action optional and add vibe-triggered attack #4315
• Implement bidirectional resource transfer system with vibe-based triggers #4314
• Replace number_of_vibes with explicit vibes list in change_vibe action config #4489
• Refactor inventory system into dedicated InventoryConfig class #4356 : 2 other dependent PRs (#4328 , #4355 )
• Add ignore_limits parameter to inventory updates for initial setup #4360
• Rename test_performance to run_performance in mettagrid to avoid running in pytest #4370
• main

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

[openhands-ai]

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

• GitHub Actions are failing:
  • Test and Benchmark

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

@OpenHands please fix the failing actions on PR #4539 at branch `daveey-observer-obs`

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

_{^{You can manage your notification settings}}