- changed version to 3.2.0
All print function need get a parameter ostream
All print function need get a parameter ostream. For easy print to file or other ostream like string stream.
If not give parameter of ostream, it will use std::cout. (default parameter).
Comments (8)
-
reporter -
I'm thinking it might be better to have stream operators for every class and then reduce print as calling output on stdout (and maybe even retire it). Comments?
-
reporter I thinks this solution is also good. Can you do it for gtsam 3.2.0?
-
GTSAM Is an open source project. If you want to add some functionality, you can fork and create a pull request. It might be good to think about how to do this so we can cherry pick the changes into develop as well.
-
-
assigned issue to
- edited description
-
assigned issue to
-
- changed status to wontfix
Thanks for the feature request. We would like to see this feature soon as well and will probably be a part of GTSAM 5.
-
reporter Why not fix?
Why not added to gtsam3 and gtsam4?
-
Actually, the feature I'd like to see is just a stream output operator that replaces print, as of GTSAM 5. That could be added now, if you want. Since GTSAM is open source, you could create a PR for a few of the basic types as a design review, and then complete the PR for all types after comments.
- Log in to comment