search

Btibert3 / pypeds

Python package to facilitate and standardize enrollment science within higher education and Enrollment Management
https://www.linkedin.com/pulse/introducing-pypeds-brock-tibert/
GNU General Public License v3.0
9 stars 6 forks source link

Updated zip_parser for Windows users #21

Closed phaller0513 closed 1 year ago

phaller0513 commented 2 years ago

I believe this should also work on Mac, but I have no way to test it at the moment.

/tmp/ is not a valid directory in Windows and os.mkdir cannot create multiple paths at once. Moved mkdir calls into their own sections for both /tmp/ and path. Updated glob to only pull csv files. The zip files were being deleted each time on load.

Btibert3 commented 1 year ago

I get the direction of the change here, but its worth noting that this will leave artifacts on the computer which was the idea of /tmp/ to keep a short memory. However, not the end of the world and something to revisit later. Thanks for thinking of this.