Correctly name copied Position in Plate.from_positions

[ieivanov]

Plate.from_positions takes

positions : dict[str, Position]
    Dictionary where keys are destination path names ('row/column/fov')
    and values are :py:class:`iohub.ngff.Position` objects.

In this PR, I'm updating the name of the copied FOV in the omero metadata to its new name: fov

[ziw-liu]

Looks good! Can you add a regression test for this fix?

[ieivanov]

I think it will be best to add this in the test_combine_fovs_to_hcs test, which we currently skip - see #255. I think it has something to do with the external ome_zarr Reader but I couldn't quite figure it out. Would you be able to fix #255 or would you suggest I add the tests elsewhere?

[ziw-liu]

I think it will be best to add this in the test_combine_fovs_to_hcs test, which we currently skip - see #255. I think it has something to do with the external ome_zarr Reader but I couldn't quite figure it out. Would you be able to fix #255 or would you suggest I add the tests elsewhere?

That test needs to be fixed (actually should be fixed before merging the original PR without skipping, not introducing a regression blocking other PRs like this). I'll see if I can do it.

[ieivanov]

@ziw-liu have you had a chance to check why the ome_zarr Reader fails here?