-
Notifications
You must be signed in to change notification settings - Fork 7
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
Let the user get a link in the email when training a classifier is done, and then upload an unlabeled set #301
Comments
@monajalal @davidatbu this method contradicts with the current one. I still don't know how to do it subtly. |
I think we have to do it in the backend. Frontend modification is kind of limited. |
Hi @edward-edberg, I don't see how modifying the backend will help with this. The backend already sends an email when training finishes, and then another one when predicting finishes. I understand what you are saying in that, in the frontend, this is not possible if the user is going to upload training dataset and the unlabeled dataset at the same time. But, if we have different pages for the two, imagine this: The user goes to On the frontend, the JS inside And then we send an email to the user with the results. |
Okay, but how about if the client wants to use default classifier? They will upload the test file in the first place, right? |
Okay, but how about if the client wants to use default classifier? They will upload the test file in the first place, right?
Yup!
If we want to implement a new mechanism that you have mentioned, then we will separate training from scratch page and testing-only page.
Yes, exactly.
…On Tue, Jul 14, 2020 at 8:12 PM Edward Edberg Halim < ***@***.***> wrote:
Okay, but how about if the client wants to use default classifier? They
will upload the test file in the first place, right?
If we want to implement a new mechanism that you have mentioned, then we
will separate training from scratch page and testing-only page.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#301 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMQIHC3V2OBZMWWGY22ZHN3R3TX6ZANCNFSM4OZ4HTYQ>
.
|
@davidatbu I have a question, how do you provide unique link for each task to upload test file? |
Great question Edward!
We actually already do this with topic model previews. If you go and train
a topic model right now, you'll get an email, with a unique link to your
topic model, for example
www.openframing.org/topicModelPreviews.html?topic_model_id=2 (you can
actually click on it). Since the backend has access to the topic model id,
that's all we have to change to generate a "unique link".
…On Wed, Jul 15, 2020 at 10:47 AM Edward Edberg Halim < ***@***.***> wrote:
@davidatbu <https://github.com/davidatbu> I have a question, how do you
provide unique link for each task to upload test file?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#301 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMQIHC47Z3HYOPQDGC2MO6TR3W6RDANCNFSM4OZ4HTYQ>
.
|
@davidatbu ok, you can provide me topic_model_id, then how can I get that topic_model_id? since I have to make post request |
parse the url? or do you have any suggestion? because I have never done this before |
Here's how we do it in topicModelPreviews.html Here's the function: Here's how we call it. |
My code looks terrible compared to this lol. |
I copy pasted it from Stackoverflow :) I usually live a link to where I
found it but I forgot to this time somehow.
…On Wed, Jul 15, 2020 at 2:03 PM Edward Edberg Halim < ***@***.***> wrote:
My code looks terrible compared to this lol.
Got it. I think I can build them.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#301 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMQIHCZYPFYZJRFY3GAL3CTR3XVNJANCNFSM4OZ4HTYQ>
.
|
Right now, the user has to keep their tab open, to make sure that they can train and predict on a custom dataset of their own.
The text was updated successfully, but these errors were encountered: