Skip to content

Commit 192288c

Browse files
committed
fix: import of vars into SymbolicUnits sybmodule
1 parent 3b6525f commit 192288c

File tree

4 files changed

+13
-15
lines changed

4 files changed

+13
-15
lines changed

src/register_units.jl

-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ function update_all_values(name_symbol, unit)
1414
ALL_MAPPING[name_symbol] = INDEX_TYPE(length(ALL_MAPPING) + 1)
1515
end
1616

17-
1817
# Register
1918
macro register_unit(name, value)
2019
return esc(_register_unit(name, value))

src/symbolic_dimensions.jl

+8-12
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ end
9090
function SymbolicDimensionsSingleton{R}(s::Symbol) where {R}
9191
i = get(ALL_MAPPING, s, INDEX_TYPE(0))
9292
iszero(i) && error("$s is not available as a symbol in `SymbolicDimensionsSingleton`. Symbols available: $(ALL_SYMBOLS).")
93-
SymbolicDimensionsSingleton{R}(i)
93+
return SymbolicDimensionsSingleton{R}(i)
9494
end
9595

9696
# Traits:
@@ -367,15 +367,14 @@ to enable pretty-printing of units.
367367
"""
368368
module SymbolicUnits
369369

370-
371370
import ..UNIT_SYMBOLS
372371
import ..CONSTANT_SYMBOLS
373372
import ..SymbolicDimensionsSingleton
374-
import ...constructorof
375-
import ...DEFAULT_SYMBOLIC_QUANTITY_TYPE
376-
import ...DEFAULT_SYMBOLIC_QUANTITY_OUTPUT_TYPE
377-
import ...DEFAULT_VALUE_TYPE
378-
import ...DEFAULT_DIM_BASE_TYPE
373+
import ..DEFAULT_SYMBOLIC_QUANTITY_TYPE
374+
import ..constructorof
375+
import ..DEFAULT_SYMBOLIC_QUANTITY_OUTPUT_TYPE
376+
import ..DEFAULT_VALUE_TYPE
377+
import ..DEFAULT_DIM_BASE_TYPE
379378

380379
# Lazily create unit symbols (since there are so many)
381380
module Constants
@@ -393,10 +392,7 @@ module SymbolicUnits
393392
@eval begin
394393
const $unit = constructorof(DEFAULT_SYMBOLIC_QUANTITY_TYPE)(
395394
DEFAULT_VALUE_TYPE(1.0),
396-
SymbolicDimensionsSingleton{DEFAULT_DIM_BASE_TYPE}(
397-
$(QuoteNode(disambiguate_symbol(unit))),
398-
),
399-
)
395+
SymbolicDimensionsSingleton{DEFAULT_DIM_BASE_TYPE}($(QuoteNode(disambiguate_symbol(unit)))))
400396
push!(_SYMBOLIC_CONSTANT_VALUES, $unit)
401397
end
402398
end
@@ -412,7 +408,7 @@ module SymbolicUnits
412408
@eval begin
413409
const $unit = constructorof(DEFAULT_SYMBOLIC_QUANTITY_TYPE)(
414410
DEFAULT_VALUE_TYPE(1.0),
415-
SymbolicDimensionsSingleton{DEFAULT_DIM_BASE_TYPE}($(QuoteNode(unit))),
411+
SymbolicDimensionsSingleton{DEFAULT_DIM_BASE_TYPE}($(QuoteNode(unit)))
416412
)
417413
push!($symbolic_unit_values, $unit)
418414
end

test/runtests.jl

+3
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,9 @@ else
1919
@safetestset "Measurements.jl integration tests" begin
2020
include("test_measurements.jl")
2121
end
22+
@safetestset "Meshes.jl integration tests" begin
23+
include("test_meshes.jl")
24+
end
2225
@safetestset "Unit tests" begin
2326
include("unittests.jl")
2427
end

test/unittests.jl

+2-2
Original file line numberDiff line numberDiff line change
@@ -688,8 +688,8 @@ end
688688
@test !iszero(sym)
689689
end
690690

691-
q = 1.5us"km/s"
692-
@test q == 1.5 * us"km" / us"s"
691+
q = 1.5us"km/s"
692+
@test q == 1.5 * us"km" / us"s"
693693
@test typeof(q) <: with_type_parameters(DEFAULT_QUANTITY_TYPE, Float64, SymbolicDimensions{DEFAULT_DIM_BASE_TYPE})
694694
@test string(dimension(q)) == "s⁻¹ km"
695695
@test uexpand(q) == 1.5u"km/s"

0 commit comments

Comments
 (0)