fix load_parameter_dict_from_yaml test failure on windows#956
fix load_parameter_dict_from_yaml test failure on windows#956
Conversation
Signed-off-by: Brian Chen <brian.chen@openrobotics.org>
Signed-off-by: Brian Chen <brian.chen@openrobotics.org>
|
Basically I believe that each PR should have complete fix w/o build break for maintenance. this could be problem when we do cherry-pick. @clalancette @Blast545 what do you think? |
|
I think that would be good too, especially since I forgot to hit squash merge on the original PR 😅. But reverting would involve rewriting history which wouldn't be good |
Not quite. We can do a I don't feel strongly about it either way, but I would strongly prefer that by end of day we either a) get the revert in, or b) get this fix in (I notice that it is failing all CI). |
|
I don't have any preference as well, @fujitatomoya maybe just do what you think is best? I passed in the wrong argument to CI, try these ones instead: |
|
@fujitatomoya I've created a revert PR at #958 and I will submit a new PR with the new feature & all the fixes included |
See discussion @ #956 Signed-off-by: Brian Chen <brian.chen@openrobotics.org>
Resolves windows test failure for
load_parameter_dict_from_yaml_filecaused by windows handling of tempfiles. (See: #945 (comment))I've also bundled in a small fix in this PR to change the default depth of
list_parametersto the full depth instead of 1 which to me is a saner default.