feat: use semantic colors for heatmap in event probability columns#371
feat: use semantic colors for heatmap in event probability columns#371johanneskoester merged 2 commits intomasterfrom
Conversation
WalkthroughThis change updates the YAML configuration for plotting heatmaps. The regex matching logic is modified to separate probability values. The original pattern has been replaced with one that excludes the terms "artifact" and "absent", and a new pattern is introduced to specifically target these terms. The two branches now apply different color ranges and scales based on the matched value. Changes
Sequence Diagram(s)sequenceDiagram
participant Data as Probability Data
participant Regex as Regex Filter
participant Plot as Heatmap Plotter
Data->>Regex: Evaluate input "prob: <value>"
alt Value matches "artifact" or "absent"
Regex-->>Plot: Apply regex("prob: (artifact|absent)")
Plot->>Plot: Set color scale white to #d4322c
else Other value
Regex-->>Plot: Apply regex("prob: (?!artifact|absent)")
Plot->>Plot: Set color scale white to #1f77b4
end
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
⏰ Context from checks skipped due to timeout of 90000ms (7)
🔇 Additional comments (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Summary by CodeRabbit