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

Reworking of the Athena repository retrieval #10236

Open
FelixTJDietrich opened this issue Jan 30, 2025 · 0 comments
Open

Reworking of the Athena repository retrieval #10236

FelixTJDietrich opened this issue Jan 30, 2025 · 0 comments
Labels
athena Pull requests that affect the corresponding module bug exercise Pull requests that affect the corresponding module iris Pull requests that affect the corresponding module programming Pull requests that affect the corresponding module

Comments

@FelixTJDietrich
Copy link
Contributor

FelixTJDietrich commented Jan 30, 2025

Describe the bug

The Athena repository data handling needs some reconsideration.

Artemis sends a link to a repository to Athena and Athena requests them from Artemis again once they are needed.

We are currently using AthenaRepositoryExportService.exportRepository which uses ProgrammingExerciseExportService.getRepositoryWithParticipation under the hood which will send the whole repository as zip once Athena requests it.

To Reproduce

Configure Athena with a programming exercise for immediate AI feedback requests for students and for the tutor side.

Expected behavior

Artemis sends the repository content similarly to PyrisDTOService.getRepositoryContents, for instance using getFilteredRepositoryContents(exercise.getTemplateParticipation()) which is much more performant.

This will reduce the number of requests, moves Athena to be less stateful, increases performance of the repository sending, and reduces the amount of uneccessary data accessed and sent.

Which version of Artemis are you seeing the problem on?

7.9.0

@github-actions github-actions bot added athena Pull requests that affect the corresponding module exercise Pull requests that affect the corresponding module iris Pull requests that affect the corresponding module programming Pull requests that affect the corresponding module labels Jan 30, 2025
@FelixTJDietrich FelixTJDietrich removed exercise Pull requests that affect the corresponding module iris Pull requests that affect the corresponding module programming Pull requests that affect the corresponding module labels Jan 30, 2025
@github-actions github-actions bot added exercise Pull requests that affect the corresponding module iris Pull requests that affect the corresponding module programming Pull requests that affect the corresponding module labels Jan 30, 2025
@maximiliansoelch maximiliansoelch moved this to Backlog in Athena Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
athena Pull requests that affect the corresponding module bug exercise Pull requests that affect the corresponding module iris Pull requests that affect the corresponding module programming Pull requests that affect the corresponding module
Projects
Status: Backlog
Development

No branches or pull requests

1 participant