Skip to content

Implementation of get_lr_system and get_reference_data#23

Merged
PimMeulensteen merged 5 commits into
mainfrom
scratch-api
Mar 11, 2026
Merged

Implementation of get_lr_system and get_reference_data#23
PimMeulensteen merged 5 commits into
mainfrom
scratch-api

Conversation

@PimMeulensteen

Copy link
Copy Markdown
Contributor

Het bleek toch handig om een beknopte reference_data.csv mee te geven, gezien er accf en n/cmc in hetzelfde bestand zit. Zo voorkom je het probleem dat je het onderscheid hiertussen moet maken.

Geplaatst in __init__, zodat het makkelijk te importeren is.

Hierna wil ik nog een soort end-to-end test toevoegen, waarbij een model gemaakt wordt, en gekeken worden of dit model daarna weer in te lezen is.

@wowtor

wowtor commented Mar 10, 2026

Copy link
Copy Markdown
Contributor

De lr system pickle wordt niet gegenereerd door de yaml -> die nog toevoegen?

Misschien ook een unit test voor de api calls?

Comment thread lrmodule/__init__.py
@wowtor

wowtor commented Mar 10, 2026

Copy link
Copy Markdown
Contributor

Ik zie nu dat bij een van de modellen wel een save_model gedaan wordt, maar die geeft een error

Traceback (most recent call last):                                              
  File "/home/wb/workspace/lr_module_scratch/.venv/bin/lir", line 8, in <module>
    sys.exit(main())
             ~~~~^^
  File "/home/wb/workspace/lr_module_scratch/.venv/lib/python3.13/site-packages/lir/main.py", line 189, in main
    experiment.run()
    ~~~~~~~~~~~~~~^^
  File "/home/wb/workspace/lr_module_scratch/.venv/lib/python3.13/site-packages/lir/experiments/base_experiment.py", line 96, in run
    self._generate_and_run()
    ~~~~~~~~~~~~~~~~~~~~~~^^
  File "/home/wb/workspace/lr_module_scratch/.venv/lib/python3.13/site-packages/lir/experiments/predefined_experiment.py", line 60, in _generate_and_run
    self._run_lrsystem(lrsystem_config, split_data, parameters, run_name, data_config)
    ~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/wb/workspace/lr_module_scratch/.venv/lib/python3.13/site-packages/lir/experiments/base_experiment.py", line 82, in _run_lrsystem
    output.report(results)
    ~~~~~~~~~~~~~^^^^^^^^^
  File "/home/wb/workspace/lr_module_scratch/.venv/lib/python3.13/site-packages/lir/persistence.py", line 53, in report
    save_model(dirname / self.filename, data.lrsystem)
    ~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/wb/workspace/lr_module_scratch/.venv/lib/python3.13/site-packages/lir/persistence.py", line 24, in save_model
    f.write(pickle.dumps(model))
            ~~~~~~~~~~~~^^^^^^^
AttributeError: Can't get local object 'parse_bandwidth.<locals>.<lambda>'

@PimMeulensteen

Copy link
Copy Markdown
Contributor Author

Er staat hier een PR voor in lir: NetherlandsForensicInstitute/lir#331

@wowtor

wowtor commented Mar 10, 2026

Copy link
Copy Markdown
Contributor

Eventueel kunnen we in validation.yaml tijdelijk kde vervangen door logit

@PimMeulensteen
PimMeulensteen requested a review from wowtor March 10, 2026 15:17
@PimMeulensteen
PimMeulensteen merged commit 344a166 into main Mar 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants