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

Report Viewer Demo #1448

Merged
merged 5 commits into from
Jan 12, 2024
Merged

Report Viewer Demo #1448

merged 5 commits into from
Jan 12, 2024

Conversation

Kr0nox
Copy link
Member

@Kr0nox Kr0nox commented Dec 15, 2023

This PR adds a demo for the report viewer. This demo will be hosted in a new repository (called demo) and will then be accessible on jplag.github.io/Demo/
The demo gets updated on each push to main.
We build jplag from its sources and then run it on the progpedia dataset. The results are inserted into the report viewer

To preview the demo locally, you have to add a report called example.zip to the public folder in the report-viewer and then run:
npx vite --mode demo

@tsaglam Before the newly created workflow gets run (so either before the merge of develop into main or before it is run manually) please create an new repo called Demo and create the gh-pages branch.

@dfuchss Do you know whether we can reuse the token for the dev deploy in the workflow?

@Kr0nox Kr0nox added the report-viewer PR / Issue deals (partly) with the report viewer and thus involves web-dev technologies label Dec 15, 2023
@Kr0nox Kr0nox linked an issue Dec 15, 2023 that may be closed by this pull request
@dfuchss
Copy link
Member

dfuchss commented Dec 20, 2023

You should be able to reuse the token

Copy link

Quality Gate Passed Quality Gate passed for 'JPlag Report Viewer'

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@Kr0nox
Copy link
Member Author

Kr0nox commented Jan 10, 2024

Demo can be viewed here

Copy link
Member

@sebinside sebinside left a comment

Choose a reason for hiding this comment

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

I like the solution, let's see whether it will eventually break the build 😁 I just created an empty Demo repository.

For a future version, we should discuss whether the report viewer should directly open up the overview page in demo mode because the file upload view is a little bit pointless.

@sebinside sebinside merged commit 2a5813a into develop Jan 12, 2024
16 checks passed
@sebinside sebinside deleted the report-viewer/demo branch January 12, 2024 16:02
@Kr0nox
Copy link
Member Author

Kr0nox commented Jan 23, 2024

A test run for deploying it failed. Here the link to the run link

The reason is : remote: Permission to JPlag/Demo.git denied to SDQBot.
@dfuchss Do you think that mean we need a special token?

@dfuchss
Copy link
Member

dfuchss commented Jan 24, 2024

@Kr0nox should be fixed now. I've retriggered the Workflow :)

@dfuchss
Copy link
Member

dfuchss commented Jan 24, 2024

See https://jplag.github.io/Demo/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
report-viewer PR / Issue deals (partly) with the report viewer and thus involves web-dev technologies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Host a demo of the report viewer
3 participants