Skip to content
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

Some sequences in youtube-vos dataset are filtered out wrongly #5

Open
zanglam opened this issue Nov 28, 2019 · 4 comments
Open

Some sequences in youtube-vos dataset are filtered out wrongly #5

zanglam opened this issue Nov 28, 2019 · 4 comments

Comments

@zanglam
Copy link

zanglam commented Nov 28, 2019

Hi, I found that some sequences in youtube-vos dataset are filtered out wrongly when initializing the data.
In function _init_data in dataset_loaders/ytvos_v2.py, in order to filter out some short sequences, you wrote:

self._viable_seqs = [seq for seq in self._all_seqs if
                             len(self._nonempty_frame_ids[seq]) > 0
                             and len(self.get_image_frame_ids(seq)[min(self._nonempty_frame_ids[seq]) :
                                                                   max(self._visible_objects[seq].keys()) + 1])
                             >= self._seqlen]
print("{} sequences remaining after filtering on length (from first anno obj appearance to last anno frame.".format(len(self._viable_seqs)))

However, in the original youtube-vos dataset, in some cases, the correct length of the sequence can not be obtained using the following code(although it is correct for davis dataset):

len(self.get_image_frame_ids(seq)[min(self._nonempty_frame_ids[seq]) :                                                      
     max(self._visible_objects[seq].keys()) + 1])

For example, for sequence 8be56f165d
self.get_image_frame_ids(seq) is [40, 45, 50, ..., 125, 130, 135]
min(self._nonempty_frame_ids[seq]) is 40
max(self._visible_objects[seq].keys()) + 1 is 136
In this case, I guess you hope to get the length of the list [40, 45, 50, ..., 125, 130, 135], but the function len() will return 0.

@joakimjohnander
Copy link
Owner

Hey,

Thank you! That is indeed incorrect. I will adress it and update the code.

Best regards,
Joakim

@pytho12
Copy link

pytho12 commented Apr 18, 2021

Is this problem solved now ?

@pytho12
Copy link

pytho12 commented Apr 18, 2021

f you see this message, please reply to me as soon as possible. I really need your help. Thank you very much

@pytho12
Copy link

pytho12 commented Apr 18, 2021

if you see this message, please reply to me as soon as possible. I really need your help. Thank you very much

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants