-
Notifications
You must be signed in to change notification settings - Fork 276
Output vizualizer #2744
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
Open
vmcj
wants to merge
54
commits into
DOMjudge:main
Choose a base branch
from
vmcj:output_vizualizer
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Output vizualizer #2744
Changes from 1 commit
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
ebb0b43
Fix judgehost check if its enabled
vmcj a43209a
Future work
d3eec0c
Store new visualizer type
cdc1692
Make the doctrine links
88c0e88
This should be part of the ProblemZip
131abb5
Add simple output_validator
5e3406f
Assume this will be put in the spec
3b40afa
Allow upload of the output_visualizer
d4f4401
First try on import
101cc03
Give icon for new executable type
69d99dc
Display the problem badge
df61e4a
Display on the executable page itself
a956b58
Add button to generate the visualization
fbe7358
Show executable in problem page
2986727
Leave this for the demo
d79783d
Remember if we already requested visualization
8de3e52
Create the needed judgetasks
f3b96e1
Fix PHPStan issues
f6baa3a
Fix syntax error
76deaaf
Just move the whole code away for now
aa062d7
And more errors as the needed code was commented out
2963350
Revert later - Fixate hostname for api/doc debugging
f0dcaec
Fix API for when you test on a deeper dir
1636944
New migrations
1af147a
Test the search in an unrelated page
d88d0c0
Also get the new visualization `JudgeTask`s
e56a83f
Return the visualizer jobs
3a05304
Store the judgetask
87ca226
Working on the judgehost side
5d5d6c6
Fix output to create the image
52fde24
Finish
ac0679c
Add forgotten class
1edc428
Fix the visualizer script
9362b92
Cleanup the judgehost file
812042e
Check returncode as `FIXME`.
eca9e35
Merge migrations
de746eb
Cleanup unrelated testing
70ee0e4
Update webapp/src/Entity/Visualization.php
vmcj 7bcac2b
Update webapp/src/Controller/API/JudgehostController.php
vmcj 0a7daca
Update webapp/migrations/Version20241018061817.php
vmcj 7190795
Update webapp/migrations/Version20241018061817.php
vmcj c424d21
Update webapp/src/Controller/API/JudgehostController.php
vmcj fa3e2ed
Update webapp/src/Controller/API/JudgehostController.php
vmcj 7641e17
Update webapp/src/Controller/API/JudgehostController.php
vmcj 5b9c970
Update webapp/src/Controller/Jury/UserController.php
vmcj 031d24a
Remove dump and debug statements
51d3cc3
Cleanup unused code
d34fce3
Fix missing `$run`
d616057
Prevent duplication of code
ed5e47c
Not used
f9de23c
Fix W3C test
f7e7e23
Fixup
28c3904
Add last forgotten code
2b6afcd
Fixup
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Add button to generate the visualization
Getting this directly via submission.problem.output_visualizer_executable (the property) seems to fail. It does show up in the twig dump but the translation fails when using it.
- Loading branch information
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add
and not visualization is defined
?