Conversation
Tutur baba coco patch 1
|
Hello @bgruening A colleague is experiencing the same problem in the tools-iuc (galaxy.org) repository, but it doesn't happen every time. |
I think I am also facing similar issues in my PR: #1817 Locally all tool tests and linting pass. |
We had some CI cache issues, let me try to re-trigger manually |
|
So normally, we can merge. |
|
The failing test looks relevant: https://github.com/bgruening/galaxytools/actions/runs/23297276348?pr=1816 |
Oh yes I don't see sorry |
| <param name="path_mode" value="anylabeling"/> | ||
| </conditional> | ||
| <output_collection name="outputs" type="list"> | ||
| <element name="5827603936_3f1d5d715c_z" file="5827603936_3f1d5d715c_z.json" compare="contains"/> |
There was a problem hiding this comment.
I changed compare = diff to contains so the test can pass. I’m not sure if this is a good idea. What do you think about it?
The error may be because GitHub adds a space. When I try to remove this space, I don’t see it in the file, but I see it in the HTML.
--- local_file
+++ history_data
@@ -3589,4 +3589,4 @@
"imageHeight": 640,
"imageWidth": 426,
"text": ""
-}
+}
Actually, I’m not sure if the problem is really a space. When I download the HTML, I see one, but maybe it is just because of the format.
There was a problem hiding this comment.
its probably not the space, but a line break.
You can generate this file with planemo test --biocontainers --update_test_data
This tool converts COCO annotation files (JSON) into LabelMe JSON files, one per image.
The SAM3 export creates results in COCO format. However, if you want to check or fix the annotations by hand using AnyLabeling, you cannot do this because AnyLabeling needs the LabelMe JSON format
Notes
imagePathfield is written in the output, to match different folder structures (simple, AnyLabeling default, or custom path).