ESHackathon / CiteSource

http://www.eshackathon.org/CiteSource/
GNU General Public License v3.0
16 stars 1 forks source link

RE-Importing a csv file exported from CiteSource into Shiny app #177

Closed rootsandberries closed 1 month ago

rootsandberries commented 4 months ago

When I import a csv file I exported from the Shiny app, it seems to upload but then there is nothing showing under the Step 2 header. When I click on find duplicates, I get the message "Data needed: Please import your citations first", but then when I click on visualize, it seems to have the data. I think this is just a matter of adjusting some of the messaging when users are uploading CiteSource files?

TNRiley commented 3 months ago

Same, and agreed. I think we just need to make it clear that the user can skip to the visualization tab.

LukasWallrich commented 3 months ago

Or actually enable re-deduplication - users might have fixed meta-data. So maybe upload as both raw and deduplicated data so that user can decide.

TNRiley commented 2 months ago

Currently, CSV is able to be uploaded. Heatmap and Upsetplot are able to be created.

No deduplication is possible at this time and both tables will throw an error.

LukasWallrich commented 2 months ago

For now, I added a popup confirming that reimport was successful and that the user can continue to figures and tables, and specific information on editing meta-data and deduplication.

It will be difficult to enable deduplication without getting nonsense in the tables that refer back to the 'original' data - so I think that will have to wait until we refactor these.

TNRiley commented 1 month ago

This is an issue once again since I reverted the script to resolve #198, need to find another fix.

EDIT: actually running through this all is working well now. There is one table that is not appearing on re-upload, but I will make an issue for that.