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.

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

Updates for CSV downloads: Header letter casing unification & add column headers for edge downloads #647

Open
2 tasks done
LibbyUX opened this issue Aug 14, 2024 · 2 comments
Assignees
Labels
app: cde-ui Cell Distance Explorer enhancement A feature enhancement to a current interface high priority Priority task for product team

Comments

@LibbyUX
Copy link
Contributor

LibbyUX commented Aug 14, 2024

@katyb requests:

  • CSV files downloaded from the CDE have headers with unified letter casing.
  • Edge CSV file downloads have a header (with unified letter casing).

This is ready for development now and may be implemented in the CDE immediately.

Node CSV header columns in question:

image

Node CSV download with unified letter casing:

image

Edge downloads without a header:

image

Notes

  • These two updates were initially requested on Create Vis App MVP2 #579, but it evolved into a different feature.
  • This was identified as the priority 1 feature for the CDE paper by @J-Yash today, see CDE Y3 feature requests #489
  • Regarding the edge downloads: I know this download is for embedding the application. Let me know if we need to break this up into two appropriately placed buttons: a "vanity" button (for lack of a better term at the moment) for edges CSVs with a header and another for edge downloads without a header for embedding the app.
@LibbyUX LibbyUX added enhancement A feature enhancement to a current interface high priority Priority task for product team app: cde-ui Cell Distance Explorer product team: developers labels Aug 14, 2024
@LibbyUX LibbyUX added this to the Cell Distance Explorer MVP2 milestone Aug 14, 2024
@LibbyUX LibbyUX changed the title Updates for CSV downloads: Header letter casing unification & add edge headers Updates for CSV downloads: Header letter casing unification & add column headers for edge downloads Aug 14, 2024
@edlu77 edlu77 self-assigned this Sep 27, 2024
@LibbyUX
Copy link
Contributor Author

LibbyUX commented Oct 2, 2024

@axdanbol & I collaborate:

  • Ignore casing when loading data. Link headers are optional
  • Cell CSV headers: X, Y, Cell Type, [Z, Cell Ontology ID]
  • Link CSV headers: TBD, I am sending an email on this now.

@LibbyUX
Copy link
Contributor Author

LibbyUX commented Oct 4, 2024

@axdanbol, @bherr2 - per @J-Yash and for tracking here:

The column names/order Cell ID, X1, Y1, Z1, X2, Y2, Z2 is finalized.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app: cde-ui Cell Distance Explorer enhancement A feature enhancement to a current interface high priority Priority task for product team
Projects
None yet
Development

No branches or pull requests

2 participants