-
Notifications
You must be signed in to change notification settings - Fork 47
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
AttributeError: 'str' object has no attribute 'appendEvents' #22
Comments
No idea right now, we will try to reproduce it.
The exception is somehow caused by a string object called aedat2... we
have to try your exact command line.
Did you pull latest master?
Did you try running with no arguments? i.e. just "v2e"?
|
By the way, could you check if the pip install -e . works for you? Does
v2e get installed as script to your conda shell?
|
Hi Tobi, Yup im running on the latest branch, and pip install =e . works as well for me. Running with no arguments just v2e gives me the same error unfortunately. I was running into the same issue described here: #20 |
Yuhuang, will you look at this?
|
@tobidelbruck I will look at this. @DanBigioi From what I can see here, maybe the error is due to the AEDAT writer doesn't support arbitrary output dimensions. I will get back to you ASAP. Meanwhile, try to append some arguments such as |
seems like this only works with --dvs346 or lower. anything else breaks at : |
Yes, that's exactly it. Look in
https://github.com/SensorsINI/v2e/blob/296c66de4dbfcda07f5eaa9822de92661f01f6ab/v2ecore/output/aedat2_output.py
There are only 2 cameras supported now, Davis346 and Davis240.
What do you need?
The AEDAT output must match some AEChip type in jAER since jAER depends
on this AEChip software object model of the hardware to understand any
AEDAT file input.
|
By the way, the better way to install using setup.py is "python setup.py develop". The pip install does not seem to work properly sometimes, e.g. in google colab. We don't know why. The paths are not setup correctly. |
I just tried to reproduce the original error by running exactly the command line
I did not get the exception. The conversion completed successfully. One recent error I've seen is related to "rounding". Since v2e now uses torch to accelerate DVS modeling, it is necessary to install the very latest pytorch stuff..... It's a bit of a pain. I managed to get it working on windows by using conda installs of the packages following the requirements in the pip setup.py file. You may want to simply try the google colab option if your conversions take <40m (or if you have colab pro). See the "Open in colab" button at top of README in v2e github. |
Here is the output log
|
What about when you feed 640x480 or high resolution images/video? I tried that and if set use dvs346 it still generate something. Is it just downscaled events in coordinates?? the slomo can generate the same resolution though. |
I finally had some time and added a bunch of sanity checking for valid
v2e inputs and options for input folders and AEDAT output width and
height options.
Please pull changes and give it a try?
BTW, if you do "python setup.py develop" you can pull changes and get
immediate fixes.
|
Sorry for the delay in response, I went off on vacation the last couple of weeks 😅 I reinstalled everything pulling all new changes and conda installed all packages like you mentioned in a response above, its working! I think we can close this issue now 😃 |
@DanBigioi, not necessarily, when you pass |
Ah I see. Under what conditions will the frame get resized to the chip size? And are there any downsides to using a custom size over the original 346/240 ones? |
When I run the following command with the tennis.mov video as input:
python v2e.py -i input/tennis.mov --overwrite --timestamp_resolution=.003 --auto_timestamp_resolution=False --dvs_exposure duration 0.005 --output_folder=output/tennis --overwrite --pos_thres=.15 --neg_thres=.15 --sigma_thres=0.03 --dvs_aedat2 tennis.aedat --output_width=346 --output_height=260 --stop_time=3 --cutoff_hz=15
I get the following error:
Any clue what is happening?
The text was updated successfully, but these errors were encountered: