Skip to content

UnknownMass called with bad argument #132

Open

Description

@cscherrer

Currently (v0.14.7) we have

massof(m::AbstractMeasure) = UnknownMass(m)

But there's no such method:

julia> methods(MeasureBase.UnknownMass)
# 4 methods for type constructor:
 [1] MeasureBase.UnknownMass()
     @ ~/git/MeasureBase.jl/src/mass-interface.jl:19
 [2] (::Type{T})(x::AbstractChar) where T<:Union{AbstractChar, Number}
     @ Base char.jl:50
 [3] (::Type{T})(x::T) where T<:Number
     @ Core boot.jl:792
 [4] (::Type{T})(x::Base.TwicePrecision) where T<:Number
     @ Base twiceprecision.jl:266

I think this should be

massof(m::AbstractMeasure) = UnknownMass()

Activity

  1. self-assigned this
    on Sep 1, 2023
  2. cscherrer commented on Sep 1, 2023

    @cscherrer
    CollaboratorAuthor

    Oh good, looks like this is already fixed by #120

  3. linked a pull request that will close this issueAdd HierarchicalMeasure #120on Sep 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions