Skip to content

Update glvis-js v0.4 -> v0.5#26

Open
justinlaughlin wants to merge 4 commits intomasterfrom
update-to-glvis-js-v0.5
Open

Update glvis-js v0.4 -> v0.5#26
justinlaughlin wants to merge 4 commits intomasterfrom
update-to-glvis-js-v0.5

Conversation

@justinlaughlin
Copy link
Contributor

@justinlaughlin justinlaughlin commented Feb 10, 2026

Updating to glvis-js v0.5

-  mfem: "v4.8",
-  glvis: "v4.4",
+  mfem: "v4.9",
+  glvis: "v4.5",

@najlkin
Copy link
Contributor

najlkin commented Feb 10, 2026

Are all (new) keys supported? 🤔

@najlkin
Copy link
Contributor

najlkin commented Feb 11, 2026

Also there are more examples now.

@tzanio tzanio self-requested a review February 11, 2026 16:09
@najlkin
Copy link
Contributor

najlkin commented Feb 13, 2026

TODO:

  • Update data submodule
  • Add examples to src/live/index.html
  • fix resolution
  • fix example 22 - vector

Optional:

  • Add buttons for new(ish) features, but as they are context-dependent, it cannot be statically in HTML, but dynamically enabled/added by JS perhaps? 🤔

@najlkin
Copy link
Contributor

najlkin commented Mar 3, 2026

I have updated data and added (hopefully) all missing examples, but I have noticed some bugs:

  • The new support for all commands is nice, but changes of window size cause some blurring do to rescaling, which is carried over to other examples. Possible solutions:
    • Filter out that command
    • Reset size after loading a different example
    • Remove the command from the streams (the only bad guys are distance.saved, shifted.saved and ex6-dofs-numbering.saved)
    • Remove the examples
  • Loading example 22 (vector) gives different results sometimes, not clear what is happening

@najlkin
Copy link
Contributor

najlkin commented Mar 5, 2026

Ok, I fixed the resolution problem, so all examples can be loaded 👍 .
That example 22 - vector depends on timing, sometimes U key is ignored 😬 . That is a bug in GLVis itself and has nothing (directly) to do with glvis-js or complex functions visualization. It is just the only example, which uses U key. Anyway, I would leave it for now and proceed. We will just know one of the examples is a bit funky 🤪 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants