Skip to content

Add ocular health visualization suite#1

Open
zanax1990 wants to merge 1 commit intomainfrom
codex/create-correlation-heatmaps-and-plots
Open

Add ocular health visualization suite#1
zanax1990 wants to merge 1 commit intomainfrom
codex/create-correlation-heatmaps-and-plots

Conversation

@zanax1990
Copy link
Owner

Summary

  • add a curated ocular health dataset, shared utilities, and a scripted analysis pipeline that generates correlation, box/violin, and clustering visuals
  • introduce a Plotly Dash dashboard so the demographic, disease, and Schwann cell metrics can be explored interactively
  • document the workflow with a focused README and dependency list tailored for the new project

Testing

  • ⚠️ pip install -r requirements.txt (fails: ProxyError – Tunnel connection failed: 403 Forbidden)

Codex Task

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +115 to +119
violin_fig = px.violin(
filtered_long[filtered_long["Disease"] == disease],
x="Disease",
y="SchwannCellDensity",
box=True,

Choose a reason for hiding this comment

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

P1 Badge Prevent callback errors when filters yield no rows

The Dash callback builds the violin plot from filtered_long[filtered_long['Disease'] == disease] without checking whether that slice contains data. If the age slider is narrowed to a range that has no patients with the selected disease (which is easy to hit with this 30‑row sample set), plotly.express.violin raises ValueError: DataFrame is empty, causing the callback to fail and the dashboard graphs to stop updating. Guard for empty frames or return a placeholder figure before calling Plotly.

Useful? React with 👍 / 👎.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant