solgenomics / sgn

The code behind the Sol Genomics Network, Cassavabase and other Breedbase websites
https://solgenomics.net
MIT License
67 stars 35 forks source link

Add xlsx format option to pedigree upload #4671

Open titima15 opened 9 months ago

titima15 commented 9 months ago

Expected Behavior

For Bugs:

Environment

Steps to Reproduce

ch728 commented 1 month ago

This would be very helpful! Explaining to staff the difference between an "Excel file" and a tab-delimited file can be very difficult. Also would improve user experience to have a consistent file type for uploads. I believe this is one of the few tab-delimited uploads left in the system.

titima15 commented 1 month ago

@ch728 I will change priority for this ticket to high. Hopefully I can start working on it early next week.

dwaring87 commented 3 weeks ago

@titima15 Have you started working on this? If not, I've started working on a Generic Data File Parser that would support csv, tab-delimited, and excel files that could be used for the different file uploads.

titima15 commented 3 weeks ago

@dwaring87 Yes, I'm working on it and it's almost done. However, If you are going to finish that File Parser soon. We can use that for pedigrees as well.

dwaring87 commented 3 weeks ago

@titima15 Sorry, I should have started this sooner! The file parser is working now, I've just updated the locations upload as an example (in the topic/generic_file_parser branch). Would you like me to convert the pedigree upload to this or would you like to give it a try?

titima15 commented 3 weeks ago

@dwaring87 No problem. Yes, I would like to give it a try. I will be traveling Monday and Tuesday. I will take a look after I get back. Thanks.

dwaring87 commented 3 weeks ago

@titima15 Great, here is the Location upload I just converted if you want to use that as an example. Let me know if you have any questions using it.