-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: playground and data update #52
base: master
Are you sure you want to change the base?
Conversation
@Berkmann18 are there any TODO left? I'm willing to help. |
@tenshiAMD Yes, improving the ML model being the main one. |
@Berkmann18 if you have time, can post the details? so I can have some ideas about where to start. 😄 |
I don't know if you saw my replies to the CLI PR that implements this tool, but basically there is the plumbing to finish on that end. As for this project, gathering and labelling more data and potentially considering improving the feature extraction (what turns the data into digestable chunks for the ML model) and the model itself if needed. Also, testing on data and seeing if it labels stuff well enough. |
@Berkmann18 I tried to fix the branch and encountered failing tests that always timeout https://app.circleci.com/pipelines/github/all-contributors/ac-learn/220/workflows/fcb11e51-839b-46cf-b100-b9827a7672fb/jobs/470 how long it should run? [edit] I disabled those failing tests for now |
7e39113
to
47719c3
Compare
More labels, updated diagrams and playground stuff
Use the latest documentation version and update the docs to test if it works fine.
after a merge conflict resolution
@@ -19,15 +19,17 @@ | |||
"private": false, | |||
"scripts": { | |||
"build": "kcd-scripts build", | |||
"lint": "kcd-scripts lint", | |||
"lint": "", |
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.
@tenshiAMD Was that to work around the errors it was throwing?
re #37
fix #91