GOOD EXPORTS

docs/src/index.md

@@ -171,6 +171,7 @@ ProcessBasedModelling.lhs
 ## Utility functions
 
 ```@docs
+default_value
 has_variable
 new_derived_named_parameter
 ```

src/API.jl

@@ -7,7 +7,7 @@ A process subtype `p::Process` extends the following unexported functions:
 - `rhs(p)` which is the right-hand-side expression, i.e., the "actual" process.
 - (optional) `timescale`, which defaults to [`NoTimeDerivative`](@ref).
 - (optional) `lhs(p)` which returns the left-hand-side. Let `τ = timescale(p)`.
-  Then default `lhs(p)` behavior depends on `τ` as follows:
+  Then default `lhs(p)` behaviour depends on `τ` as follows:
   - Just `lhs_variable(p)` if `τ == NoTimeDerivative()`.
   - `Differential(t)(p)` if `τ == nothing`.
   - `τ_var*Differential(t)(p)` if `τ isa Union{Real, Num}`. If real,
@@ -18,7 +18,7 @@ A process subtype `p::Process` extends the following unexported functions:
 abstract type Process end
 
 """
-    NoTimeDerivative()
+    ProcessBasedModelling.NoTimeDerivative()
 
 Singleton value that is the default output of the [`timescale`](@ref) function
 for variables that do not vary in time autonomously, i.e., they have no d/dt derivative

src/ProcessBasedModelling.jl

@@ -17,21 +17,14 @@ include("utils.jl")
 include("make.jl")
 include("processes_basic.jl")
 
-
-# TODO: In MAKE, make it so that if a variable does not have a process
-# a constant process is created for it if it has a default value.
-# Add a keyword `use_default` which would warn if no process but there is default
-# and otherwise would error.
 # TODO: Make an "addition process" that adds to processes
 # It checks whether they target the same variable
-# TODO: Package should compose with ODESystem
-# so that component-based modelling can be utilized as well.
-
 
-# TODO: Perhaps not don't export `t, rhs`?
+# TODO: Perhaps not don't export `t`?
 export t
 export Process, ParameterProcess, TimeDerivative, ExpRelaxation
 export processes_to_mtkmodel
 export new_derived_named_parameter
+export hs_variable, default_value
 
 end

src/utils.jl

@@ -1,6 +1,3 @@
-export print_system_info, has_variable, default_value
-export new_derived_named_parameter, @named_parameters
-
 """
     has_variable(eq, var)
 
@@ -13,6 +10,12 @@ function has_variable(eq::Equation, var)
 end
 has_variable(eqs, var) = any(eq -> has_variable(eq, var), eqs)
 
+"""
+    default_value(x)
+
+Return the default value of a symbolic variable `x` or `nothing`
+if it doesn't have any. Return `x` if `x` is not a symbolic variable.
+"""
 default_value(x) = x
 default_value(x::Num) = default_value(x.val)
 function default_value(x::ModelingToolkit.SymbolicUtils.Symbolic)