yarp icon indicating copy to clipboard operation
yarp copied to clipboard

Allow chaining `yarp::sig::VectorOf<T>` to LogStream

Open PeterBowman opened this issue 3 years ago • 2 comments
trafficstars

Follow-up to https://github.com/robotology/yarp/pull/2758. Similarly to STL containers, C arrays, pairs and tuples, now yarp::sig::VectorOf<T> is also chainable to stream logging macros, e.g.:

yarp::sig::Vector v {1.23, 5.0};
yInfo() << v;

(Disclaimer: one can obviously call v.toString() instead; this patch harmonizes YARP and STL vectors, though.)

PeterBowman avatar Jul 17 '22 09:07 PeterBowman

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would update the release notes by adding a file in doc/release/<target_branch>, based on your changes.

update-docs[bot] avatar Jul 17 '22 09:07 update-docs[bot]

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would update the release notes by adding a file in doc/release/<target_branch>, based on your changes.

I would very much like to reflect this change somewhere, but the doc/release/master/ directory is missing. I think it should be restored as release notes are quite an important tool for developers aiming to adapt their code to newer YARP releases.

Also, I noticed the version is no longer updated for master (experimental) branch:

https://github.com/robotology/yarp/blob/05ca89400b36bf26aea8cb4707f3029d14ccbda7/CMakeLists.txt#L18-L22

PeterBowman avatar Jul 17 '22 09:07 PeterBowman

I would very much like to reflect this change somewhere (...)

Done, I dropped a line in 000_master.md.

PeterBowman avatar Sep 13 '22 10:09 PeterBowman