-
Notifications
You must be signed in to change notification settings - Fork 84
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.
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
Installation instructions incomplete & example code misleading #38
Comments
Addendum: Regarding the code example: How would the procedure shown in this notebook look like in pyimagej? |
Shouldn't be. Not if you use conda. There are conda packages for both openjdk and maven.
I'm sorry, I don't understand what you mean by "expected" here. What doesn't happen, that you expect should happen?
I think that is too complex for a first code example. But I agree it would be a great code example, introduced relatively early. However, it takes work to figure out how to do those things. Do you have time to figure it out, and report any problems you find? I expect there will be several obstacles.
I agree that showing ROIs would be good. However, please keep in mind the target audience for this project: people who want to mix in ImageJ to PyData-based workflows. That means the people we're targeting already know NumPy, and the first thing they'll want to know is "how do I hand my NumPy image to ImageJ?" On the flip side: people who don't know PyData are unlikely to care so much about PyImageJ, since there are easier and more battle-tested ways of calling ImageJ via scripts than to use CPython via this bridge layer. |
Hi again, I just tried to install it on another Windows machine. The error message of
I tried to fix this by installing pyjnius (which worked as described on their website) but it doesn't help. The error message stayed the same. I'm now trying to install via linux subsystem.... |
Sorry. My fault. pyjnius had issues comiling. Maybe related to this issue I made it work by calling
Feel free to put that somewhere in a troubleshooting section ;-) |
One more thing: In the installation instructions it says:
But that file |
Ok, after some additional trouble, I made it run on windows using Anaconda. The full installation command list is:
Maybe that helps. Cheers, |
It's a YAML file: But those installation instructions are only for the pip-installable version of pyimagej. I strongly encourage you to use the conda version, and avoid pip installation of this package.
Why are you using
If that doesn't work on Windows, please file a new issue with details of what goes wrong. |
Hey @ctrueden , I just posted about what's going wrong on Windows above And the solution further down. Furthermore, I'm using pip, because that's what I found when googling for pyimagej. I landed on that website: Feel free to copy my solution to a trouble-shooting section of the documentation. I will create another issue about installation on Ubuntu. Thanks for your support! Cheers, |
@haesleinhuepf from my own experience with Python on Windows, I'd recommend to:
In my trials on Windows 10, the following worked (from Anaconda Prompt):
(or, in a similar way, without any
) Then the simple test script works via command-line python:
It still doesn't work from
I think all this is not yet well reflected in the README, and we should try to update it as soon as possible (and before demoing it at the NEUBIAS Training School...). |
As discussed in #45, the README is better now. As of scyjava 0.2.3, pyimagej should also now have fewer problems on Windows; see scijava/scyjava@35010bb. |
Clean up the build process
Dear all,
I was just struggling to get started with pyimagej on Windows. I'm documenting the issue here hoping that the documentation can be improved in order to lower the entry-level boundary. Details can be found here
The installation instructions are incomplete. Either it is necessary to create an environment variable JAVA_HOME or it is necessary to install maven before the first pyimagej program runs.
The very first example located here doesn't work as expected. I would recommend changing it something like:
I hope that helps!
Cheers,
Robert
The text was updated successfully, but these errors were encountered: