-
Couldn't load subscription status.
- Fork 2
CU-869anj8ub: Add initial commit of WWC conversion #153
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
CU-869anj8ub: Add initial commit of WWC conversion #153
Conversation
|
Task linked: CU-869anj8ub Move medcat-scripts to cogstack-nlp |
|
Hey can you add some detail on why some of the files were left out? EG I see that 1_create_model and 2_train_model aren't included. One guess is that these are both completely covered in the tutorials as well. |
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.
Approved - just wondering if this is the compete set of files or if you should add the others. Would probably prefer to move unnecessary ones, as we can delete later, over accidentally missing some useful ones, so that we dont have to go through the other repo as much
The creation of new models isn't really something most end users do. The idea was to only keep things that are actually relevant to the majority of the people who used I renamed the training to With that said, there is some meta cat training that I didn't port over. And I can't quite recall why that is. I'll see if I've got something documented. |
|
@alhendrickson The other one doesn't really fit in the way that it's laid out. |
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.
Approved, thanks for explaining
This is the
working_with_cogstackreplacement.It will use
medcat_denunder the hood for centralised model storage.working_with_cogstackmedcat-den