Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Voeg labels toe aan vereisten voor filtering beslishulp #612

Merged
merged 1 commit into from
Feb 14, 2025

Conversation

ravimeijerrig
Copy link
Contributor

@ravimeijerrig ravimeijerrig commented Feb 14, 2025

Beschrijf jouw aanpassingen

Doordat we ervoor hadden gekozen om de "lege" metadata altijd te laten zien, bleken combinaties van labels geen goed filtering te hebben. Hierdoor stel ik voor om aan elke vereisten uit de AIA de mogelijke labels toe te voegen.

Let op:

  • We zouden naar mijn mening de categorieën "operationeel" en "conformiteitsbeoordelingsinstantie" van de beslishulp helemaal buiten de filteren kunnen beschouwen (dus ze niet ophalen uit de beslishulp en ze ook niet uitvragen zonder beslishulp).
  • Open-source wordt alleen indirect gebruikt voor filtering (in het geval van een uitzondering), deze hoeft dus ook niet terug te komen in het geval van uitvraag zonder beslishulp. Als gebruikers weten dat ze in een uitzondering vallen, zullen ze dit zelf aanklikken.
  • Het is van belang dat risicogroep en transparantieverplichting beide enkel kunnen gelden voor AI-systeem of AI-systeem voor algemene doeleinden. Systeemrisico kan enkel gelden voor AI-model voor algemene doeleinden. Is het mogelijk om vragen automatisch in te vullen (niet-van-toepassing) op basis van het antwoord op welke toepassing het is of te disabelen? Zo niet, zou het fijn zijn dit met een [i]tje erbij te zetten @uittenbroekrobbert
  • Moeten we voor filtering dan ook labels toevoegen aan de andere vereisten? Die niet van de AIA zijn?

Bij welk issue hoort deze pull-request?

Checklist before requesting a review

  • Ik heb de contributing guidelines van deze repository gelezen en gevolgd.
  • Ik heb mijn aanpassingen gecheckt op spelfouten.
  • Als ik gebruik heb gemaakt van links, dan heb ik gecheckt of deze werken.
  • Ik heb gebruik gemaakt van de templates en formats van het algoritmekader.

Copy link

github-actions bot commented Feb 14, 2025

PR Preview Action v1.6.0
Preview removed because the pull request was closed.
2025-02-14 17:25 UTC

@ravimeijerrig ravimeijerrig marked this pull request as ready for review February 14, 2025 12:54
@ravimeijerrig ravimeijerrig changed the title Add_labels_to_vereisten Voeg labels toe aan vereisten voor filtering beslishulp Feb 14, 2025
@ravimeijerrig ravimeijerrig force-pushed the Align_labels_decision_tree branch 2 times, most recently from 7b2f408 to e8bafde Compare February 14, 2025 14:58
Copy link
Collaborator

@noutvandeijck noutvandeijck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ravimeijerrig ravimeijerrig force-pushed the Align_labels_decision_tree branch from 49abdb3 to dce3e95 Compare February 14, 2025 15:40
@noutvandeijck noutvandeijck merged commit 4b37e1b into main Feb 14, 2025
1 check 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