Repository navigation
TimeProfile addition/subtraction #91
Description
Can support for addition and subtraction for TimeProfile's be added?
One could implement this by
+(a::FixedProfile{T}, b::FixedProfile{T}) where {T} = FixedProfile(a.val + b.val)
function +(a::OperationalProfile{T}, b::OperationalProfile{T}) where {T}
return OperationalProfile(a.vals .+ b.vals)
end
function +(a::StrategicProfile{T}, b::StrategicProfile{T}) where {T}
return StrategicProfile(a.vals .+ b.vals)
end
function +(a::ScenarioProfile{T}, b::ScenarioProfile{T}) where {T}
return ScenarioProfile(a.vals .+ b.vals)
end
function +(a::RepresentativeProfile{T}, b::RepresentativeProfile{T}) where {T}
return RepresentativeProfile(a.vals .+ b.vals)
end
-(a::TimeProfile{T}, b::TimeProfile{T}) where {T} = a + (-b)or one could implement length(::TimeProfile) and iterate(::TimeProfile) with the .+ and .- convention.
Activity
JulStraus commentedon Mar 23, 2026
The proposed approach would work at least the way I see it. What are your thoughts @trulsf ? I do not see any issue with incorporating these, except for cases in which the profiles have different lengths.
In this case, we would need to have also
@assertto check that the profiles have the same length. The error message for different length is:Do we consider the error message to be sufficiently descriptive?
trulsf commentedon Mar 24, 2026
I have also felt had the need for this quite recently, so maybe there is a need to add it.
For me the logical behavior is to ensure that
With the current practice of handling too short operational profiles by repeating the last entry, we need to consider this also when adding, e.g. we should have
We should also think through whether we should support combinations of profiles like
OperationalProfile + FixedProfile.There may also be some challenges if presented with more complex profiles on a strategic, scenario or representative level, e.g.
The suggested implementation would generate a MethodError, but it is possible to imagine ways to define this by adding the
OperationalProfileto each of the profiles of theRepresentativeProfileJulStraus commentedon Mar 24, 2026
I am more inclined to remove this and instead provide an error if the profile is too short for the time structure. The current approach can, especially in the case of
OperationalProfile, lead to potential problems if people are not aware of this behavior.trulsf commentedon Mar 26, 2026
I have opened a PR with changes in line with the one proposed here. @Zetison Please join the discussion there if you have suggestions for improvement.
trulsf commentedon Apr 13, 2026
PR is now merged (#92 ).