-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Group nearby tags #22
Comments
Deduplicate tags from different snapshots on the timeline, and group the nearby ones together and maintain a constant spacing between them. |
Just an idea, but instead of trying to group the tags from the different snapshots, why not have the tags tied to the individual snapshot times. As you change what time you're viewing you change what tags are shown too. In other words, instead of showing all tags across all times for each individual time, only show the tags that belong to that time. IIRC (I'm not on my home PC at the moment), you already have the tags grouped by time in the json data you save, so why not show the tags for the individual times instead of trying to figure out how to group them with some kind of fuzzy logic about how close they have to be to group together. This also opens up the possibility of removing the option "no-tag" from the auto.py script altogether. Instead of making the choice at the time of map creation, move the choice of displaying the tags or not to the browser display. This way they can be turned on and off at will long after the map has been generated. |
Thanks for the feedback, if I ever decide to implement this I will keep this into account, however realistically speaking it's unlikely I'll be implementing this anytime soon as I'm quite burned out on factorio at the moment. (Also quite busy) As a sidenote, I am always open to pull requests :p |
I can certainly understand that. No guarantees of course but I might be able to help out a little bit here and there (my own time allowing). I'm still pulling your mod apart to find out just exactly how it works though. |
By the way, you're always free to message me on discord for explanation or
help if you're stuck on something :)
|
No description provided.
The text was updated successfully, but these errors were encountered: