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.

Sign up for GitHub

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

Jump to bottom

Client work for AI Expression Summary on gene page #1330

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

Client work for AI Expression Summary on gene page #1330

wants to merge 28 commits into from

Conversation

bobular
Copy link
Member

@bobular bobular commented Feb 20, 2025 •
edited
Loading

To do

  • links to ExpressionGraphs need mouse pointer and possible debugging when section is not open?
  • use # Microarray Datasets and # RNASeq Datasets record attributes as an additional "gate" to show the new feature or "This feature will be made available when there are X or more transcriptomics experiments available for this gene.
  • feature flag to ready this for merging web-monorepo WON'T DO
  • double-click in the table cells should not open-close the child row - then hopefully users can select and copy text from there. - WON'T FIX - see Provide better click handling for Mesa DataCells #1339

@bobular bobular mentioned this pull request Feb 25, 2025
Open
@bobular bobular requested review from dmfalke and asizemore February 26, 2025 17:15
@bobular bobular mentioned this pull request Feb 28, 2025
Merged
8 tasks
@bobular bobular marked this pull request as ready for review March 2, 2025 16:46
@bobular bobular requested a review from aurreco-uga March 2, 2025 16:46
bobular
bobular commented Mar 4, 2025
View reviewed changes
packages/libs/components/src/components/tidytree/TreeTable.tsx
@@ -4,7 +4,7 @@ import {
HorizontalDendrogramProps,
} from '../../components/tidytree/HorizontalDendrogram';
import Mesa from '@veupathdb/coreui/lib/components/Mesa';
import { MesaStateProps } from '../../../../coreui/lib/components/Mesa/types';
import { MesaStateProps } from '@veupathdb/coreui/lib/components/Mesa/types';
Copy link
Member Author

Choose a reason for hiding this comment

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

This was just a tiny fix while in the neighbourhood. Has no impact on the AI stuff!

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

@aurreco-uga aurreco-uga aurreco-uga approved these changes

@dmfalke dmfalke Awaiting requested review from dmfalke

@asizemore asizemore Awaiting requested review from asizemore

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bobular @aurreco-uga