-
Notifications
You must be signed in to change notification settings - Fork 36
Merge Tutorial Branch Into Main #665
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
Merged
Conversation
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
* refactor web-ui tasks, events, notifications - use EventStream for sending events * Bug Fixes - SSE behavior, login. * modify security token flow to support URL-based login * resolve 'open button' confusion * resolve OPERATION -> OPERATIONAL in all entities * make valid/invalid all caps * remove is valid: (true/false) from dataset details and export dataset. modify export dataset ui * fix Registering Model -> Registering Container * remove the approved flag in benchmark details * remove report file field if it does not exist - temp (bug exists) * remove hashes from entities details * remove the additional N/A text - add "Owner" before owner ID * add navbar color (green) and add footer * fix footer - stick at the bottom * allow users to select containers based on their types instead of writing the IDs - benchmark registration * change name, description, and location maxlength - apply migrations * apply ui changes - footer and navbar * modify reload time into 1 second - medperf login and logout * auto approve benchmark for tutorial * autopopulate some mock benchmarks for tutorials * fix line length problem * fix association dropdown ui * fix hidden error - RuntimeError: Event object at ... is bound to a different event loop * fix print logs delays * add codespaces scripts --------- Co-authored-by: hasan7n <[email protected]>
Contributor
|
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
* Add html attributes required for tests * fix missing function in common.js * fix compatibility/workflow tests popup modal title * fix auto-scroll lag issue * Add E2E tests for webui * ci: add GitHub workflow for WebUI E2E tests * fix tests issues * fix webui-ci * fix typo in local-auth env that caused errors while seeding db * remove auto-login when getting tutorial files - add profile activation test * remove logout button duplicate, add comment --------- Co-authored-by: hasan7n <[email protected]>
hasan7n
requested changes
Nov 12, 2025
add 'webui' prefix for maxlogmessages, add other webui configs
hasan7n
approved these changes
Nov 13, 2025
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
No description provided.