Skip to content

Commit 1fd9a7e

Browse files
committed
docs(log): update log
1 parent 55154f1 commit 1fd9a7e

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

docs/log.md

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -795,8 +795,14 @@ This took 26 seconds, so implemented with option for parallel execution.
795795

796796
I need to output the occupancy frequencies / probability of delay for each replication, so moved this code into the package.
797797

798-
## Linting and testing
798+
## Linting, testing and parameter validation
799799

800800
Had been a while since linted, so did that.
801801

802-
This raised some errors in tests, which I hadn't run in ages, and these were no longer working properly.
802+
This raised some errors in tests, which I hadn't run in ages, and these were no longer working properly.
803+
804+
I then add new tests based on the python template and on the tests ran in https://github.com/pythonhealthdatascience/llm_simpy/. This included unit tests, functional tests and back tests.
805+
806+
I add methods to check parameter validity in `Param`. This flagged that rehab other routing probability don't sum to 100% (88% and 13%) - but this is as described in the paper, and presumed to be due to rounding, so altered the validation test to allow.
807+
808+
> 💡 When explain tests, could do all in one section, like Tests > Back tests, Tests > Functional tests, Tests > Unit tests - and then on each of those pages, it's like, if you have parameter validation... if you have warm-up... etc. etc. suggesting tests could include.

0 commit comments

Comments
 (0)