brendanjmeade / celeri

Next generation earthquake cycle kinematics
BSD 3-Clause "New" or "Revised" License
24 stars 7 forks source link

Needs testing/fixing: get_shared_sides last_occurence #34

Closed brendanjmeade closed 3 years ago

brendanjmeade commented 3 years ago

This variable should be the index of the row in which a unique value last appears in an array a la Matlab's unique(arr, "rows", "last"). I don't the current arr = np.flipud(arr) is sufficient. Maybe try last_occurence = n_tris - last_occurence?

brendanjmeade commented 3 years ago

Discussed with @jploveless and fixed along the lines described above and committed sometime before: https://github.com/brendanjmeade/celeri/commit/934fad1c9c25046b047bda805f6c01d6a98b4a01