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.
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
[WIP] TimeSeries Feature #192
base: main
Are you sure you want to change the base?
[WIP] TimeSeries Feature #192
Changes from 8 commits
1eb1890
b8685d2
46c7dd1
212e3a2
2d7ad72
55a4558
11c6220
658290c
bbbcc92
0747f51
6dc456a
a51ec76
261a8e8
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
I'd try to make the UI in examples slightly more less "ornate" (more minimal, self-explanatory), if you can.
An example: rather than presenting all UI elements at the same time, you could hide all but the ones that are relevant currently - or, change their labels based on what they do in the current step of the process.
I'd strive to not having to have an introductory text on the page itself.
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.
Hi professor, I did this for both the mouse gesture and the weather predictor, they now only have buttons but have helpful texts when pressed. For the hand gestures, specifically for the train and save, I think there is a way to sort of minimize the text at the bottom, however, I patterned the UI from the train and save of the Neural network and it might be helpful if there is a consistency between the two. I am open to both and am currently drafting code for alternate UI