[BUG] -out=spupng with EIA608/teletext: offset values in XML may be not correct #361

Closed
opened 2026-01-29 16:42:01 +00:00 by claunia · 6 comments
Owner

Originally created by @harrynull on GitHub (Jan 9, 2018).

Please prefix your issue with one of the following: [BUG], [PROPOSAL], [QUESTION].

CCExtractor version (using the --version parameter preferably) : 0.86

In raising this issue, I confirm the following (please check boxes, eg [X]):

  • I have read and understood the contributors guide.
  • I have checked that the bug-fix I am reporting can be replicated, or that the feature I am suggesting isn't already present.
  • I have checked that the issue I'm posting isn't already reported.
  • I have checked that the issue I'm porting isn't already solved and no duplicates exist in closed issues and in opened issues
  • I have checked the pull requests tab for existing solutions/implementations to my issue/suggestion.
  • I have used the latest available version of CCExtractor to verify this issue exists.

My familiarity with the project is as follows (check one, eg [X]):

  • I have never used CCExtractor.
  • I have used CCExtractor just a couple of times.
  • I absolutely love CCExtractor, but have not contributed previously.
  • I am an active contributor to CCExtractor.

Necessary information

  • Is this a regression (did it work before)?
  • What platform did you use? All platforms should have this issue.
  • What where the used arguments? -out=spupng

Additional information

While I am not sure if it worked before, the issue may be introduced in pr #877 , when changing the old implementation to the new FreeType one.

The old implementation just used fixed values for the offset, and when the font size, canvas width can be customized, the offset values should also be changed accordingly.

PS: Make sure you set an alert in GitHub so you get notifications about your ticket. We may need to ask questions and we do everything inside GitHub's system.

Originally created by @harrynull on GitHub (Jan 9, 2018). Please prefix your issue with one of the following: [BUG], [PROPOSAL], [QUESTION]. CCExtractor version (using the --version parameter preferably) : 0.86 **In raising this issue, I confirm the following (please check boxes, eg [X]):** - [X] I have read and understood the [contributors guide](https://github.com/CCExtractor/ccextractor/blob/master/.github/CONTRIBUTING.md). - [X] I have checked that the bug-fix I am reporting can be replicated, or that the feature I am suggesting isn't already present. - [X] I have checked that the issue I'm posting isn't already reported. - [X] I have checked that the issue I'm porting isn't already solved and no duplicates exist in [closed issues](https://github.com/CCExtractor/ccextractor/issues?q=is%3Aissue+is%3Aclosed) and in [opened issues](https://github.com/CCExtractor/ccextractor/issues) - [X] I have checked the pull requests tab for existing solutions/implementations to my issue/suggestion. - [X] I have used the latest available version of CCExtractor to verify this issue exists. **My familiarity with the project is as follows (check one, eg [X]):** - [ ] I have never used CCExtractor. - [ ] I have used CCExtractor just a couple of times. - [ ] I absolutely love CCExtractor, but have not contributed previously. - [X] I am an active contributor to CCExtractor. **Necessary information** - Is this a regression (did it work before)? - What platform did you use? All platforms should have this issue. - What where the used arguments? `-out=spupng` **Additional information** While I am not sure if it worked before, the issue may be introduced in pr #877 , when changing the old implementation to the new FreeType one. The old implementation just used fixed values for the offset, and when the font size, canvas width can be customized, the offset values should also be changed accordingly. PS: Make sure you set an alert in GitHub so you get notifications about your ticket. We may need to ask questions and we do everything inside GitHub's system.
claunia added the needs-confirmation-of-being-brokendifficulty: easyHacktoberfest labels 2026-01-29 16:42:01 +00:00
Author
Owner

@gagan-baghel commented on GitHub (Feb 20, 2021):

hey i would like to work on this

@gagan-baghel commented on GitHub (Feb 20, 2021): hey i would like to work on this
Author
Owner

@gagan-baghel commented on GitHub (Feb 20, 2021):

@cfsmp

@gagan-baghel commented on GitHub (Feb 20, 2021): @cfsmp
Author
Owner

@gagan-baghel commented on GitHub (Feb 20, 2021):

can you help me to understand this a little more

@gagan-baghel commented on GitHub (Feb 20, 2021): can you help me to understand this a little more
Author
Owner

@gaurav02081 commented on GitHub (Dec 16, 2025):

#claim

I’m reviewing ccextractor/src/lib_ccx/ccx_encoders_spupng.c and understanding how currently works.
I’ll share a proposed approach once I validate it against existing behavior.

The issue is a mismatch between the hardcoded offset values and the actual image dimensions generated by the text renderer for EIA608/Teletext.

the offsets are hardcoded
The Conflict: The code writes the xOffset of 88 into the XML, but the image is 600px wide.

i am working on this issue and trying to learning about it in this week .

@gaurav02081 commented on GitHub (Dec 16, 2025): #claim I’m reviewing **`ccextractor/src/lib_ccx/ccx_encoders_spupng.c`** and understanding how currently works. I’ll share a proposed approach once I validate it against existing behavior. The issue is a mismatch between the hardcoded offset values and the actual image dimensions generated by the text renderer for EIA608/Teletext. the offsets are hardcoded The Conflict: The code writes the xOffset of 88 into the XML, but the image is 600px wide. i am working on this issue and trying to learning about it in this week .
Author
Owner

@Ritish017 commented on GitHub (Dec 25, 2025):

Hi 👋
I’d like to help with confirming whether this issue is still reproducible
on the current version of CCExtractor.

I’ll test -out=spupng with EIA608/teletext and share the results here.
Please let me know if that’s okay.

Thanks!

@Ritish017 commented on GitHub (Dec 25, 2025): Hi 👋 I’d like to help with confirming whether this issue is still reproducible on the current version of CCExtractor. I’ll test `-out=spupng` with EIA608/teletext and share the results here. Please let me know if that’s okay. Thanks!
Author
Owner

@cfsmp3 commented on GitHub (Jan 10, 2026):

Closed by #1850

@cfsmp3 commented on GitHub (Jan 10, 2026): Closed by #1850
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#361