From: Anton Khirnov <anton@khirnov.net>
To: ffmpeg-devel@ffmpeg.org
Subject: [FFmpeg-devel] [PATCH 3/4] lavfi/vf_v360: fix barrelsplit transform with padding
Date: Thu, 19 May 2022 15:47:27 +0200
Message-ID: <20220519134728.15354-3-anton@khirnov.net> (raw)
In-Reply-To: <20220519134728.15354-1-anton@khirnov.net>
Make it match Facebook's transform360
(https://github.com/facebook/transform360)
Fixes one part of #9725.
---
libavfilter/vf_v360.c | 46 ++++++++++++++++++++-----------------------
1 file changed, 21 insertions(+), 25 deletions(-)
diff --git a/libavfilter/vf_v360.c b/libavfilter/vf_v360.c
index 620a995733..940e6fc5f6 100644
--- a/libavfilter/vf_v360.c
+++ b/libavfilter/vf_v360.c
@@ -3683,30 +3683,33 @@ static int xyz_to_barrelsplit(const V360Context *s,
u_shift = 2 * ew;
+ uf = vec[0] / vec[1] * scalew;
+ vf = vec[2] / vec[1] * scaleh;
+
if (theta <= 0.f && theta >= -M_PI_2 &&
phi <= M_PI_2 && phi >= -M_PI_2) {
- uf = -vec[0] / vec[1];
- vf = -vec[2] / vec[1];
+ uf *= -1.0f;
+ vf = -(vf + 1.f) * scaleh + 1.f;
v_shift = 0;
v_offset = -eh;
} else if (theta >= 0.f && theta <= M_PI_2 &&
phi <= M_PI_2 && phi >= -M_PI_2) {
- uf = vec[0] / vec[1];
- vf = -vec[2] / vec[1];
+ vf = -(vf - 1.f) * scaleh;
v_shift = height * 0.25f;
} else if (theta <= 0.f && theta >= -M_PI_2) {
- uf = vec[0] / vec[1];
- vf = vec[2] / vec[1];
+ // back top
+ vf = (vf - 1.f) * scaleh + 1.f;
v_shift = height * 0.5f;
v_offset = -eh;
} else {
- uf = -vec[0] / vec[1];
- vf = vec[2] / vec[1];
+ // back bottom
+ uf *= -1.0f;
+ vf = (vf + 1.f) * scaleh;
v_shift = height * 0.75f;
}
- uf = 0.5f * width / 3.f * (uf * scalew + 1.f);
- vf = height * 0.25f * (vf * scaleh + 1.f) + v_offset;
+ uf = 0.5f * width / 3.f * (uf + 1.f);
+ vf *= height * 0.25f;
}
ui = floorf(uf);
@@ -3764,29 +3767,22 @@ static int barrelsplit_to_xyz(const V360Context *s,
const float scalew = s->fout_pad > 0 ? 1.f - s->fout_pad / (width / 3.f) : 1.f - s->out_pad;
const float scaleh = s->fout_pad > 0 ? 1.f - s->fout_pad / (height / 4.f) : 1.f - s->out_pad;
- const int face = floorf(y * 4.f);
+ const float facef = floorf(y * 4.f);
+ const int face = facef;
const float dir_vert = (face == 1 || face == 3) ? 1.0f : -1.0f;
float uf, vf;
uf = x * 3.f - 2.f;
switch (face) {
- case 0:
- vf = y * 2.f;
+ case 0: // front top
+ case 1: // front bottom
uf = 1.f - uf;
- vf = 0.5f - vf;
- break;
- case 1:
- vf = y * 2.f;
- uf = 1.f - uf;
- vf = 1.f - (vf - 0.5f);
- break;
- case 2:
- vf = y * 2.f - 0.5f;
- vf = 1.f - (1.f - vf);
+ vf = (0.5f - 2.f * y) / scaleh + facef;
break;
- case 3:
- vf = y * 2.f - 1.5f;
+ case 2: // back top
+ case 3: // back bottom
+ vf = (y * 2.f - 1.5f) / scaleh + 3.f - facef;
break;
}
l_x = (0.5f - uf) / scalew;
--
2.34.1
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next prev parent reply other threads:[~2022-05-19 13:48 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-05-19 13:47 [FFmpeg-devel] [PATCH 1/4] lavfi/vf_v360: drop nonsense inline specifier Anton Khirnov
2022-05-19 13:47 ` [FFmpeg-devel] [PATCH 2/4] lavfi/vf_v360: factorize vector computation for barrelsplit Anton Khirnov
2022-05-19 13:47 ` Anton Khirnov [this message]
2022-05-19 13:47 ` [FFmpeg-devel] [PATCH 4/4] lavfi/vf_v360: implement output mask " Anton Khirnov
2022-05-24 11:57 ` [FFmpeg-devel] [PATCH 1/4] lavfi/vf_v360: drop nonsense inline specifier Anton Khirnov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220519134728.15354-3-anton@khirnov.net \
--to=anton@khirnov.net \
--cc=ffmpeg-devel@ffmpeg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git