Skip to content

Add CSV export #192

Open
johnarban wants to merge 13 commits intocosmicds:devfrom
johnarban:download-csv
Open

Add CSV export #192
johnarban wants to merge 13 commits intocosmicds:devfrom
johnarban:download-csv

Conversation

@johnarban
Copy link
Contributor

This PR adds CSV export for each dataset, as well as the ability to copy it to the clipboard so it can be pasted into CODAP or Google Sheets.

@patudom
Copy link
Contributor

patudom commented Jan 30, 2026

Thank you @johnarban, this is working well. I have a few suggestions that I think would help the user. We can discuss further if that would be helpful.

  • I ended up with a bunch of download files with no way of figuring out what goes with which dataset other than comparing them one by one. Is it possible to use the dataset label as the filename instead of "download (xx)"?

  • Use consistent labeling between displayed table data and downloaded csv file

Screenshot 2026-01-30 at 5 16 42 PM
  • In a similar vein, I'm wondering if the table display should put the uncertainty in its own column to match the downloaded table format rather than using the "value (uncertainty)" notation.

  • In general, I'm wondering if this table is the right format for displaying the data. No one is going to scroll through 500+ screens of numbers to get to the end of the table. What are we hoping students are going to learn from / do with the table display? If it's meant to be a view of the format of the data (kind of like how pandas shows the first ~10 and last ~10 rows of a dataframe), maybe we can just show the start and end of the table? But unless it's using up lots of additional memory to create the 500 pages of table display, maybe it's not hurting anything to leave it as is. 🤷‍♀️

Screenshot 2026-01-30 at 5 31 16 PM
  • In both the table & downloaded file, we use "column density" which I don't think is terminology we use anywhere else. Do we want that to say "Quantity"? And is there a non-cumbersome way to indicate the unit in the column header label?

  • Have you tried doing a copy/paste from the clipboard to codap, and was it successful? I tried copy/pasting to excel, and it gave me one giant column with all my comma-delimited content in each cell. If codap isn't smarter than that, then the copy to clipboard function won't be that useful, and I'm wondering if we want to have only the csv download button to simplify the layout.

  • Is there a reason the Download Table text label is a different color than the Copy to clipboard label?

Screenshot 2026-01-30 at 5 06 31 PM

@patudom
Copy link
Contributor

patudom commented Feb 3, 2026

Thanks for making these updates, @johnarban!

I think we need to handle the copy/paste vs. the file download differently (and keep the downloaded file as comma separated). The copy/paste is now working well, but when I download the file and open it in Excel (which I think is likely what the user would do), it treats all the data as being in one column.

Also, for the non-aggregated data, the columns in the copy/paste and download are out of order.

Screenshot 2026-02-03 at 11 17 54 AM

@patudom patudom marked this pull request as draft February 3, 2026 16:31
@Carifio24 Carifio24 mentioned this pull request Feb 3, 2026
@johnarban johnarban marked this pull request as ready for review February 4, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants