Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#FRAME_END# token wrong behaviour #1129

Open
TanJeremy opened this issue Apr 19, 2022 · 6 comments · May be fixed by #1320
Open

#FRAME_END# token wrong behaviour #1129

TanJeremy opened this issue Apr 19, 2022 · 6 comments · May be fixed by #1320
Labels
bug Something isn't working

Comments

@TanJeremy
Copy link

Hello all.
We have an issue with the #FRAME_END# token when the frame range is not divisible by the chunk size.

For exemple, if we have 72 frames with a chunk size of 10,
the last job will have the correct #FRAMESPEC# "71,72", but the #FRAME_END# will be 80,
rendering 8 more frame than necessary.

unfortunatly, i don't have knowledge in java to offer you a ready to use "pull request", and only suggest you some idea like
.replaceAll("#END_FRAME#", String.valueOf(Math.min(frameNumber+frame.chunkSize-1), fs.getAll()[ fs.size() - 1]))
i'm not sure the syntax is correct but the idea is there.

Many thanks

Opencue Version Number
0.14.5

@TanJeremy TanJeremy added the bug Something isn't working label Apr 19, 2022
@carlosfelgarcia
Copy link

Does anyone know if this issue has been addressed? I did have the same issue and I already resolved, it's been under production for a month and not issues. Please let me know, so I can implement a PR for it.

Thanks,

@TanJeremy TanJeremy reopened this Sep 21, 2022
@TanJeremy
Copy link
Author

TanJeremy commented Sep 21, 2022

Does anyone know if this issue has been addressed? I did have the same issue and I already resolved, it's been under production for a month and not issues. Please let me know, so I can implement a PR for it.

Thanks,

it doesn't seem to be the case

@KernZag
Copy link

KernZag commented Mar 1, 2023

Would you mind sending a PR with your fix ?

@carlosfelgarcia
Copy link

Would you mind sending a PR with your fix ?

Definitely I can, probably next week as I'm a busy at work.

@antonshack
Copy link

antonshack commented Oct 3, 2023

Hi there.

Any news regarding that issue ?

I came up with that workaround in the meantime, when providing the command to the layer:

For Windows/CMD:

command = [
        f"for /f %i in ('powershell -command \"[math]::min(#FRAME_END#,{frame_range.end})\"')",
        "do",
    ] + py_cmd

Cheers.

@carlosfelgarcia
Copy link

Thanks for reminding me about this one, I just created a draft PR (#1320) as it might need some tests to be created before I put it in review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants