<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">чт, 11 янв. 2024 г., 17:51 Andrea paz <<a href="mailto:gamberucci.andrea@gmail.com" rel="noreferrer noreferrer" target="_blank">gamberucci.andrea@gmail.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> int sar_num = (aspect_w/aspect_h)*w;<br>
> int sar_den = (aspect_w/aspect_h)*h;<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">if this part was from setformat_static_diff.diff then I reasoned wrong there ... please do not use (for now)</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Are these the formulas that CinGG (and ffmpeg?) uses to calculate SAR?<br>
aspect_w would be W Ratio of the Set Format window? The same for aspect_h?<br>
While w and h would be Width and Height of the Canvas Size?<br>
Are they integers?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">in cinelerra/edlsession.h</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div dir="auto">Aspect ratio for video</div><div dir="auto">        float aspect_w;</div><div dir="auto">        float aspect_h;</div><div dir="auto"> {..}</div><div dir="auto"><div dir="auto">int output_w;</div><div dir="auto">int output_h;</div></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">so aspect is float, w/h are integers.</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
These are the patches I have:<br>
<br>
0001-Add-ffmpeg-rawdv-profiles-can-use-1440x1080-with-yuv.patch.patch<br>
0001-Add-guessed-detected-SAR-DAR-values-to-ffmpeg-asset-.patch<br>
0001-Set-also-sample-aspect-ratio-for-encoded-ffmpeg-stre.patch<br>
0002-Guard-ffmpeg.C-stream-sample-aspect-ratio-only-for-f.patch<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
0003-Add-diff-between-codec-sar-dar-and-container-sar-dar.patch<br>
0003-fix-guard-for-stream-sample-aspect-ratio-for-more-fm.patch<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I think you should apply them like 0003-fix-guard then 0003-add-diff-between</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
0004-Clarify-in-setformat.C-that-we-have-Display-aspect-r.patch<br>
0005-Update-cingg-team-copyright-info-for-2024.patch<br>
0006-Document-that-W-H-ratio-fields-mean-in-this-window.patch<br>
setformat_static_diff.diff<br>
<br>
Do I have to install all of them?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">only up to 0006. setformat.diff is wrong, sorry.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
(Should I install them all except 0003-Add-diff..., then give<br>
autogen.sh and configure; then install 0003-Add-diff... and give a new<br>
configure?)<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I think even just recompile in cinelerra dir should be ok, (versioninfo.h included in few more files, I just touch them manually to trigger rebuild on make) but on fast machine you can do full recompile just in case, too.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
PS: I tried to ask Igor Vladimirsky for clarification, but so far he<br>
has not replied.<br>
</blockquote></div></div></div>