Skip to content

Elaborated on HQ Camera gasket assembly #1554

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

Merged
merged 5 commits into from
Jun 8, 2021

Conversation

Adam-Langley
Copy link
Contributor

Described the gasket assembly, added some images, detailed how to protect from dust ingress.

@Adam-Langley Adam-Langley changed the title Elaborated on gasket Elaborated on HQ Camera gasket assembly May 20, 2020
@lurch
Copy link
Contributor

lurch commented May 20, 2020

You've got the identical camera gasket alt-text on 3 of the images - is that intentional?

@Adam-Langley
Copy link
Contributor Author

thanks @lurch - fixed my mistaken duplication

@lurch
Copy link
Contributor

lurch commented May 20, 2020

ping @simon-martin-rpi for comment

@JamesH65
Copy link
Contributor

Simon is loaded up with work at the moment.

I'm not entirely convinced we need the first image, but I guess it's a minor niggle.

@JamesH65
Copy link
Contributor

@davidplowman @helenlynn Can I have your opinions on this change?

@davidplowman
Copy link
Contributor

Seems OK to me, though I was slightly confused where it tells you to push down on the filter so that it comes off, and then afterwards tells you about applying isopropyl alcohol as if that's a separate step (was it?). But then, I'm not a h/w engineer... To me, this feels like a good candidate for a HOW-TO video!

@aallan
Copy link
Contributor

aallan commented May 26, 2020

To me, this feels like a good candidate for a HOW-TO video!

Perhaps. Except I think we want to avoid extensive documentation — and support — for something that will destroy the camera if the user does it wrong. It's documented here, because people will want to do it and we should tell them how, but I don't think Raspberry Pi should be seen as supporting it. It's not something we look like we recommend users attempt.

But yes, it's far more of a video thing than a text thing. For what it's worth, I think the addition of images in this pull really helps the documentation.

@Adam-Langley
Copy link
Contributor Author

@davidplowman happy to remove the more specific points of using ISO and heat - I just did it because I thought it would reduce the risk of breaking the filter. I've only done this once, so no empirical evidence that it helped.
Also happy to reassemble and record a video. My thinking was just the same as @aallan - I felt like pics would have given more of an idea of what I was getting myself into.
Dont think you can host or embed videos on github however... so we'd have to link out, making the documentation a little less 'self contained'.

@davidplowman
Copy link
Contributor

No - don't do anything on my account! I was just confused because I'm not sure how this thing works, probably others are less easily confused. Also, I fully take the point about not doing a video, I was thinking about it from the point of view of making it easier for people to do this, but I accept that may not quite be the goal here...

@lurch
Copy link
Contributor

lurch commented May 26, 2020

I don't have a HQ camera myself, but I've always found the "push down on the filter" phrase confusing. Is it supposed to be "push sideways", or is there a specific location where you're supposed to be "pushing down"?

An attempt to make the wording less "instructional" and more a "suggestion"
@Adam-Langley
Copy link
Contributor Author

@lurch the assumption is by that stage you have the lens module lying flat on the table. The IR filter is seated inside a recess - so you need the module oriented such that the lens would "fall" out of the recess were it not glued in.
I actually noticed that it's not a particularly snug fit for the filter - so in my case the filter was off-center, leading to one edge having a lesser surface area adhered to the ledge inside the recess. That's the edge that I chose to push against for least resistance.

@lurch
Copy link
Contributor

lurch commented May 26, 2020

Ahhh! I didn't realise the filter was attached to the lens assembly! That makes more sense now. I thought your pictures were showing the IR filter attached to the sensor itself 😆 That'll teach me for sticking my nose where it's not wanted 😉

1. Lift up the board and place it down on a very clean surface. Make sure the sensor does not touch the surface.
1. Before completing the next step, read through all of the steps and decide whether you are willing to void your warranty. **Do not proceed** unless you are sure that you are willing to void your warranty.
1. Turn the lens around so that it is "looking" upwards and place it on a table. Using a pen top or similar soft plastic item, push down on the filter. The glue will break and the filter will detach from the lens mount.
1. Turn the lens around so that it is "looking" upwards and place it on a table.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this say lens mount?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've asked for Simon's input here too.

1. Turn the lens around so that it is "looking" upwards and place it on a table.
You may try some ways to weaken the adhesive, such as a little isopropyl alcohol and/or heat (~20-30 C). Using a pen top or similar soft plastic item, push down on the filter only at the very edges where the glass attaches to the aluminium - to minimise the risk of breaking the filter. The glue will break and the filter will detach from the lens mount.
![camera ir filter](rpi_hq_cam_ir_filter.jpg)
1. Given that changing lenses will expose the sensor, at this point you could affix a clear filter (OHP plastic) to minimize the chance of dust entering the sensor cavity.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OHP plastic is one option, this implies its the only option.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've committed a tiny wording change to make clear that OHP is an example and not the only possibility.

@@ -2,15 +2,21 @@

The High Quality Camera contains an IR filter, which is used to reduce the camera’s sensitivity to infrared light. This ensures that outdoor photos look more natural. However, some nature photography can be enhanced with the removal of this filter; the colours of sky, plants, and water can be affected by its removal. The camera can also be used without the filter for night vision in a location that is illuminated with infrared light.

**This procedure cannot be reversed:** the adhesive that attaches the filter will not survive being lifted and replaced, and the IR filter may crack when it is removed. Removing it will void the warranty on the product. Nevertheless, removing the filter will be desirable to some users.
**This procedure cannot be reversed:** the adhesive that attaches the filter will not survive being lifted and replaced, and while the IR filter is about 1.1mm thick, it may crack when it is removed. Removing it will void the warranty on the product. Nevertheless, removing the filter will be desirable to some users.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given @l;urch comment about not knowing the filter is attached o the lens mount, perhaps that should be mentioned?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe - or perhaps that'll be more obvious if you actually have an HQ camera in front of you?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've asked Simon for his input, busy as he is, because it needs someone who has a unit in front of them to refer to.

@helenlynn
Copy link

@JamesH65 I'm happy with this addition of detail and I'll copy-edit once we're done discussing wording. I agree with @aallan that we needn't go to the lengths of making a video to demonstrate how one can go about voiding one's warranty. I can't see the images - they 404 for me. If there's an obvious dumb thing I'm probably doing, please suggest what it might be.

@lurch
Copy link
Contributor

lurch commented May 27, 2020

@aallan
Copy link
Contributor

aallan commented Jun 7, 2021

I'd encourage you to wrap this PR up in the next week or so as we're in the process of transitioning the documentation from the current Markdown-based source format to Asciidoc. At some point soon — probably around the end of June, beginning of July — we will freeze the current documentation repo. After that time contributions and PRs based on the Markdown source will not be accepted, and any PRs that are still open will be closed.

See #1911 for full details.

@aallan
Copy link
Contributor

aallan commented Jun 8, 2021

Reading the thread it seems everyone is happy with the changes. I'll mark this as ready to merge ahead of #1911. Speak now or forever hold your peace…

@aallan aallan added the ready to merge The OP says this PR is ready to merge? Anyone object? label Jun 8, 2021
@JamesH65 JamesH65 merged commit 8033d48 into raspberrypi:master Jun 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge The OP says this PR is ready to merge? Anyone object?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants