Julenmendieta/MILAB-6246_labelVisibility - #43
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request ensures that the 'Clone Id' axis label is visible by default across VDJ import workflows. Changes include adding a visibility annotation in the bulk processing template and updating a flag in the single-cell template. Feedback suggests using the 'a' helper function in the bulk template for better consistency and to ensure correct order priority.
| annotations: { | ||
| "pl7.app/label": "Clone Id", | ||
| "pl7.app/description": "A short, human-readable identifier for the clonotype." | ||
| "pl7.app/description": "A short, human-readable identifier for the clonotype.", | ||
| "pl7.app/table/visibility": "default" | ||
| } |
There was a problem hiding this comment.
For consistency with other column definitions in this file and with process-single-cell.tpl.tengo, it is recommended to use the helper function a (defined at line 20) instead of manually defining the annotations map. This also ensures that the orderPriority is consistently applied to the clonotype label column.
annotations: a(100000, true, {
"pl7.app/label": "Clone Id",
"pl7.app/description": "A short, human-readable identifier for the clonotype."
})
1 participant
No description provided.