Implement separation

CHANGELOG.md

@@ -24,6 +24,11 @@
 - Fix Mekko charts: The main axis handled as dimension. 
 - LabelLevel can be used to handle measure axis as dimension axis.
 
+### Changed
+
+- Separate Channel properties to AxisChannel properties at config.
+- Channels 'set' rewrite doesn't clear AxisChannel properties.
+
 ## [0.15.0] - 2024-10-28
 
 ### Fixed

src/apps/weblib/typeschema-api/config.yaml

@@ -76,6 +76,12 @@ definitions:
                 oneOf:
                     - type: number
                     - { type: string, enum: [auto] }
+
+    AxisChannel:
+        description: |
+            AxisChannels are special channel settings for the positional channels.
+        type: object
+        properties:
             axis:
                 description: Enables the axis line on axis channels.
                 $ref: AutoBool
@@ -109,14 +115,18 @@ definitions:
         type: object
         properties:
             x:
-                $ref: Channel
+                allOf:
+                    - $ref: Channel
+                    - $ref: AxisChannel
                 description: |
                     Parameters for the X-axis, determining the position of the markers on the 
                     x-axis - or their angle when using polar coordinates. 
                     Note: leaving x and y channels empty will result in a 
                     chart "without coordinates" like a Treemap or a Bubble Chart.
             y:
-                $ref: Channel
+                allOf:
+                    - $ref: Channel
+                    - $ref: AxisChannel
                 description: |
                     Parameters for the Y-axis, determining the position of the markers on the 
                     y-axis - or their radius when using polar coordinates) .

src/chart/generator/guides.cpp

@@ -20,16 +20,19 @@ bool GuidesByAxis::operator==(const GuidesByAxis &other) const
 
 Guides::Guides(const Options &options)
 {
-	const auto &xOpt = options.getChannels().at(AxisId::x);
-	const auto &yOpt = options.getChannels().at(AxisId::y);
-	if (xOpt.isEmpty() && yOpt.isEmpty()) return;
+	const auto &xChannel = options.getChannels().at(AxisId::x);
+	const auto &yChannel = options.getChannels().at(AxisId::y);
+	if (xChannel.isEmpty() && yChannel.isEmpty()) return;
 
 	auto xIsMeasure = options.isMeasure(ChannelId::x);
 	auto yIsMeasure = options.isMeasure(ChannelId::y);
 	auto isPolar = options.coordSystem.get() == CoordSystem::polar;
 	auto isCircle = options.geometry.get() == ShapeType::circle;
 	auto isHorizontal = options.isHorizontal();
 
+	const auto &xOpt = options.getChannels().axisPropsAt(AxisId::x);
+	const auto &yOpt = options.getChannels().axisPropsAt(AxisId::y);
+
 	x.axis = xOpt.axis.getValue(yIsMeasure);
 	y.axis = yOpt.axis.getValue(xIsMeasure && !isPolar);
 
@@ -58,7 +61,7 @@ Guides::Guides(const Options &options)
 	x.labels = xOpt.labels.getValue(
 	    (!isPolar || yIsMeasure)
 	    && ((xIsMeasure && (x.axisSticks || x.interlacings))
-	        || (!xIsMeasure && !xOpt.isEmpty())));
+	        || (!xIsMeasure && !xChannel.isEmpty())));
 
 	auto stretchedPolar =
 	    isPolar && !yIsMeasure
@@ -67,7 +70,7 @@ Guides::Guides(const Options &options)
 	y.labels = yOpt.labels.getValue(
 	    !stretchedPolar
 	    && ((yIsMeasure && (y.axisSticks || y.interlacings))
-	        || (!yIsMeasure && !yOpt.isEmpty())));
+	        || (!yIsMeasure && !yChannel.isEmpty())));
 }
 
 GuidesByAxis &Guides::at(AxisId channel)

src/chart/generator/plotbuilder.cpp

@@ -353,7 +353,7 @@ void PlotBuilder::calcLegendAndLabel(const Data::DataTable &dataTable)
 				calcLegend.measure = {std::get<0>(stats.at(type)),
 				    meas->getColIndex(),
 				    dataTable.getUnit(meas->getColIndex()),
-				    scale.step.getValue()};
+				    {1}};
 			}
 		}
 		else if (!scale.isEmpty()) {
@@ -404,6 +404,10 @@ void PlotBuilder::calcAxis(const Data::DataTable &dataTable,
 	const auto &scale = plot->getOptions()->getChannels().at(type);
 	if (scale.isEmpty()) return;
 
+	auto &axisProps =
+	    plot->getOptions()->getChannels().axisPropsAt(type);
+	;
+
 	auto &axis = plot->axises.at(type);
 
 	auto isAutoTitle = scale.title.isAuto();
@@ -419,12 +423,12 @@ void PlotBuilder::calcAxis(const Data::DataTable &dataTable,
 			axis.measure = {Math::Range<>::Raw(0, 100),
 			    meas.getColIndex(),
 			    "%",
-			    scale.step.getValue()};
+			    axisProps.step.getValue()};
 		else
 			axis.measure = {std::get<0>(stats.at(type)),
 			    meas.getColIndex(),
 			    dataTable.getUnit(meas.getColIndex()),
-			    scale.step.getValue()};
+			    axisProps.step.getValue()};
 	}
 	else {
 		for (auto merge =
@@ -449,7 +453,7 @@ void PlotBuilder::calcAxis(const Data::DataTable &dataTable,
 				    merge);
 		}
 		if (auto &&series = plot->getOptions()->labelSeries(type);
-		    !axis.dimension.setLabels(scale.step.getValue(1.0))
+		    !axis.dimension.setLabels(axisProps.step.getValue(1.0))
 		    && series && isAutoTitle)
 			axis.title = series.value().getColIndex();
 	}

src/chart/options/channel.cpp

@@ -151,12 +151,6 @@ void Channel::reset()
 	set = {};
 	labelLevel = {};
 	title = {};
-	axis = Base::AutoBool();
-	labels = Base::AutoBool();
-	ticks = Base::AutoBool();
-	guides = Base::AutoBool();
-	markerGuides = Base::AutoBool();
-	interlacing = Base::AutoBool();
 }
 
 bool Channel::isEmpty() const

src/chart/options/channel.h

@@ -118,9 +118,22 @@ struct ChannelSeriesList
 	bool removeSeries(const Data::SeriesIndex &index);
 };
 
-class Channel
+struct AxisChannelProperties
+{
+	Base::AutoBool axis{};
+	Base::AutoBool labels{};
+	Base::AutoBool ticks{};
+	Base::AutoBool interlacing{};
+	Base::AutoBool guides{};
+	Base::AutoBool markerGuides{};
+	Base::AutoParam<double> step{};
+
+	[[nodiscard]] bool operator==(
+	    const AxisChannelProperties &) const = default;
+};
+
+struct Channel
 {
-public:
 	using OptionalIndex = ChannelSeriesList::OptionalIndex;
 	using IndexSet = std::set<Data::SeriesIndex>;
 	using DimensionIndices = ChannelSeriesList::DimensionIndices;
@@ -161,13 +174,6 @@ class Channel
 	ChannelRange range{};
 	Base::AutoParam<std::size_t> labelLevel{};
 	Base::AutoParam<std::string, true> title{};
-	Base::AutoBool axis{};
-	Base::AutoBool labels{};
-	Base::AutoBool ticks{};
-	Base::AutoBool guides{};
-	Base::AutoBool markerGuides{};
-	Base::AutoBool interlacing{};
-	Base::AutoParam<double> step{};
 };
 
 [[nodiscard]] constexpr ChannelId operator-(const AxisId &axis)

src/chart/options/channels.h

@@ -15,6 +15,7 @@ namespace Vizzu::Gen
 struct Channels : Refl::EnumArray<ChannelId, Channel>
 {
 	using IndexSet = std::set<Data::SeriesIndex>;
+	EnumArray<AxisId, AxisChannelProperties> axisProps;
 
 	[[nodiscard]] bool anyAxisSet() const;
 	[[nodiscard]] bool isEmpty() const;
@@ -37,6 +38,17 @@ struct Channels : Refl::EnumArray<ChannelId, Channel>
 		return at(-id);
 	}
 
+	[[nodiscard]] const AxisChannelProperties &axisPropsAt(
+	    const AxisId &id) const
+	{
+		return axisProps[id];
+	}
+
+	[[nodiscard]] AxisChannelProperties &axisPropsAt(const AxisId &id)
+	{
+		return axisProps[id];
+	}
+
 	void removeSeries(const Data::SeriesIndex &index);
 
 	[[nodiscard]] bool isSeriesUsed(

src/chart/options/config.cpp

@@ -33,6 +33,12 @@ std::string Config::paramsJson()
 		for (const auto &param : channelParams)
 			arr << "channels." + std::string{channelName} + "."
 			           + std::string{param};
+
+		if (channelName == "x" || channelName == "y")
+			for (const auto &param :
+			    getAccessorNames<AxisChannelProperties>())
+				arr << "channels." + std::string{channelName} + "."
+				           + std::string{param};
 	}
 	return res;
 }
@@ -112,6 +118,14 @@ void Config::setChannelParam(const std::string &path,
 
 	if (auto &&accessor = getAccessor<Channel>(property).set)
 		accessor(channel, value);
+	else if (auto &&axisAccessor =
+	             getAccessor<AxisChannelProperties>(property).set;
+	         (channelId == AxisId::x || channelId == AxisId::y)
+	         && axisAccessor)
+		axisAccessor(
+		    options.getChannels().axisPropsAt(
+		        channelId == AxisId::x ? AxisId::x : AxisId::y),
+		    value);
 	else
 		throw std::logic_error(
 		    path + "/" + value
@@ -130,6 +144,11 @@ std::string Config::getChannelParam(const std::string &path) const
 
 	if (auto &&accessor = getAccessor<Channel>(property).get)
 		return accessor(channel);
+	if (auto &&axisAccessor =
+	        getAccessor<AxisChannelProperties>(property).get;
+	    (id == AxisId::x || id == AxisId::y) && axisAccessor)
+		return axisAccessor(options.get().getChannels().axisPropsAt(
+		    id == AxisId::x ? AxisId::x : AxisId::y));
 
 	throw std::logic_error(path + ": invalid channel parameter");
 }

src/chart/options/options.h

@@ -207,7 +207,9 @@ class Options : public OptionProperties
 	    []<std::size_t... Ix>(std::index_sequence<Ix...>)
 	    {
 		    return decltype(channels){
-		        Channel::makeChannel(static_cast<ChannelId>(Ix))...};
+		        {{Channel::makeChannel(
+		            static_cast<ChannelId>(Ix))...}},
+		        {}};
 	    }(std::make_index_sequence<
 	        std::tuple_size_v<decltype(channels)::base_array>>{})};