Skip to content

fix: remove fill from bbox overlays#124

Open
MateoLostanlen wants to merge 2 commits intomainfrom
fix/bbox-no-fill
Open

fix: remove fill from bbox overlays#124
MateoLostanlen wants to merge 2 commits intomainfrom
fix/bbox-no-fill

Conversation

@MateoLostanlen
Copy link
Copy Markdown
Member

  • The semi-transparent fill inside bounding boxes obscured pixels and made fire/smoke analysis harder. Keep only the colored border.
  • Removes bg-*/20 fill from AI-prediction overlays (sequence player, full-image sequence, BoundingBoxOverlay) and from the in-progress drawing rectangle.
  • Drops the now-unused background field from SmokeTypeColors and replaces the switch with a typed Record lookup.
  • Closes Sequences : Remove semi-transparent red fill colour #109.

The semi-transparent fill obscured pixels inside bounding boxes,
which made fire/smoke analysis harder. Keep only the colored border.

- Drop bg-*/20 fill on AI-prediction overlays (sequence player,
  full-image sequence, ImageOverlays.BoundingBoxOverlay)
- Drop bg-blue-400/20 on the in-progress drawing rectangle
- Drop the unused background field from SmokeTypeColors and update
  user-annotation / drawn-rectangle overlays accordingly

Closes #109
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sequences : Remove semi-transparent red fill colour

2 participants