scenerygraphics/SciView

Come for the sciview, stay for the scenery

Ulrik Günther

@skalarproduktraum

now i get a hard crash in the c runtime :-D

Kyle I S Harrington

@kephale

jgo's directory naming probably makes sense but seems a little silly

Ulrik Günther

@skalarproduktraum

same behaviour btw from git bash and cmd.exe, just checked for testing

Kyle I S Harrington

@kephale

if only it was the string's hash or smth

Ulrik Günther

@skalarproduktraum

windows 10 should not have a problem with that anymore (see link), but the behaviour is opt-in

nevertheless, there's a cpp exception uncaught:

Stack: [0x0000001459600000,0x00000014597f0000],  sp=0x00000014597e7630,  free space=1949k
Native frames: (J=compiled Java code, j=interpreted, Vv=VM code, C=native code)
C  [KERNELBASE.dll+0x34b59]
C  [VCRUNTIME140.dll+0x4411]
C  [ntdll.dll+0xa1406]
C  [_jpype.cp38-win_amd64.pyd+0x392ce]
C  [_jpype.cp38-win_amd64.pyd+0x3d3d2]
C  [_jpype.cp38-win_amd64.pyd+0x32585]
C  0x000001abcfb1ef97

Java frames: (J=compiled Java code, j=interpreted, Vv=VM code)
j  org.jpype.manager.TypeFactoryNative.newWrapper(JJ)V+0
j  org.jpype.JPypeContext.newWrapper(J)V+16
v  ~StubRoutines::call_stub
j  org.jpype.manager.TypeFactoryNative.newWrapper(JJ)V+0
j  org.jpype.JPypeContext.newWrapper(J)V+16
v  ~StubRoutines::call_stub
j  org.jpype.manager.TypeFactoryNative.newWrapper(JJ)V+0
j  org.jpype.JPypeContext.newWrapper(J)V+16
v  ~StubRoutines::call_stub

siginfo: EXCEPTION_UNCAUGHT_CXX_EXCEPTION (0xe06d7363), ExceptionInformation=0x0000000019930520 0x00000014597ec660 0x00007ff99217e7d0 0x00007ff992100000

Kyle I S Harrington

@kephale

jpype-project/jpype#928 ?

i wonder if it could be the jvm version as well

Ulrik Günther

@skalarproduktraum

@kephale that issues looks spot-on

and the jvm is the 11 azul jdk it's probably getting via conda

Kyle I S Harrington

@kephale

Hmph

Ulrik Günther

@skalarproduktraum

so i guess, we'd have to fix that nasty transitive dependency issue

7 replies

and i'd argue that probably most or even all of scenery's deps need to be transitive

@kephale it should be possible to use the versions of @elect86's PRs in the python scripts too, right?

just essentially replace the commit hashes with those from the PRs

Kyle I S Harrington

@kephale

@skalarproduktraum yeah should work (just need jitpack to build and such)

but i'd love to have it working with the gradle-based PRs

Ulrik Günther

@skalarproduktraum

@kephale along similar lines, what update site should be used if you want to put sciview in a current fiji atm?

the ageratum one?

Kyle I S Harrington

@kephale

yeah https://sites.imagej.net/sciview-ageratum/

which is maven based

gradle has yet to work on update sites

due to the transitive dep stuff

which is another reason that i was trying to get this working in python since it seems to reproduce the transitive dep issue without requiring fiddling with update sites

Ulrik Günther

@skalarproduktraum

@kephale the ageratum update site is missing essentially all natives

so i guess getting it going on gradle has ultrahigh prio now :D

Kyle I S Harrington

@kephale

ageratum is the last maven

thats the idea

it is managed here https://github.com/scenerygraphics/sciview-update-sites

and when ageratum is stable we archive it to a branch and move on

gradle has so many obstacles that we needed something to actually work

glad that ageratum is finally being tested on windows

folks are already using it, but afaik linux users only

if natives are missing, then we should solve it

Ulrik Günther

@skalarproduktraum

lwjgl natives are missing, spirvcrossj natives are missing

ffmpeg, jogl, jocl are there

Kyle I S Harrington

@kephale

so i'll note that it would be much better to switch to gradle here, but it has been multiple months and it just doesnt move fast enough so we had to do something since the sciview update site has been literally just bitrotting

Ulrik Günther

@skalarproduktraum

understood

Kyle I S Harrington

@kephale

hm it should still have the code for the lwjgl and spirvcrossj management https://github.com/scenerygraphics/sciview/blob/92add67189798228dbe68c164a87f5be965130d9/populate_fiji.sh#L148

Ulrik Günther

@skalarproduktraum

ya i was suspecting that, they're not there however

seems like they also never made it to the update site: https://sites.imagej.net/sciview-ageratum/jars/win64/

Kyle I S Harrington

@kephale

upps wrong link

Ulrik Günther

@skalarproduktraum

the linux binaries are there, though

macos is missing too

Kyle I S Harrington

@kephale

yeah i see the issue

i will try to resolve when i'm off babysitting duty

Ulrik Günther

@skalarproduktraum

supercool, thx

Kyle I S Harrington

@kephale

did someone disable travis for scenerygraphics?

@skalarproduktraum you did, this broke the update site uploader

can you please at least leave it on for https://github.com/scenerygraphics/sciview-update-sites

Where communities thrive

People

Repo info

Activity