Skip to content
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

wxGUI/datacalog: distinguish vector map layer type by icon #909

Closed

Conversation

tmszi
Copy link
Member

@tmszi tmszi commented Aug 15, 2020

Describe the solution you'd like

Distinguish vector map layer type by icon (quick and clear visual orientation in vector map layers in the data catalog tree):

  • vector3d 3D vector map layer ('vector3d.png' icon). Modified original vector map layer icon with 3D char.

  • vector_db_disconnect Vector map layer (geometry) is stored in the db, but db connection is refused, db service isn't running ('vector_db_disconnect.png' icon). Modified original vector map layer icon with DB char (DB -> Database), and x char (x -> connection is refused).

  • vector_link_without_src Link external vector map layer, but source path changed/deleted ('vector_link_without_src.png' icon). Modified original vector map layer icon with LK char (LK -> Link), and x char (x -> source path changed/deleted).

Source files (*.xcf format):

vector_icons.tar.gz

Disadvantage of the current implementation of this solution

Slowing down of loading the catalog data tree.

It's up to the discussion to see if such functionality is needed and useful or not.

* 3D vector map layer ('vector_3d.png' icon)
* Vector map layer is stored in the db, but db connection is refused ('vector_db_disconnect.png' icon)
* Link external vector map layer, but source path changed/deleted ('vector_link_without_src.png' icon)
@tmszi tmszi changed the title wxGUI/datacalog: distinguish vector map layer type icon wxGUI/datacalog: distinguish vector map layer type by icon Aug 15, 2020
@tmszi tmszi force-pushed the wxgui_datacalog_distinguish_vector_map_type branch from 811bdde to 1574fb6 Compare August 15, 2020 07:14
@tmszi tmszi marked this pull request as draft August 15, 2020 07:15
@petrasovaa
Copy link
Contributor

I am not sure vector_db_disconnect and vector_link_without_src make too much sense, it seems like very specific and uncommon case. Showing 3D/2D makes sense, and if we decide to show that, we could also add the geometry type (points, lines, areas, mixed). But I am concerned about loading, so not sure whether to recommend certain version of this at all.

@tmszi
Copy link
Member Author

tmszi commented Aug 17, 2020

I am not sure vector_db_disconnect and vector_link_without_src make too much sense, it seems like very specific and uncommon case.

At first I thought only of distinguishing these two types 2D and 3D. However, the problem is with vector map layers that have a stored topology in the database and if the database service is not available, it is not possible to obtain information about whether the vector map layer is 3D or not (topology information coor file which hold info about 2D/3D, is stored in db). That's why I distinguished the vector layer with icons by vector_db_disconnect and vector_link_without_src type (however 3D information is available for the link vector map layer even if the source is changed or removed, but it is also appropriate inform the user that the resource has changed or deleted and that was the reason why I also distinguished this type).

@tmszi tmszi added enhancement New feature or request GUI wxGUI related labels Aug 19, 2020
@neteler neteler added this to the 8.0.1 milestone Dec 9, 2021
@tmszi tmszi modified the milestones: 8.0.1, 8.2.0 Feb 20, 2022
@wenzeslaus wenzeslaus modified the milestones: 8.2.0, 8.4.0 Feb 27, 2022
@wenzeslaus wenzeslaus modified the milestones: 8.3.0, 8.4.0 Feb 10, 2023
@wenzeslaus wenzeslaus modified the milestones: 8.4.0, Future Apr 27, 2024
@github-actions github-actions bot added Python Related code is in Python docs labels Oct 17, 2024
@echoix
Copy link
Member

echoix commented Oct 17, 2024

Solved conflicts

@petrasovaa
Copy link
Contributor

Given the number of open PRs, I will take the liberty to close this one. To me, the disadvantage of slowing down the loading is bigger than the advantage of being to able to visually distinguish 3D vector. That said, I didn't actually measure the slow down and how significant it is, so feel free to reopen if you feel like that wouldn't be an issue.

@petrasovaa petrasovaa closed this Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs enhancement New feature or request GUI wxGUI related Python Related code is in Python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants