Unexpected behaviors in `omerotools`: `parse_url` and `upload_image_to_omero`

[loicsauteur]

Hi all

while testing your OMERO_batch_projection.py script I noticed:

parse_url:

1. Lines 113-117 should be removed (they do not do anything)

python-imcflibs/src/imcflibs/imagej/omerotools.py

Lines 113 to 117 in 1f6b508

	image_ids = (
	[s.split("%")[0].replace("|", "") for s in omero_str.split("image-")[1:]]
	if "image-" in omero_str
	else omero_str.split(",")
	)

2. Does not work with plain dataset ID. Running on dataset IDs results in an error.This would require checking if the supplied ID (e.g. 12345) is a Dataset, Project, or Image.

upload_image_to_omero:

It would be nice if it would check if the path exists and if not: raise a FileNotFoundError.

I can do a PR if you like, let me know!

[ehrenfeu]

Hi Loïc,

thanks for the report. I have edited your comment above to link to the respective functions and code blocks. Could you double-check this is correct, i.e. is what you meant please?

🤗

Thanks,
Niko