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

Iris: Create and call endpoint to check if Iris is active #6933

Merged
merged 33 commits into from
Jul 26, 2023

Conversation

SolizerCodes
Copy link

@SolizerCodes SolizerCodes commented Jul 18, 2023

Checklist

General

Server

  • Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
  • I followed the coding and design guidelines.
  • I added multiple integration tests (Spring) related to the features (with a high test coverage).
  • I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
  • I documented the Java code using JavaDoc style.

Motivation and Context

We want an endpoint that initially checks if the specific Iris model is active (might change to a websocket connection in the future).

Description

For now adds an endpoint which returns for a session if the model is available.

Testing

Testing if a model is not available is difficult. Try if Iris still works without problems.

@github-actions github-actions bot added the server Pull requests that update Java code. (Added Automatically!) label Jul 18, 2023
@github-actions github-actions bot added the client Pull requests that update TypeScript code. (Added Automatically!) label Jul 18, 2023
@dmytropolityka dmytropolityka self-assigned this Jul 18, 2023
@dmytropolityka dmytropolityka changed the title Iris: Create endpoint for server to check if Iris is active Iris: Create and call endpoint to check if Iris is active Jul 18, 2023
@dmytropolityka dmytropolityka added the deploy:artemis-test9 Testserver for Project Theia label Jul 20, 2023
@dmytropolityka dmytropolityka temporarily deployed to artemis-test9.artemis.cit.tum.de July 20, 2023 00:28 — with GitHub Actions Inactive
@github-actions github-actions bot added lock:artemis-test9 and removed deploy:artemis-test9 Testserver for Project Theia labels Jul 20, 2023
@dmytropolityka dmytropolityka marked this pull request as ready for review July 20, 2023 07:55
@SolizerCodes SolizerCodes added deploy:artemis-test9 Testserver for Project Theia and removed lock:artemis-test9 labels Jul 23, 2023
@SolizerCodes SolizerCodes temporarily deployed to artemis-test9.artemis.cit.tum.de July 23, 2023 15:59 — with GitHub Actions Inactive
@github-actions github-actions bot added lock:artemis-test9 and removed deploy:artemis-test9 Testserver for Project Theia labels Jul 23, 2023
@SolizerCodes SolizerCodes added deploy:artemis-test9 Testserver for Project Theia and removed lock:artemis-test9 labels Jul 24, 2023
@SolizerCodes SolizerCodes temporarily deployed to artemis-test9.artemis.cit.tum.de July 24, 2023 07:31 — with GitHub Actions Inactive
@github-actions github-actions bot added lock:artemis-test9 and removed deploy:artemis-test9 Testserver for Project Theia labels Jul 24, 2023
@SolizerCodes SolizerCodes added deploy:artemis-test9 Testserver for Project Theia and removed lock:artemis-test9 labels Jul 24, 2023
@SolizerCodes SolizerCodes temporarily deployed to artemis-test9.artemis.cit.tum.de July 24, 2023 08:39 — with GitHub Actions Inactive
@github-actions github-actions bot added lock:artemis-test9 and removed deploy:artemis-test9 Testserver for Project Theia labels Jul 24, 2023
@bassner bassner merged commit b5a797d into develop-iris Jul 26, 2023
1 check passed
@bassner bassner deleted the feature/iris-active-endpoint branch July 26, 2023 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client Pull requests that update TypeScript code. (Added Automatically!) component:Iris ready for review server Pull requests that update Java code. (Added Automatically!)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants