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

[BUG] If a field is outside the image nothing is indicated in overlay manager #3317

Closed
EricClaeys opened this issue Jan 14, 2024 · 4 comments
Assignees
Labels
bug Needs software change
Milestone

Comments

@EricClaeys
Copy link
Collaborator

There log file contains ERROR: Field 'ZWO ' is outside of the image but nothing in the Overlay Editor to indicate that OR to fix it. The user has to know which .json file to fix and manually do so.

@EricClaeys EricClaeys added the bug Needs software change label Jan 14, 2024
@EricClaeys EricClaeys added this to the v2024.xx.xx milestone Jan 14, 2024
@morrowwm
Copy link

morrowwm commented Jan 19, 2024

I had this issue also. While I was learning how to use the overlay editor, I think I put ${CAMERA_TYPE} ${CAMERA_MODEL} at a huge "y" position. Or perhaps something put the item there by default?

The error log reported:

ERROR: Field 'RPi ArduCam_16_MP' is outside of the image

I found this configuration in allsky/config/overlay/config/overlay.json, with "y" value of 3008. I changed that to 200, then the item showed up in the overlay editor, where I could delete it.

You could test this by typing in an off-image value in the x or y Position fields.

Is it only ever _overlayConfigFile which can have these items?

For a quick fix, maybe just add the filename to the error message?

    652             if outOfBounds:
    653                 s.log(0, f"ERROR: Field '{fieldLabel}' is outside of the image. Check config in {self._overlayConfigFile}")  

@EricClaeys
Copy link
Collaborator Author

@morrowwm, the default position for CAMERA_TYPE, CAMERA_MODEL is fixed in the next release. The current location assumed it was on an RPi HQ camera or a high-resolution ZWO camera.

This Issue is to have the Overlay Editor tell you when a field is outside the image and to put it somewhere in the image so you can fix it or delete it. That should eliminate the need to change the outOfBounds message, although I suppose it should say "Fix in the Overlay Editor".

@morrowwm
Copy link

OK, good to know.

@Alex-developer
Copy link
Collaborator

Fix committed to the next release branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Needs software change
Projects
None yet
Development

No branches or pull requests

3 participants