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

Add troubleshooting page #533

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

melissawm
Copy link
Member

References and relevant issues

Closes #40, #493

Description

Adds a troubleshooting page under Usage. Not complete by any means, but a starting point. It has two sections, one for known issues and another for an FAQ.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Dec 5, 2024
@psobolewskiPhD
Copy link
Member

Wowza this is great!!

docs/troubleshooting.md Outdated Show resolved Hide resolved
@psobolewskiPhD
Copy link
Member

psobolewskiPhD commented Dec 6, 2024

Eep, the circleCI fail is on main too now.
constraints were supposed to protect us from this, what happened?

napari/napari runs seem fine
Edit: i did a fresh merge of main into our my PR to check...
well, maybe not fine -- there are a lot of tracebacks, but it does complete:
https://app.circleci.com/pipelines/github/napari/napari/7473/workflows/a35c80e6-75ff-4556-8298-6c3501c4b0f0/jobs/7280

docs/troubleshooting.md Outdated Show resolved Hide resolved
docs/troubleshooting.md Outdated Show resolved Hide resolved

### My image renders as all black/all white

This can happen when the contrast limits are not set correctly. You can reset the contrast limits by right-clicking "contras limits", then clicking the "Reset" button in the dialog shown.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
This can happen when the contrast limits are not set correctly. You can reset the contrast limits by right-clicking "contras limits", then clicking the "Reset" button in the dialog shown.
This can happen when the contrast limits are not set correctly. You can reset the contrast limits by right-clicking "contrast limits", then clicking the "Reset" button in the dialog shown.

But I think the advice should be more extensive here:

The contrast limits are the minimum and maximum values displayed by the layer. Any value under the smaller contrast limit will appear as black, while any value above the larger limit will appear as white. Additionally, when all the values are near the bottom of the range, the image can still appear black even if there are some very dark but not quite black pixels there.

You can reset the contrast limits by clicking the Auto-Contrast: "Once" button, or, if that fails, right clicking on the contrast limits to do manual adjustment, including directly editing the numbers shown. When in doubt, you can hover over the canvas with your image layer selected, and check the status message as you move the mouse around. That will give you an idea of the range and variability of the data in your layer, helping you set meaningful contrast limits.

This maybe points to writing a more extensive contrast limits and colormaps tutorial that we can link to from here, but otherwise I think it's worth doing a couple of paragraphs to help people understand what they are reading. (Screenshots would also be good here I think.)

Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe this is content for the viewer tutorial? Or do you think we need a dedicated page for this?

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, I think we need better documentation for contrast limits and then we can keep this part brief. see also #535

Copy link
Member Author

Choose a reason for hiding this comment

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

Ok - I created #542 and updated this PR to link to that doc (when it's live).

docs/troubleshooting.md Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document napari --reset as a way to reset settings to default (and solve problems)
3 participants