Built-in observation aggregators drop bounds-preserving specifications
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 75/100
Direção de pesquisa
Start by locating base.AGGREGATORS and Updater.observation_spec(), then reproduce the AttributeError and the bounded Generic observable case described in the issue. Check that min, max, mean and median retain scalar or per-element bounds while sum remains unbounded, without changing value, shape/dtype, custom-aggregator or unbounded-input behavior.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Reproduction
Built-in observation aggregators declare a bounds_reserving attribute, while Updater.observation_spec() reads preserves_bounds. Consequently, even min, max, mean and median are treated as having unknown bound behavior: the updater logs a warning and returns an unbounded Array instead of retaining a BoundedArray specification.
The problem is visible directly:
from dm_control.composer.observation.observable import base
assert base.AGGREGATORS['mean'].preserves_bounds is True
# AttributeError: functools.partial has no attribute preserves_bounds
A bounded Generic observable reduced with the built-in mean loses its limits through the actual Updater. Returned values remain in bounds, but the resulting specification no longer rejects out-of-bounds inputs.
Expected behavior
Use the attribute name already documented and consumed by Updater. The four bounds-preserving reducers should retain scalar or per-element limits, while sum must remain unbounded. Preserve reduced values, shape/dtype inference, custom-aggregator behavior and unbounded inputs.
Reproduced on main at a04e3e4cf56c12117d2294bb090f9acec21e5c67, including a native MuJoCo observation sequence. No renderer, external services or model inference is needed.
- Linguagem predominante
- Python
- Estrelas
- 4.7k
- Forks
- 765
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de google-deepmind/dm_control
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
google-deepmind/dm_control#559 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 74/100
google-deepmind/dm_control#557 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 76/100
google-deepmind/dm_control#552 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 45/100
google-deepmind/dm_control#540 · 1 comentário ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 55/100
google-deepmind/dm_control#537 · 1 comentário ·
Todas as issues de google-deepmind/dm_control
Issues semelhantes
-
[Bug] @deck.gl/arcgis dist import resolves to unpublished @deck.gl/core source path (9.3.11, 9.4.0)Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 1 dia
-
workflow: a tick's dispatch counts as 'only this step', and no review self-grants a round unattendedAbertaworkflow
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
kristofdegrave/homeassistant-smart-charging#1505 ·
Mantenedores costumam responder em até 1 dia
-
New Submission: TropWATERAbertametadata submission
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
canonical/content-cache-operator#163 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
[submission]Abertasubmission
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 65/100
leanprover/lean-eval-submissions#1852 ·
Mantenedores costumam responder em até 1 dia