Update README in tools to make the example implemenation working again

[AndreHauschild]

Makefile modification not necessary any more as content of tools folder is inlcuded via wildcard.

Adapted to changes in DrawCube method and use DrawCubeV instead.

Added missing element in tools_registry.cpp entry.

[AndreHauschild]

Dear development team,

first of all: WOW! This tool is awesome. Beautiful visualization and excellent performance even when showing hundreds of satellites, that is rare to find. Thank you SO MUCH for making this tool available to the public.

I am currently working on a plugin for a very specialized analysis, which may never be relevant enough for other users to enter the main version. I have discovered that you have provided the perfect framework for that by providing the option to add self-contained tools (WOW!). When working through your excellent documentation for it (again: WOW!), I noticed that the example was not quite working out of the box, so I made some modifications to adjust for changes that must have happened after the last update of the README. I hope you find them useful.

There is one remaining issue, which I was not able to figure out: after the first build (and each clean build following make clean) with the additional tool, I noticed that the app comes up with the satellite manager tool being displayed both in the left and right panel. I am able to click this away in the menu which puts the right configuration in the settings.json, but I suspect that the initial settings are written incorrectly. I launch the app from dist/TLEscope-Linux-Portable/ and build on Ubuntu 24.04.5 LTS.

I have one more suggestion: it is not so obvious how easy it is for users to extend the functionality of TLEscope with these modular tools. Maybe a link to the corresponding README in the main README would help?

Thanks a lot and all the best!

[aweeri]

Hi! first of all, thank you for your contribution and nice words ^^

TLEscope has been in the making for a long time and it makes me really glad to see people appreciate it this much.

I've had a small break because of uni, but as far as I remember the issue with the duplicated satellite manager stems from an unchanged number corresponding to the max panels count. the manager is ID 0, which makes some part of the code loop back around to display it again. this of course it not indended and will be corrected.

Thank you for your feedback on the documentation as well. I will move things around as suggested :)

[aweeri]

The aforementioned issue has been addressed :P