Skip to content

Update readme#34

Open
PimMeulensteen wants to merge 1 commit into
mainfrom
update-readme
Open

Update readme#34
PimMeulensteen wants to merge 1 commit into
mainfrom
update-readme

Conversation

@PimMeulensteen
Copy link
Copy Markdown
Contributor

There was a TODO item for the exposed methods, this is fixed in this PR.

@PimMeulensteen PimMeulensteen requested a review from JoBraNFI May 12, 2026 05:46
@JoBraNFI
Copy link
Copy Markdown
Contributor

JoBraNFI commented May 12, 2026

I think it would be nice to add a __all__ section to https://github.com/NetherlandsForensicInstitute/lr_module_scratch/blob/main/lrmodule/__init__.py as well to explicitly expose these functions and support autocompletion in editors like PyCharm.

__al__ = [
    'get_lr_system',
    'get_reference_data',
    'get_validation_experiment',
]

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