gexf icon indicating copy to clipboard operation
gexf copied to clipboard

Code review: inconsistent thicknesses scale definition

Open Copilot opened this issue 2 months ago • 5 comments

Code Review of GEXF 1.3.1 Legend and Scale Features

Issues Fixed:

  • [x] Inconsistency in thicknesses-scale-content: Updated thicknesses scale to use the new transform element with math functions (pow, sqrt, log, log10, exp, exp10, spline) instead of the deprecated scalepoint approach, matching the sizes implementation
  • [x] Schema files regenerated: Rebuilt XSD and RNG files from RNC sources using trang

Changes Made:

  • Modified specs/1.3.1/_viz.rnc to replace scalepoint with transform in thicknesses-scale-content
  • Regenerated specs/1.3.1/_viz.xsd and specs/1.3.1/_viz.rng from the updated RNC file
  • Minor namespace updates in common.xsd and dynamics.xsd from schema regeneration

Summary:

The implementation now has consistent scale definitions across sizes and thicknesses, both using the transform approach with math functions. Title element successfully added to meta section, and layout changed to ordered list with explicit step attribute.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot avatar Dec 09 '25 16:12 Copilot