Skip to content

Commit

Permalink
[CI] re-activate the config test after upstream sim fix (facebookrese…
Browse files Browse the repository at this point in the history
…arch#1915)

* re-activate the test after upstream sim fix
  • Loading branch information
aclegg3 authored and dannymcy committed Jun 26, 2024
1 parent c970bbb commit 52eba41
Showing 1 changed file with 12 additions and 5 deletions.
17 changes: 12 additions & 5 deletions test/test_sim_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -547,6 +547,15 @@ def test_ao_open_close_queries():
sim.metadata_mediator.ao_template_manager.register_template(
fridge_template, "new_fridge_template"
)
new_fridge_template_check = (
sim.metadata_mediator.ao_template_manager.get_template_by_handle(
"new_fridge_template"
)
)
assert (
new_fridge_template_check.get_user_config().get("default_link")
== 0
)
new_fridge = sim.get_articulated_object_manager().add_articulated_object_by_template_handle(
"new_fridge_template"
)
Expand All @@ -558,11 +567,9 @@ def test_ao_open_close_queries():
new_default_link = sutils.get_ao_default_link(
new_fridge, compute_if_not_found=True
)
print(
f" new_default_link (== {new_default_link}) should be 0, waiting on sim bug fix."
)
# TODO: habitat-sim bug. "default_link" does not get copied over after instantiation if set in the template programmatically.
# assert new_default_link == 0

# "default_link" should get copied over after instantiation if set in the template programmatically.
assert new_default_link == 0

# test setting the default link in instance metadata
fridge.user_attributes.set("default_link", 0)
Expand Down

0 comments on commit 52eba41

Please sign in to comment.