Opened 6 weeks ago

Last modified 6 weeks ago

#24678 closed enhancement

[PATCH] Possibility to specify custom MapPaintSettings for MapCSS rendering. — at Initial Version

Reported by: zkir Owned by: team
Priority: normal Milestone: 26.03
Component: Core Version:
Keywords: MapCSS, MapPaintSettings, StyledMapRenderer, UrbanEye3D Cc:

Description

Problem

The UrbanEye3D plugin provides a 3D rendering of OSM data. To display the ground, it uses JOSM's RenderingHelper and StyledMapRenderer to render a 2D map image(s) with a custom MapCSS style.

The problem is, that the global JOSM preferences from OSM Data tab (e.g. "Draw oneway arrows", "DrawSegment numbers", mappaint.shownames ), are affecting rendering in 3D window also.
This is not a desired behaviour, and it looks quite surprising. Style for the groundplane in 3D window was designed to look "realistic", and all titles and arrows were excluded from it.
However, they reappear if the settings in question are activated. (see screenshots).

Proposed Solution

The proposed solution introduces a mechanism to create and use a "neutral" set of MapPaintSettings for the rendering process, ensuring it is independent of global user preferences.
The patch is composed of three main changes:

  1. In MapPaintSettings.java:
    • A new public constructor is added to allow for the creation of custom MapPaintSettings instances.
    • A new static factory method, createNeutralSettings(), is introduced. This method returns a settings object specifically configured to be non-intrusive for realistic rendering. It disables settings that add symbolic cartographic elements (e.g., showDirectionArrow=false, showOnewayArrow=false, showNamesDistance=0), while inheriting other defaults (like colors and node sizes) from the global INSTANCE.
  1. In StyledMapRenderer.java:
    • A new constructor is added that accepts a MapPaintSettings object.
    • The getSettings() method is updated to use these custom settings if they are provided. If no custom settings are passed, it falls back to the global MapPaintSettings.INSTANCE to maintain existing behavior.
  1. In RenderingHelper.java:
    • The class is updated to take advantage of the new mechanism. It now instantiates StyledMapRenderer with the neutral settings provided by MapPaintSettings.createNeutralSettings().

Performance and memory impact

  • No negative performance impact. No addtional operations or activites are added.
  • Additional instance of MapPaintSettings is created, but only if RenderingHelper is used. In normal scenario there is no additional memory consumption.

Change History (2)

by zkir, 6 weeks ago

Attachment: patch-24678.diff added

by zkir, 6 weeks ago

Attachment: 24678-pic.png added
Note: See TracTickets for help on using tickets.