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.

Sign up for GitHub

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

Jump to bottom

Extending pipeline, pipeline as component and component with /explain endpoint to explain themself #293

Merged
merged 43 commits into from
Aug 20, 2024
Merged

Extending pipeline, pipeline as component and component with /explain endpoint to explain themself #293

merged 43 commits into from
Aug 20, 2024

Conversation

dschiese
Copy link
Collaborator

@dschiese dschiese commented Aug 6, 2024

Added:

  • /explain endpoint
  • OpenAPI definition
  • QanaryExplanation interface, implemented by
    • QanaryComponent
    • QanaryPipelineComponent
    • QanaryPipeline
  • default explain() implementation for each case

@dschiese
Added explanation controller
069e140
@dschiese
Added template for test
fb34381
@dschiese
Changed RequestParam to PathVariable, added test
1112c56
@dschiese
Undo package change
b375d91
@dschiese
Create Explanation service with placeholder values
a46f78a
@dschiese
Added component selection query and extended explanation process
a4b69ff
@dschiese
Merge branch 'master' into explanation_api
d1b5cf1
@dschiese
WIP
41f2c8a
@dschiese
Merge branch 'master' into explanation_api
64f8ae7
@dschiese
Merge branch 'master' into explanation_api
f75c486
@dschiese
WIP
560b8c9
@dschiese
Merge branch 'explanation_api' of github.com:WDAqua/Qanary into expla…
32d214d 
…nation_api
@dschiese
WIP
d85ad88
@dschiese
[Done]: Implementation, [WIP]: Tests
a35c7ce
@dschiese
[Done]: Implementation, [WIP]: Tests
38dc2e1
@dschiese
WIP: Changed approach, added functions
1f399ae
@dschiese
explain() method with standard implementation for Qanary components, …
e531b86 
…overwritable for PaC's
@dschiese
WIP
2199b3e
@dschiese
WIP
9eb5600
@dschiese
Merge branch 'master' into explanation_api
f153e41
@dschiese
WIP: Extend pipeline explanation
a80d88f
@dschiese
Moved explanation capability to qanary_commons
ebfb455
@dschiese
[Done]: Extend pipeline, PaC, component with /explain endpoint and co…
ddbe706 
…nnect them to the external qanary explanation service
@dschiese
Deleted unnecessary classes, removed redundant dependencies
d9883a6
@dschiese
Fix component names
81c9513
@dschiese
Increment version, minor changes
ae28d6c
@dschiese
minor changes
4d1731c
@dschiese dschiese marked this pull request as ready for review August 8, 2024 08:44
@dschiese
Copy link
Collaborator Author

dschiese commented Aug 9, 2024

Please mind https://github.com/WSE-research/team-tasks/issues/443 !

@dschiese dschiese merged commit 0a5847e into master Aug 20, 2024
4 checks passed
@dschiese dschiese deleted the explanation_api branch August 20, 2024 09:31
@dschiese dschiese restored the explanation_api branch August 20, 2024 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anbo-de anbo-de anbo-de left review comments

@heinpa heinpa Awaiting requested review from heinpa

@Perevalov Perevalov Awaiting requested review from Perevalov

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dschiese @anbo-de