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

typo in camera_index.h OV2460 -> OV2640 #7461

Closed
1 task done
beta-tester opened this issue Nov 10, 2022 · 7 comments
Closed
1 task done

typo in camera_index.h OV2460 -> OV2640 #7461

beta-tester opened this issue Nov 10, 2022 · 7 comments
Assignees
Labels
Resolution: Expired More info wasn't provided Type: Example Issue is related to specific example.

Comments

@beta-tester
Copy link

Board

n/a

Device Description

n/a

Hardware Configuration

n/a

Version

v2.0.5

IDE Name

Arduino

Operating System

n/a

Flash frequency

n/a

PSRAM enabled

yes

Upload speed

n/a

Description

the title of the Camera web site is wrong for the OV2640 sensor
camera_index.h

it extracts to:

<!doctype html>
<html>
    <head>
        <meta charset="utf-8">
        <meta name="viewport" content="width=device-width,initial-scale=1">
        <title>ESP32 OV2460</title>
...

but it should be:

<!doctype html>
<html>
    <head>
        <meta charset="utf-8">
        <meta name="viewport" content="width=device-width,initial-scale=1">
        <title>ESP32 OV2640</title>
...

Sketch

n/a

Debug Message

n/a

Other Steps to Reproduce

No response

I have checked existing issues, online documentation and the Troubleshooting Guide

  • I confirm I have checked existing issues, online documentation and Troubleshooting guide.
@beta-tester beta-tester added the Status: Awaiting triage Issue is waiting for triage label Nov 10, 2022
@SuGlider
Copy link
Collaborator

Thanks for reporting!
@me-no-dev can fix it for sure!

@SuGlider SuGlider added Type: Bug 🐛 All bugs Type: Example Issue is related to specific example. and removed Status: Awaiting triage Issue is waiting for triage labels Nov 10, 2022
@me-no-dev
Copy link
Member

yup, will do. Working on updated API of the camera driver itself and once that is done, I will update the HTML as part of the changes necessary here.

@SuGlider I would not call this a "bug" though :) it's just string typo

@SuGlider
Copy link
Collaborator

@SuGlider I would not call this a "bug" though :) it's just string typo

I couldn't find a softer tag in the list :-) - I agree it is just a typo.

@SuGlider SuGlider removed the Type: Bug 🐛 All bugs label Nov 11, 2022
@beta-tester
Copy link
Author

any progress here?

@Parsaabasi
Copy link

Hello,

I close this since this report contains the release we no longer support. Please try the new versions and in case the issue persists, feel free to reopen it.

Thanks

@beta-tester
Copy link
Author

beta-tester commented Jan 10, 2025

but the current master of camera_index.h still shows this wrong title. so the typo still exist.

to verify,

  1. copy the content (comma separated hex-values) between the curled brackets of
    const unsigned char index_ov2640_html_gz[] = {
      ...
    };
  2. and paste it to the input field of the CyberChef page with recipe: From Hex, Gunzip
    and see the output:
    <!doctype html>
    <html>
        <head>
            <meta charset="utf-8">
            <meta name="viewport" content="width=device-width,initial-scale=1">
            <title>ESP32 OV2460</title>

(to reverse, from html to hex-values again, your can also use the CyberChef page with recipe: Gzip, To Hex, To Upper case, Find / Replace )

@beta-tester
Copy link
Author

the issue is not resolved nor done yet.
how can i re-open this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Expired More info wasn't provided Type: Example Issue is related to specific example.
Projects
Development

No branches or pull requests

4 participants