remove utf8 encoding from csv read for latest CEC modules#141
Conversation
|
Goes with SAM pull request NatLabRockies/SAM#1066 |
|
@sjanzou It looks like we originally added UTF-8 to support unicode in file names: 0fbe95d. I don't see a related issue in WEX, SSC, or SAM so am not sure what problem that fixed, but we should avoid reintroducing whatever that problem was. The current proposed fix is to avoid issues when reading CSV files created from the CEC module and inverter equipment list Excel files. I think we can address this in our effort to improve our CEC library process instead of modifying WEX: When we convert the CEC data from Excel into SAM's inverter and module library CSV files, the unicode characters can cause problems in the SAM user interface, and may be the cause of some problems we are having with SSC/6parsolve. One workaround is to manually remove unicode characters from the CSV file with regular expression search and replace |
@cpaulgilman, agreed. I will remove this pull request and the related SAM pull request 1066 and we will keep the CEC processing of Unicode separately and can revisit in the future, if necessary. |
No description provided.