Add observer_agent_id parameter to obs_features method

[daveey]

Adds dynamic pricing to assembler protocols

• Implements a two-phase cost adjustment system for protocols:
  • sigmoid: Number of discounted uses where cost scales linearly from 0 (free) to 1 (full price)
  • inflation: Compound rate for exponential cost adjustment after sigmoid uses
• Adds get_cost_multiplier() method to calculate the appropriate price multiplier based on activation count
• Tracks protocol usage with activation_count counter
• Updates Python type definitions and configuration parsers to support the new parameters

[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 assembler per-agent cooldown #4386
• Add observer_agent_id parameter to obs_features method #4384 👈 (View in Graphite)
• Add building demolition functionality to MettaGrid #4323
• Add build action to mettagrid #4321
• Enhance attack system with armor, weapons, and configurable loot #4313 : 1 other dependent PR (#4477 )
• Make Transfer action optional and add vibe-triggered attack #4315
• Implement bidirectional resource transfer system with vibe-based triggers #4314
• feat: support metta://policy/<short_name_or_class_path> URIs for built-in policies #4561
• feat: allow negative inventory_regen_amounts for decay #4564
• Add clarification about when to run install commands #4563
• main

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

[graphite-app]

This line causes CI to fail because the method signature with the new 'observer_agent_id' parameter doesn't match the base class 'GridObject' method signature. The base class 'GridObject' needs to be updated to have the same method signature: 'virtual std::vector obs_features(unsigned int observer_agent_id = UINT_MAX) const;'

Spotted by Graphite Agent (based on CI logs)



Is this helpful? React 👍 or 👎 to let us know.

[graphite-app]

These lines cause CI to fail because the method signature with the new 'observer_agent_id' parameter doesn't match the base class 'GridObject' method signature. The base class needs to be updated to include this parameter.

Spotted by Graphite Agent (based on CI logs)



Is this helpful? React 👍 or 👎 to let us know.

[graphite-app]

These lines cause CI to fail because the method signature with the new 'observer_agent_id' parameter doesn't match the base class 'GridObject' method signature. The base class needs to be updated to include this parameter.

Spotted by Graphite Agent (based on CI logs)



Is this helpful? React 👍 or 👎 to let us know.

[graphite-app]

This line causes a compilation error because the method signature with the new parameter (observer_agent_id) doesn't match the base class method signature. Either update the base class method to include this parameter or remove the 'override' keyword.

Spotted by Graphite Agent (based on CI logs)



Is this helpful? React 👍 or 👎 to let us know.

[openhands-ai]

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

• GitHub Actions are failing:
  • Build and Deploy to WASM Mettascope

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

@OpenHands please fix the failing actions on PR #4384 at branch `daveey-protocol-pricing`

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

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