MPEGGroup / FileFormatConformance

MPEG File Format Conformance Framework
https://mpeggroup.github.io/FileFormatConformance/
BSD 3-Clause Clear License
10 stars 3 forks source link

Issue generating *_gpac.json files #99

Closed LionelTocze closed 1 year ago

LionelTocze commented 1 year ago

I try to use the https://github.com/MPEGGroup/FileFormatConformance/commit/897c4dc10d5505d329552c6657767b43d8d5ee75 to check some test vector, and have the following issues :

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

https://github.com/MPEGGroup/FileFormatConformance/blob/2dcb214bd1f55798cee1258c830e9e680b71f6ed/src/feature_extractor/file_features.py#L466

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

DenizUgur commented 1 year ago

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.

LionelTocze commented 1 year ago

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.

LionelTocze commented 1 year ago

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:

DenizUgur commented 1 year ago
  • 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.