-
Notifications
You must be signed in to change notification settings - Fork 26
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
Improves and fixes the web-client UI #668
Merged
Merged
Changes from all commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
9faab14
Remove list filters in the predefined tasks web-client page
JulienVig 44750f2
Tmp: save cache model use toggle button
JulienVig 880ef11
Rework ModelCaching and move it to Trainer rather than task description
JulienVig b35d72e
Add loss charts to the web-client training informations
JulienVig 9aa6d0c
Clean and refactor default tasks tests
JulienVig f3aa68a
Fix web-client and docs/example tests
JulienVig 449e994
Add pre-defined task description and fix mlo logo
JulienVig 6af385d
Add data selection description
JulienVig aac0863
Remove some default tasks and add sample dataset links in task displa…
JulienVig e3aa0b4
Reword home page text
JulienVig 874292b
Bug fix and add error handling to data selection UI
JulienVig bfad11e
Fix web-client task test
JulienVig dcef3f5
Fix discojs-node image loader test
JulienVig 22e7ed2
Move model library toggle button to the model library panel
JulienVig 4ae7f12
Add hover description to model library buttons
JulienVig 5d1959d
Add new sidebar links to custom task and model evaluation
JulienVig 1289096
Move the about us section to the footer MLO logo
JulienVig 6752003
Harmonize UI design on information page with the rest of the web-client
JulienVig 2413e48
Fix web-client unused import
JulienVig 87e0ffe
Fix Cifar10 test
JulienVig d7abb74
Remove i18n internationalizaion plugin
JulienVig 75ddf94
Make display_information dataset link a URL type
JulienVig a90d6c7
Update cli/src/data.ts with Repeat rather than Range
JulienVig 98ded2b
Update image_loader.spec.ts with Repeat rather than Range
JulienVig e21f60e
Update federated.spec.ts with Repeat rather than Range
JulienVig 73c107c
Update web-client/src/components/data/dataset_input/DatasetInput.vue
JulienVig d7c6b10
Update web-client/src/components/training/TrainingInformation.vue
JulienVig 4d9ad94
Update web-client/src/components/training/TrainingInformation.vue
JulienVig c66925d
Make task information URLs type URL
JulienVig 4696c84
Make training logs scrollable
JulienVig 06378a8
Rollback url as type string but add URL checks
JulienVig 2e7db2e
Make buttons wrap when screen not wide enough
JulienVig ec26c8f
Add buttons wrap
JulienVig aed256d
Remove geotag label logic remnants
JulienVig b6ef8e5
Remove unused task tradeoffs and limitations
JulienVig 708dae6
Add missing Repeat import
JulienVig 1ae94e3
Update Aobut Us design
JulienVig 7b301c5
Merge branch 'develop' into 662-improve-tasks-julien
JulienVig 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
There are no files selected for viewing
This file contains 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 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 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 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 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 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 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 was deleted.
Oops, something went wrong.
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,8 +1,6 @@ | ||
export { cifar10 } from './cifar10.js' | ||
export { geotags } from './geotags.js' | ||
export { lusCovid } from './lus_covid.js' | ||
export { mnist } from './mnist.js' | ||
export { simpleFace } from './simple_face.js' | ||
export { skinMnist } from './skin_mnist.js' | ||
export { titanic } from './titanic.js' | ||
export { wikitext } from './wikitext.js' |
This file contains 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 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 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 was deleted.
Oops, something went wrong.
This file contains 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
Oops, something went wrong.
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.
consider adding that to
datasets/populate
instead of pulling from our own archiveThere 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.
Would it work when the webapp is deployed? I think datasets/populate can only be used in local isn't it?
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.
correct, do keep it in cifar10's
Task
and add it todatasets/populate
so that we ensure that we also train our tests with the sample dataset.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.
Ah so some cifar10 data is already included in the
example_training_data
(which requires manipulation to used in the webapp...) but I wanted to provide a public facing link too.