comet-commonsense icon indicating copy to clipboard operation
comet-commonsense copied to clipboard

Can I use the input with a longer length?

Open RenzeLou opened this issue 3 years ago • 5 comments

Dear authors,

I found you set the max_event and max_affect to 17 and 35 respectively. However, the length of my input event is much longer than 17, can I change it according to my need?

After scanning your code, I tend to change opt['data']["maxe1"] in main_atomic.py and self.max_event in atomic.py, can it work?

RenzeLou avatar Dec 07 '21 15:12 RenzeLou

Dear authors,

I found you set the max_event and max_affect to 17 and 35 respectively. However, the length of my input event is much longer than 17, can I change it according to my need?

After scanning your code, I tend to change opt['data']["maxe1"] in main_atomic.py and self.max_event in atomic.py, can it work?

Hi, Have you solved this problem?I met the same difficulty.

Aurora-slz avatar Jun 15 '22 08:06 Aurora-slz

I have this problem too......

zhuwh04 avatar Jul 13 '22 02:07 zhuwh04

Dear authors,

I found you set the max_event and max_affect to 17 and 35 respectively. However, the length of my input event is much longer than 17, can I change it according to my need?

After scanning your code, I tend to change opt['data']["maxe1"] in main_atomic.py and self.max_event in atomic.py, can it work?

Hi, Have you already solved this problem?

zhuwh04 avatar Jul 13 '22 02:07 zhuwh04

@Aurora-slz @zhuwh04 Hi, both. Unfortunately, I didn't fix this problem yet. These hyper-parameters, namely max_event and max_affect seemed unchangeable.

Hope exact response from the authors.

RenzeLou avatar Jul 13 '22 06:07 RenzeLou

I found a way which may solve the problem cut the length of the tensor prefix in the function "set_atomic_inputs" in file functions.py into 18 %$U8`3%$FJ$)J N46~6IU Some results can be completely generated, but some results seem to be broken...... (I only test on ATOMIC dataset)

Nemo0223 avatar Apr 12 '24 08:04 Nemo0223