Skip to content

Handle streamtube coordinates in string format and handle different data orders in isosurface and volume #4431

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Dec 30, 2019
Merged

Handle streamtube coordinates in string format and handle different data orders in isosurface and volume #4431

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
07ce3d2
fix issue 4427 - streamtube should handle string values
archmoj Dec 18, 2019
942f92f
streamtube - no need for casting typed arrays to numbers
archmoj Dec 21, 2019
d015b57
create a reusable process grid function
archmoj Dec 24, 2019
5a95dab
Apply streamtube processGrid in isosurface and volume
archmoj Dec 24, 2019
1f0ee0f
improve test image for isosurface
archmoj Dec 25, 2019
990cdda
match isosurface value in the mock to be equal to sqrt(u2+v2+w2)
archmoj Dec 25, 2019
b71677a
pass len to distictVals and filter to help improve the performance
archmoj Dec 27, 2019
6d06229
add an image test for volume with different fill orders
archmoj Dec 27, 2019
df9b4c4
cast x, y, z, u, v, w, values to numbers at the start of calc and sto…
archmoj Dec 27, 2019
41504aa
remove extra cast operations
archmoj Dec 29, 2019
c82da07
rewrite without forEach
archmoj Dec 30, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions src/traces/streamtube/calc.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -206,6 +206,9 @@ function distinctVals(col) {
function filter(arr, len) {
if(len === undefined) len = arr.length;

// no need for casting typed arrays to numbers
if(Lib.isTypedArray(arr)) return arr.slice(0, len);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's better to use arr.subarray() for typed arrays, as .slice isn't supported in older browsers.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call.
Wondering if we need to slice the array at all?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

slice (or subarry) is needed for distictVals to work correctly.
But it may be better not to slice if the length is correct.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But it may be better not to slice if the length is correct.

that could be a nice optimisation for typed arrays, but for regular arrays we'll need to slice in general to cast numerical-strings items w/o mutating.


var values = [];
for(var i = 0; i < len; i++) {
values[i] = +arr[i];
Expand Down