fiftysevendegreesofrad / sdna_plus

Fully open release with all advanced sDNA+ features
Other
10 stars 1 forks source link

Simplification. Initialise with vector.assign(length, value) instead of vector.swap(<vector<...()) #58

Open JamesParrott opened 1 month ago

JamesParrott commented 1 month ago

To compile on Zig, use of temporary variables is more restricted. A number of changes were made from:

        skim_matrix_sum_distance.swap(vector<vector<long double>>(skim_origins.size(),vector<long double>(skim_destinations.size(),0.)));

etc. to:

        skim_matrix_sum_distance.assign(skim_origins.size(),vector<long double>(skim_destinations.size(),0.));

etc.

E.g.: https://github.com/fiftysevendegreesofrad/sdna_plus/blob/9d73c6cee71e205788f62eb7a17d4d6a43968045/sDNA/sdna_vs2008/calculation.cpp#L292

JamesParrott commented 1 month ago

Included in #74 on Cross_Platform branch