Closed yalinli2 closed 1 year ago
Also, this class in placed under the sanunit
dir but not a SanUnit
, might be better to make it a Process
Maybe make a Kinetics
class for 0/1/2 order kinetics that can be analytically solved, but the name Kinetics
is taken now... need to talk to Joy and see if we can change that to Rate
or similar
Also, this class in placed under the
sanunit
dir but not aSanUnit
, might be better to make it aProcess
This has been moved to processes
, but it's not a Process
either
Maybe make a
Kinetics
class for 0/1/2 order kinetics that can be analytically solved, but the nameKinetics
is taken now... need to talk to Joy and see if we can change that toRate
or similar
See #84
This should have been completed for all public units upon https://github.com/QSD-Group/QSDsan/commit/13868fc905da6c37636aca51f5b205f88e1d1cf1, all tests for the corresponding systems can pass (except for a minor change in https://github.com/QSD-Group/EXPOsan/commit/45b59e3a9166fbd0cba96234714b312b763595ba because of more consistent account of N degradation)
And also done for the private repo as well https://github.com/QSD-Group/QSDsan-private/commit/64d645186a38695208d711f22839a8f59e8e3cbf, can finally close it!
Currently almost all of the subclasses of
Decay
have the following module in_run
:It'll be good to put this part in
Decay
so that any future updates can be propagated throughout, would also be good if it can be written to include asludge
effluent with consideration of the sludge moisture content