Conversation
There was a problem hiding this comment.
Great, @elevans, I'm looking so forward to a JPype-based scyjava/pyimagej!
The Travis build is currently failing, I think what's missing is jpype (or jpype1?) instead of pyjnius here:
Line 23 in c593029
and here:
Line 20 in c593029
| usage: conda-env [-h] {create,export,list,remove,update,config} ... | ||
|
|
||
| positional arguments: | ||
| {create,export,list,remove,update,config} | ||
| create Create an environment based on an environment file | ||
| export Export a given environment | ||
| list List the Conda environments | ||
| remove Remove an environment | ||
| update Update the current environment based on environment | ||
| file | ||
| config Configure a conda environment | ||
|
|
||
| optional arguments: | ||
| -h, --help Show this help message and exit. | ||
|
|
||
| conda commands available from other packages: | ||
| env |
There was a problem hiding this comment.
This file doesn't look like a valid environment.yml, but rather like the accidental output of conda env > environment.yml (instead of maybe conda list > environment.yml).
|
Anything in particular on the Longs that presents an issue? |
| >>> LongArray = JArray(JLong) | ||
| >>> dims = LongArray([64, 16]) | ||
| >>> blank = ij.op().getClass().getMethod('create').invoke(ij.op()).img(dims) |
There was a problem hiding this comment.
One option here is to use the array notation.
dims = JLong[64, 16] # equal to "dims = new long[64][16]"
As discussed at #20 (comment)
|
The Travis failure is a failure of travis-ci.org, whereas travis-ci.com is passing. It was a mistake for both Travis platforms to be attached to this project at the same time. So there should be no issue with merging this. |
|
Thanks so much @elevans for your hard work on this!! 🍻 🏆 |
4 participants
Here's the
scyjavalayer running onJPype. Some key differences between the olderscyjavathat ran onpyjniusare:scyjava.jvmmodule.scyjava.convert.Longarrays can be a little goofy. Use Jpype'sJArrayandJLong.