Giter Club home page Giter Club logo

Comments (11)

stax76 avatar stax76 commented on May 16, 2024

Hey,

I'll investigate it later today.

Every time somebody posts an aspect ratio problem I'm very skeptical because this area was always something that staxrip did much better than other apps.

There were breaking changes in MediaInfo which could have introduced a bug, we'll see.

from staxrip.

stax76 avatar stax76 commented on May 16, 2024

Can you upload a sample and a full log file? If possible please split the post into independent issues, so far I'm not understanding anything, sorry.

from staxrip.

stax76 avatar stax76 commented on May 16, 2024

You can try setting MPEG2Source before opening the source file. You are not gaining anything with DGSource because DVDs take little CPU time to decode, the difference should be little to none.

Aspect Ratio detection might be broken for DGSource and DVD, you can set the source PAR manually to 16:11.

I've not installed DGSource so I cannot test it.

from staxrip.

Zet-Kaz avatar Zet-Kaz commented on May 16, 2024

DAR in Video vs. DAR in Container - in another way
My 30-second source file comes from the Disney XD TV channel. This is AVC 720x576 with DAR = 1.7777777777 - so PAR = 64:45, and this value is always defined by myself in StaxRip (Options → Image → Aspect Ratio → Source PAR).
I know that DGSource can have problems here and I never rely on it. A few years ago, Disney XD sent an MPEG-2 704x576 picture with DAR = 1.7777777777 - then PAR was supposed to write 16:11, but that's another story. Correctness of the calculated PAR source file, I confirm in MediaCoder.
Today I want to use StaxRip to only remove the interlace from the image and re-encode the movie in x264. I expect the resulting file will remain anamorphic and the player will display the video in the correct proportions based on the read PAR. So far, when I used to encode a 32 bit StaxRip (also using DGSource), the obtained result files were displayed in a proportional manner, regardless of the used player.
When I use StaxRip 64-bits 2.0.2.0 Stable Release (Container Options → Display Aspect Ratio: 0 / Automatic /), the received result file is treated differently by video players - keeping the proportions when using PAR 64:45 or without correct proportions, i.e. PAR 1: 1 will be used.

One more thing.
In the attached screenshot of 2 StaxRip screens, StaxRip 2.0.2.0 shows (in my opinion) an incorrect value for Error: -29.69%, and StaxRip 1.2.2.0 does not state this error.

Snap3
This is the same file (from StaxRip 2.0.2.0) played in MPC-HC, and next - in mpv Player:
Snap4

Attachments, and in them, in addition, two different interpretations of the proportions of thumbnails:
All files.zip

from staxrip.

stax76 avatar stax76 commented on May 16, 2024

Why are you setting an aspect ratio in the muxer options? That is probably the reason for the problem, don't use settings you don't understand. Another problem is your English. :-)

from staxrip.

stax76 avatar stax76 commented on May 16, 2024

In your target video file there is DAR of 1.25 reported by MediaInfo, that's the reason why the thumbnails AR is not correct and it's probably also the reason why mpc-hc shows a wrong AR, the reason for all this is the AR muxer setting, problem is wrong settings, I don't see any bug in staxrip, just use default settings and it will be fine.

from staxrip.

Zet-Kaz avatar Zet-Kaz commented on May 16, 2024

Hmmm, another question in my poor english is: what for you add option ar in container settings? I must use it! I may select 0 (default?) or e.g. 1,7777777. I ask again, why is this setting for muxer, after all it is defined in the Resize section for video?
And this is the only one thing I still do not understand. I am asking you for a bit of good will and willingness to understand - StaxRip 32-bit did not have this error. Finally, I see that you must always be right. Let it be. Best regards

from staxrip.

L4cache avatar L4cache commented on May 16, 2024

Tried your sample with default settings(didn't change anything but codec, that's doesn't affect the ratio), looks fine.
My player says it's DAR is 1.82:1, not sure why, but same as the original file.
n

from staxrip.

stax76 avatar stax76 commented on May 16, 2024

@Zet-Kaz

The setting exists in staxrip because it exists in mkvmerge and mp4box, you should never use it except you know exactly what you are doing.

Next build will have a tooltip that shows when you right click the setting or open the help of the dialog:

Display Aspect Ratio to be applied by mkvmerge/MP4Box. By default and best practice the aspect ratio should be signalled to the encoder and not to the muxer, use this setting at your own risk.

I must not always be right, sometimes I'm wrong and apologize and sometimes I don't have enough patience. Imagine every week or every month somebody screws his encode by changing a setting he doesn't understand, then he comes to the support and it's difficult for everybody to find the root cause because the user has little encoding and little English knowledge.

If you use Chrome I recommend using this extensions:

  • dict-cc
  • Grammar and Spell Checker - LanguageTool

You can also use a machine learning based translation software like this:

https://www.deepl.com/translator

With little better english you can get help in the doom9 or videohelp forum.

I was a bit stressed out because I try to learn web programming, I'm sorry for that.

from staxrip.

Zet-Kaz avatar Zet-Kaz commented on May 16, 2024

Thanks for the link to the translator.
I'm sorry, but now I've stopped understanding anything... Allow me to disagree with the content of the proposed ToolTip:
"Display Aspect Ratio to be applied by mkvmerge/MP4Box. By default and best practice the aspect ratio should be signalled to the encoder and not to the muxer, use this setting at your own risk."
In my opinion, after several years of using StaxRip, it is as below:

  1. the Display Aspect Ratio parameter (of course we are talking about the settings from the Container window) is not used by the encoder, and the StaxRip places it (this parameter) in the .mkv container (unless otherwise, because MediaInfo treats it as the next DAR parameter in the video track), which is created by the muxer,
  2. the encoder, however, takes into account the data contained in the 'Resize' section of the main StaxRip window,
  3. the above can be seen using gMKVExtractGUI (v2.4.2.0 -- By Gpower2), and extracting the .avc video track from the .mkv container and then analyzing its properties,
  4. interestingly, mkvmerge (via MKVToolNix GUI v33.1.0) does not "see" the DAR parameter specified in StaxRip in the 'Container' window at all (you can enter it again by selecting the appropriate value from the drop-down list), and using mkvmerge will create (for our, discussed .mkv) new .mkv file - now it has the form I meant from the very beginning, i.e. certainly 'unambiguously' anamorphous: with PAR 64:45 and with DAR always = 1,7777777777.

Or maybe it would be enough for the ToolTip mentioned above to contain only a short list of the most popular video players, which will interpret the DAR parameter well (or even need it) and those who will not use it. As you can see, e.g. Automatic option (value='0') may cause some confusion.
For the sake of clarity, I propose to add one more option to the DAR list, the "Do not use" type, which will simply clear the contents of this field (value=''). Well, I guess I've already gone too far...
In case of lack of time, please do not answer this entry - after all, no one is interested, as you can see, in our exchange of opinions.

Best greetings, congratulations on a job well done, useful for many people.

from staxrip.

stax76 avatar stax76 commented on May 16, 2024

StaxRip has always set the pixel/sample aspect ratio in the encoder, it has never set any aspect ratio in the muxer, please do not set any aspect ratio in the muxer (unless you are using video pass through or use a encoder that don't has AR support).

Please check the log file and the command line preview feature in the encoder and muxer dialogs.

Maybe create a thread at doom9 or videohelp.

encoder

muxer

from staxrip.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.