[RESOLVED] PVBob() requiring mod8 height?

These CUDA filters are packaged into DGDecodeNV, which is part of DGDecNV.
User avatar
Sherman
Posts: 576
Joined: Mon Jan 06, 2020 10:19 pm

Re: PVBob() requiring mod8 height?

Post by Sherman »

The code is already repacking for the VPP. :scratch:
Sherman Peabody
Director of Linux Development
User avatar
Rocky
Posts: 3557
Joined: Fri Sep 06, 2019 12:57 pm

Re: PVBob() requiring mod8 height?

Post by Rocky »

Don't feel bad, Sherman. I can't figure it out either. The chroma is misaligned but why? Let's sleep on it.
DAE avatar
Sharc
Posts: 233
Joined: Thu Sep 23, 2010 1:53 pm

Re: PVBob() requiring mod8 height?

Post by Sharc »

Sherman wrote:
Fri May 01, 2020 8:17 am
I don't see any obvious coding errors. It's really interesting.

Crop only the sides:

crop(0,4,-0,-8) # NO PROBLEM
PVBob()

Crop only the top/bottom:

crop(16,0,-24,-0) # NO PROBLEM
PVBob()

Crop all around:

crop(16,4,-24,-8) # FAILS
PVBob()

Here's the kicker. We can try to eliminate width/pitch issues by sourcing with BlankClip():

blankclip(240,640,576,pixel_type="YV12",color=$0000ff) # NO PROBLEM
PVBob()

blankclip(240,680,576,pixel_type="YV12",color=$0000ff) # FAILS
PVBob()

Everything is mod8 for both of them. :?

Still investigating...
In Avisynth converting to and from YV12 has 3 ChromaIn/OutPlacement options: "DV", "mpeg1" or "mpeg2". Does this matter for PVBob()?
User avatar
Rocky
Posts: 3557
Joined: Fri Sep 06, 2019 12:57 pm

Re: PVBob() requiring mod8 height?

Post by Rocky »

Shouldn't matter. The chroma mispositioning (hundreds of pixels) is way higher than what you can control with those options.
User avatar
Sherman
Posts: 576
Joined: Mon Jan 06, 2020 10:19 pm

Re: PVBob() requiring mod8 height?

Post by Sherman »

Rocky! Bullwinkle!

I found the problem. I was reading the cuviddec.h header and this caught my attention:

unsigned int raw_input_pitch
pitch in bytes of raw YUV input (should be aligned appropriately)

So I was wondering what is "appropriately"? Anybody would think that mod64 for the pitch should be fine. But I decided to experiment. I learned about experimenting from Antoine Lavoisier when we went back in time to 1778 to learn about oxygen. Did you know:

"In 1794 Lavoisier was branded a traitor because of his involvement with taxation. He was also unpopular with revolutionaries because he had supported foreign scientists whom the revolutionaries wished to strip of their assets.

Lavoisier was sentenced to death by the revolutionaries. Trumped-up charges against him included stealing money from France’s Treasury and giving it to France’s enemies.

Antoine Lavoisier died by the guillotine at the age of 50 on May 8, 1794 in Paris. Marie-Anne’s father and 26 other people were executed on the same occasion.

At the end of 1795, in a U-turn, the French government found Lavoisier innocent of all charges. By then, of course, it was too late: he was just another innocent victim of the revolution’s Reign of Terror."

Anyway about that VPP pitch. I thought I would just experiment with increasing the mod from 64. I tried both 128 and 256. And they both eliminated the artifact! So in Sharc's cropping case, the calculated VPP pitch increased from 704 to 768. My opinion is that we should leave it at 256.

I promise you that Mr Peabody did not help me on this. I did it all by myself.

Now, I'm gonna go learn quantum mechanics and solve the measurement problem. :twisted:
Sherman Peabody
Director of Linux Development
User avatar
Bullwinkle
Posts: 338
Joined: Thu Sep 05, 2019 6:37 pm

Re: PVBob() requiring mod8 height?

Post by Bullwinkle »

You're a good boy, Sherman.

I'll make you Moose Approved right away. But don't get your hopes up for mod. This town isn't big enough for both of us. :P
User avatar
Rocky
Posts: 3557
Joined: Fri Sep 06, 2019 12:57 pm

Re: PVBob() requiring mod8 height?

Post by Rocky »

Thank you, Sherman, and congratulations. I'm going to port the THD fix to DGIndexNV and then release that together with your fix.
User avatar
Natasha
Posts: 150
Joined: Wed Nov 20, 2019 11:11 am

Re: PVBob() requiring mod8 height?

Post by Natasha »

Ha ha. Sherman made Rocky look bad!
User avatar
Curly
Posts: 712
Joined: Sun Mar 15, 2020 11:05 am

Re: [RESOLVED] PVBob() requiring mod8 height?

Post by Curly »

Not as bad as you, Natasha, nyuk, nyuk, nyuk!
Curly Howard
Director of EAC3TO Development
User avatar
Levi
Posts: 52
Joined: Sat Apr 18, 2020 6:12 pm

Re: [RESOLVED] PVBob() requiring mod8 height?

Post by Levi »

Really great work, Sherman! We're all so proud of you. :salute:
User avatar
Sherman
Posts: 576
Joined: Mon Jan 06, 2020 10:19 pm

Re: [RESOLVED] PVBob() requiring mod8 height?

Post by Sherman »

Mr Peabody and I are coming back to meet you Levi. Be ready!

Daddy, dial up the year 1966, Detroit, USA.
Sherman Peabody
Director of Linux Development
User avatar
Sherman
Posts: 576
Joined: Mon Jan 06, 2020 10:19 pm

Re: [RESOLVED] PVBob() requiring mod8 height?

Post by Sherman »

Levi, what do you think of this cover?

https://www.youtube.com/watch?v=ZjpSvmH9BeE
Sherman Peabody
Director of Linux Development
User avatar
Levi
Posts: 52
Joined: Sat Apr 18, 2020 6:12 pm

Re: [RESOLVED] PVBob() requiring mod8 height?

Post by Levi »

It's a great high-energy version by a beloved sister and child of the divine. Gotta love it!

Unsung heroes are songwriters Brian Holland, Lamont Dozier, and Edward Holland, Jr. You should go back and talk to them!

Catch ya later Sherm. I'm working on an exciting new song. You'll be the first to hear it!

It will be great to meet you. I've invited Diana Ross, and the Temptations to join us. Diana did a great version too.

Soul is the heart.
User avatar
Sherman
Posts: 576
Joined: Mon Jan 06, 2020 10:19 pm

Re: [RESOLVED] PVBob() requiring mod8 height?

Post by Sherman »

😂😂😂😂😂
Sherman Peabody
Director of Linux Development
User avatar
Boris
Posts: 92
Joined: Sun Nov 10, 2019 2:55 pm

Re: [RESOLVED] PVBob() requiring mod8 height?

Post by Boris »

Seen not heard.

Got big shipment Transylvania vodka. The best, low price. Low methanol. Don't miss it. Your grandma will faint when she sees it. Wife will swoon. Send PM.
DAE avatar
Guest

Re: [RESOLVED] PVBob() requiring mod8 height?

Post by Guest »

Well done Sherman :hat:
User avatar
Sherman
Posts: 576
Joined: Mon Jan 06, 2020 10:19 pm

Re: [RESOLVED] PVBob() requiring mod8 height?

Post by Sherman »

Thank you, gonca. Even a little kid sometimes finds an acorn. ;)
Sherman Peabody
Director of Linux Development
DAE avatar
Sharc
Posts: 233
Joined: Thu Sep 23, 2010 1:53 pm

Re: [RESOLVED] PVBob() requiring mod8 height?

Post by Sharc »

I just tested it and it works as expected. Thank you Sherman, you are a good apprentice :bravo:
Post Reply