MPEGGroup/FileFormatConformance

Issue generating *_gpac.json files

LionelTocze opened this issue · 4 comments

I try to use the 897c4dc to check some test vector, and have the following issues :

  • from the several mp4 files, only one was taken into account
  • it require to put the directory with the test vectors under data/file_features directory, which was not previously the case : Is it correct ?

I then checkout the commit 2dcb214, where the two previous issues are no more present. However, the *_gpac.json files are no more generated. I fix this by modifying code

gpac_dict = _run_mp4box_on_file(actual_file_path)

replacing actual_file_path_ by mp4_path. This modification may also be necessary in main branch.

Hi Lionel,

In order for us to verify the issue could you please share the commands you have used to contribute the files? Along with the files perhaps?

Also the fix you have made for the function is no longer used.

If you are only interested in extracting file features could you look at the following command:

poetry run extract-file-features -h

You can supply any directory (-d, glob pattern, enclosed in quotes) or any single file (-i). It will extract the features alongside your source file(s). It is also recommended that you use the latest build of MP4Box.

Hi,
I used the following command, from src directory
poetry run contribute-files -i Files_Directory

I will run (later) the command for extracting files feature only, and will update this issue with result of testing this command.

Hi again,
I just test the 2 following commands with success:
poetry run extract-file-features -d 'Test2/*.mp4'
and
poetry run contribute-files -i '../data/file_features/Test2/'

I still have some questions:

  • command with contribute-files, requires test vectors to be in a directory under data/files_features : Correct ? (Command with extract-file-features has not this constraint.)
  • Both commands try access to some http resources, (So could failed if behind a proxy and incorrect settings of proxy environment.) : Is it mandatory or could it be bypassed.
  • command with contribute-files, requires test vectors to be in a directory under data/files_features : Correct ? (Command with extract-file-features has not this constraint.)

That's true. What we do is under data/file_features there would be a under_consideration folder. In which we create a new folder for the contribution you are making. See this simple example.

  • Both commands try access to some http resources, (So could failed if behind a proxy and incorrect settings of proxy environment.) : Is it mandatory or could it be bypassed.

The only request we make is to get the data from MP4RA and this is bypassed if your contribution was under under_consideration folder like I mentioned above.