Opened 2 weeks ago

Last modified 12 days ago

#7248 new defect

Netflix/vmaf commit 47d0655 changes API of compute_vmaf again

Reported by: LigH Owned by:
Priority: important Component: build system
Version: git-master Keywords: libvmaf
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Related to ffmpeg trac ticket #6921 – but new:

vmaf commit 47d0655 adds two more parameters to compute_vmaf_score(..., int n_thread, int n_subsample)

The whole API may have to be treated as unstable.

Change History (5)

comment:1 Changed 2 weeks ago by LigH

  • Component changed from avfilter to build system

comment:2 Changed 2 weeks ago by LigH

  • Summary changed from Netflix/vmaf commit 47d0655 changes API of compute_vmaf_score again to Netflix/vmaf commit 47d0655 changes API of compute_vmaf again

comment:3 Changed 2 weeks ago by LigH

compute_vmaf() in wrapper/src/libvmaf.cpp and wrapper/src/libvmaf.h

comment:4 Changed 12 days ago by LigH

  • Keywords api added; regression removed
  • Priority changed from normal to important
  • Version changed from unspecified to git-master

I don't have the experience to create a correct diff file, so just quoting a suggestion by 'timothymowens':

Heads-up, I was able to get it to build by replacing this on line 169 of build\ffmpeg-git\libavfilter\vf_libvmaf.c:
s->ms_ssim, s->pool);
with this:
s->ms_ssim, s->pool, 0, 1);

Not a permanent solution, but will get me by until this issue is resolved correctly by ffmpeg/vmaf teams.

comment:5 Changed 12 days ago by cehoyos

  • Keywords api removed

The most important thing about this ticket is probably that we’ll have to remove this dependency if no stable api is provided, see also the discussion about the Turing library.

Note: See TracTickets for help on using tickets.