Attaching Final Skims Fails for Large Files #308

Activity

1. mabcal commented on Oct 8, 2021

   

   mabcal

   on Oct 8, 2021 via email

   Contributor

   More actions

   In the Copenhagen version, I changed some id fields to 16 byte integers, which should also solve the problem. Will also do that for the main version. Sent from my T-Mobile 5G Device Get Outlook for Android<https://aka.ms/AAb9ysg>

   …

   ________________________________ From: Brice Nichols ***@***.***> Sent: Thursday, October 7, 2021 3:50:58 PM To: RSGInc/DaySim ***@***.***> Cc: Subscribed ***@***.***> Subject: [RSGInc/DaySim] Attaching Final Skims Fails for Large Files (#308) CAUTION - EXTERNAL EMAIL When running DaySim in estimation mode to attach final skims to daysim outputs, some files crash in Reader.cs. This particularly happens when large files like full simulation daysim data like trips are loaded. The point of failure is here, where the length of the file being imported is converted to an integer: https://github.com/RSGInc/DaySim/blob/master/DaySim.Framework/Persistence/Reader.cs#L41 For large files this causes an overflow of the integer, resulting in a negative value which fails to create a proper index. We've found that the following correction of line 41 will avoid this issue by converting the value to an integer only after dividing the file length by the file's size value: _count = (int)(file.Length / _size); — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#308>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ADWCYPB4PPQWEHFDDGZ6673UFYP5FANCNFSM5FSKU63Q>. Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.