seqeyes-python 0.3.12__tar.gz → 0.3.13__tar.gz

{seqeyes_python-0.3.12 → seqeyes_python-0.3.13}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: seqeyes-python
-Version: 0.3.12
+Version: 0.3.13
 Summary: Interactive Pulseq MRI sequence viewer for Jupyter — lightweight, beautiful, replaces seq.plot() in pypulseq.
 Project-URL: Homepage, https://github.com/bughht/seqeyes_plugin
 Project-URL: Repository, https://github.com/bughht/seqeyes_plugin

{seqeyes_python-0.3.12 → seqeyes_python-0.3.13}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "seqeyes-python"
-version = "0.3.12"
+version = "0.3.13"
 description = "Interactive Pulseq MRI sequence viewer for Jupyter — lightweight, beautiful, replaces seq.plot() in pypulseq."
 readme = "README.md"
 license = {text = "MIT"}

{seqeyes_python-0.3.12 → seqeyes_python-0.3.13}/src/seqeyes/__init__.py

@@ -29,4 +29,4 @@ from seqeyes._plot import set, reset
 from seqeyes._renderer import SeqEyesViewer
 
 __all__ = ["set", "reset", "SeqEyesViewer"]
-__version__ = "0.3.12"
+__version__ = "0.3.13"

{seqeyes_python-0.3.12 → seqeyes_python-0.3.13}/src/seqeyes/resources/pulseq-bundle.js

@@ -89,7 +89,7 @@ var Pulseq = (() => {
   });
 
   // package.json
-  var version = "0.3.12";
+  var version = "0.3.13";
 
   // src/pulseq/decompressor.ts
   function decompressShape(compressed, numSamples) {

{seqeyes_python-0.3.12 → seqeyes_python-0.3.13}/src/seqeyes/resources/webview-bundle.js

@@ -3153,6 +3153,16 @@ if(systemThemeQuery){
    WebGL state
    ═══════════════════════════════════════════════════════════════════════ */
 var gl=null, glProgram=null, glBuf=null, glN=0;
+/* Decimation index buffer.  WebGL caps a vertex attribute stride at 255 bytes
+   and this buffer is 16 bytes a point, so `16*stride` is rejected outright for
+   any stride above 15 — silently, since a failed vertexAttribPointer only sets
+   an error nobody reads.  The attribute then keeps its previous unstrided
+   setup while the draw range is still scaled down, so what reaches the screen
+   is one contiguous run of the acquisition rather than a thinned cloud: a slab
+   at one end of the encode.  Selecting the same points through an index buffer
+   has no such limit. */
+var glIdxBuf=null, glUint32Indices=false;
+var _idxFirst=-1, _idxCount=-1, _idxStride=-1, _idxN=0;
 /* Why the GPU layer has nothing to draw, or null when it is healthy.  A
    failure here used to be a console warning at most, so the panel opened
    with its axes and no points — indistinguishable from a sequence whose
@@ -3182,6 +3192,10 @@ function initWebGL(){
     kGpuFailure="the graphics context was lost while drawing K\u2011space";
     if(typeof drawKs==="function")drawKs();
   },false);
+  // WebGL2 always has 32-bit indices; WebGL1 needs the extension for a cloud
+  // longer than 65535 points, which every sequence reaching the stride has.
+  glUint32Indices=(typeof WebGL2RenderingContext!=='undefined'&&gl instanceof WebGL2RenderingContext)
+    ||!!gl.getExtension('OES_element_index_uint');
   gl.enable(gl.BLEND);gl.blendFunc(gl.SRC_ALPHA,gl.ONE_MINUS_SRC_ALPHA);
 
   var vs=gl.createShader(gl.VERTEX_SHADER), fs=gl.createShader(gl.FRAGMENT_SHADER);
@@ -3249,6 +3263,23 @@ var KSPACE_MOVING_POINT_TARGET = 600000;
  *
  * Returns 1 below the target, so narrow views and modest sequences are drawn
  * exactly even in motion.
+ *
+ * The stride is rounded up to a prime, which costs a handful of points and
+ * avoids aliasing with the readout.  Sampling every k-th sample of a sequence
+ * whose readouts are R samples long only ever lands on R/gcd(k,R) of the R
+ * positions within a readout — and on the same ones in every readout.  When an
+ * axis is swept during the readout, as in a wave or CAIPI sequence, those
+ * positions are k-space planes, so a shared factor drops whole planes rather
+ * than thinning the cloud evenly.  gre_3d_wave_FC.seq sweeps kz across its
+ * full range inside each 1000-sample readout; at stride 56, gcd 8 with 1000
+ * left 125 of the 1000 positions and 12% of the planes.  An odd prime shares
+ * no factor with a readout length it does not divide, which restores 96% —
+ * indistinguishable from sampling at random.
+ *
+ * Odd, not merely prime: readout lengths are usually even, so a stride of 2
+ * would reach only every second position and lose half the planes.  The point
+ * target is a ceiling rather than a goal, so rounding 2 up to 3 draws somewhat
+ * fewer points and stays well inside budget.
  */
 function kSpaceMovingStride(count){
   // The target is overridable so the settle-exactness guarantee can be tested
@@ -3256,7 +3287,40 @@ function kSpaceMovingStride(count){
   var target=(typeof window!=='undefined'&&window.__kMovingPointTarget>0)
     ?window.__kMovingPointTarget:KSPACE_MOVING_POINT_TARGET;
   if(!(count>target))return 1;
-  return Math.ceil(count/target);
+  return nextOddPrimeAtLeast(Math.ceil(count/target));
+}
+
+/** Smallest odd prime >= n, for n small enough that trial division is free. */
+function nextOddPrimeAtLeast(n){
+  var candidate=(n>3)?n:3;
+  if(candidate%2===0)candidate++;
+  while(!isSmallPrime(candidate))candidate+=2;
+  return candidate;
+}
+
+function isSmallPrime(n){
+  if(n<2)return false;
+  if(n%2===0)return n===2;
+  for(var d=3;d*d<=n;d+=2)if(n%d===0)return false;
+  return true;
+}
+
+/**
+ * Upload indices selecting every `stride`-th point of [first, first+count),
+ * returning how many there are.  Cached on the range, so a drag uploads once
+ * rather than per frame.
+ */
+function kSpaceStrideIndices(first,count,stride){
+  if(_idxFirst===first&&_idxCount===count&&_idxStride===stride)return _idxN;
+  var n=Math.floor(count/stride);
+  _idxFirst=first;_idxCount=count;_idxStride=stride;_idxN=n;
+  if(n<=0)return 0;
+  var idx=new Uint32Array(n);
+  for(var i=0;i<n;i++)idx[i]=first+i*stride;
+  if(!glIdxBuf)glIdxBuf=gl.createBuffer();
+  gl.bindBuffer(gl.ELEMENT_ARRAY_BUFFER,glIdxBuf);
+  gl.bufferData(gl.ELEMENT_ARRAY_BUFFER,idx,gl.DYNAMIC_DRAW);
+  return n;
 }
 
 /** True while a drag, touch gesture, or camera easing is in progress. */
@@ -3325,6 +3389,7 @@ function uploadKSpaceGPU(){
     return;
   }
   glN=n;
+  _idxStride=-1;  // the indices point into the buffer just replaced
   // Compute bounds once
   var xmin=Infinity,xmax=-Infinity,ymin=Infinity,ymax=-Infinity,zmin=Infinity,zmax=-Infinity;
   for(var a=0;a<n;a++){var xi=ax[a],yi=ay[a],zi=az[a];if(isFinite(xi)){if(xi<xmin)xmin=xi;if(xi>xmax)xmax=xi;}if(isFinite(yi)){if(yi<ymin)ymin=yi;if(yi>ymax)ymax=yi;}if(isFinite(zi)){if(zi<zmin)zmin=zi;if(zi>zmax)zmax=zi;}}
@@ -3783,17 +3848,24 @@ function drawKs_core(W,H,dpr){
 
     var win=kSpaceWindowRange(kAdcTime,glN,vs,ve);
     var stride=kSpaceCameraMoving()?kSpaceMovingStride(win.count):1;
+    if(stride>1&&!glUint32Indices)stride=1;  // draw it all rather than wrongly
 
     gl.bindBuffer(gl.ARRAY_BUFFER,glBuf);
     gl.enableVertexAttribArray(glAttribPos);
-    gl.vertexAttribPointer(glAttribPos,3,gl.FLOAT,false,16*stride,0);
+    gl.vertexAttribPointer(glAttribPos,3,gl.FLOAT,false,16,0);
     gl.enableVertexAttribArray(glAttribTime);
-    gl.vertexAttribPointer(glAttribTime,1,gl.FLOAT,false,16*stride,12);
+    gl.vertexAttribPointer(glAttribTime,1,gl.FLOAT,false,16,12);
 
     kSpaceTrajectoryDrawCount++;
-    // Stride reindexes the buffer, so the window offset and count scale with it.
-    var drawFirst=Math.floor(win.first/stride),drawCount=Math.floor(win.count/stride);
-    if(drawCount>0)gl.drawArrays(gl.POINTS,drawFirst,drawCount);
+    if(stride>1){
+      var drawn=kSpaceStrideIndices(win.first,win.count,stride);
+      if(drawn>0){
+        gl.bindBuffer(gl.ELEMENT_ARRAY_BUFFER,glIdxBuf);
+        gl.drawElements(gl.POINTS,drawn,gl.UNSIGNED_INT,0);
+      }
+    }else if(win.count>0){
+      gl.drawArrays(gl.POINTS,win.first,win.count);
+    }
   }
 
   drawKsOverlay(W,H,dpr,cs);
@@ -7273,7 +7345,13 @@ function placeTooltip(cx,cy){
   var r=tt.getBoundingClientRect(),vw=window.innerWidth,vh=window.innerHeight;
   var left=Math.min(Math.max(pad,cx+15),Math.max(pad,vw-r.width-pad));
   var top=cy+12;if(top+r.height+pad>vh)top=cy-r.height-12;if(top<pad)top=pad;
-  tt.style.left=left+'px';tt.style.top=top+'px';
+  /* `left`/`top` are client coordinates, but #tt is absolutely positioned
+     inside #cc, so its style offsets are measured from #cc's box.  Writing
+     client coordinates straight out put the tooltip #cc's own offset too low
+     and made the bottom flip fire that far past the real bottom of the
+     window. */
+  var host=tt.offsetParent?tt.offsetParent.getBoundingClientRect():{left:0,top:0};
+  tt.style.left=(left-host.left)+'px';tt.style.top=(top-host.top)+'px';
 }
 function fmtFlipAngle(value){
   if(!isFinite(value))return null;