EyeofBeholder-NLeSC / orange3-argument

Argument analysis, mining, and visualization add-on for Orange3.
https://research-software-directory.org/software/orange3-argument-add-on
Apache License 2.0
1 stars 1 forks source link

Error in running example.ipynb #91

Open atefekeshavarzi opened 9 months ago

atefekeshavarzi commented 9 months ago

After running the following code:

Compute topics of chunks

chunk_topics, chunk_embeds, df_topics = chunker.get_chunk_topic(chunks=chunks)

I got this error: InvalidParameterError: The 'ngram_range' parameter of CountVectorizer must be an instance of 'tuple'. Got [1, 1] instead.

I thought it could be because of the following line: vector_model = CountVectorizer( stop_words=language, ngram_range=[ngram_min, ngram_max] I changed it to: vector_model = CountVectorizer( stop_words=language, ngram_range=(ngram_min, ngram_max)) But, it does not work.

jiqicn commented 9 months ago

Hi @atefekeshavarzi , that could be an issue caused by using a lower version of sklearn that package then desired. The other two bugs of the widgets that you found could also be relevant. May I ask which version of sklearn do you use?

atefekeshavarzi commented 9 months ago

Hey @jiqicn. The version of sklearn is 1.3.2.

jiqicn commented 9 months ago

Emm...that is weird, 1.3.2 should work perfectly.

We can do a pair-programming to work it out together if you prefer.

atefekeshavarzi commented 9 months ago

Dear Ji,It would be great if we do it together. On Friday I can’t come to CWI. Do you think we can manage to do it online by sharing a screen? During next week I will be in CWI on Tuesday and Wednesday. But I have to participate in MISDOOM as an assistant. I can come to your place during a break in one of those days. Which one do you think work best for you?Regards,Atefeh Sent from my iPhoneOn Nov 14, 2023, at 3:05 PM, Ji Qi @.***> wrote: Emm...that is weird, 1.3.2 should work perfectly. We can do a pair-programming to work it out together if you prefer.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

jiqicn commented 9 months ago

Hi @atefekeshavarzi, Let's try doing the pair programming remotely. Will you be available tomorrow?

atefekeshavarzi commented 9 months ago

Hey Ji, I will join our meeting tomorrow. Could we try it after our meeting?

Best r egards, Atefeh

On Thu, Nov 16, 2023 at 10:42 AM Ji Qi @.***> wrote:

Hi @atefekeshavarzi https://github.com/atefekeshavarzi, Let's try doing the pair programming remotely. Will you be available tomorrow?

— Reply to this email directly, view it on GitHub https://github.com/EyeofBeholder-NLeSC/orange3-argument/issues/91#issuecomment-1814098889, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVO6PTXTXYAAX64CZOWYF3TYEXNXZAVCNFSM6AAAAAA7EUWWFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJUGA4TQOBYHE . You are receiving this because you were mentioned.Message ID: @.***>

jiqicn commented 9 months ago

@atefekeshavarzi The bug should be fixed now. Please update the package to the latest version on your laptop. In case you still suffer the same issue, please feel free to reach out :)

atefekeshavarzi commented 9 months ago

Hey Ji, Thank you for the update. I gave the same error:(

Best, Atefeh

On Tue, Nov 21, 2023 at 4:13 PM Ji Qi @.***> wrote:

@atefekeshavarzi https://github.com/atefekeshavarzi The bug should be fixed now. Please update the package to the latest version on your laptop. In case you still suffer the same issue, please feel free to reach out :)

— Reply to this email directly, view it on GitHub https://github.com/EyeofBeholder-NLeSC/orange3-argument/issues/91#issuecomment-1821116091, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVO6PTRYBR6J4UX6KL7NADDYFTAJVAVCNFSM6AAAAAA7EUWWFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRRGEYTMMBZGE . You are receiving this because you were mentioned.Message ID: @.***>

jiqicn commented 9 months ago

@atefekeshavarzi, will you be at CWI tomorrow? We can have a look at it together again.

atefekeshavarzi commented 9 months ago

Hey Ji, Tomorrow I am off. But I can join our meeting like last week.Best,Atefeh On Nov 23, 2023, at 10:02 AM, Ji Qi @.***> wrote: @atefekeshavarzi, will you be at CWI tomorrow? We can have a look at it together again.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>