Skip to content

Commit

Permalink
add work-around to load double sequence [1.0, 1.1]. (#17)
Browse files Browse the repository at this point in the history
Signed-off-by: Tomoya.Fujita <tomoya.fujita825@gmail.com>
  • Loading branch information
fujitatomoya committed Dec 28, 2023
1 parent 5b7f213 commit 5a9959b
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
- [Test](#test)
- [Build (Deprecated)](#build-deprecated)
- [Run](#run-1)
- [Known Issues](#known-issues)
- [Authors](#authors)
- [License](#license)

Expand Down Expand Up @@ -259,6 +260,12 @@ All of the test is listed with result as following
[ros2-2] [INFO] [1601447662.146011312] [client]: j. Test New Added Persistent Parameter Stores To File : PASS
```

## Known Issues

- [Error when loading a persistent parameter of type array<double>](https://github.com/fujitatomoya/ros2_persist_parameter_server/issues/13)
- `[1.0,1.1]` is deduced as `[1, 1.1000000000000001]` because of [yaml-cpp bug](https://github.com/jbeder/yaml-cpp/issues/1016), this will leads to `Failed to parse parameters` exception when loading the persistent parameters from yaml file.
- The work-around is to use string sequence `["1.0", "1.1"]` instead of double type, then change it into `float()` in the program.
## Authors
- **Tomoya Fujita** --- Tomoya.Fujita@sony.com
Expand Down

0 comments on commit 5a9959b

Please sign in to comment.