search

SkalskiP / make-sense

Free to use online tool for labelling photos. https://makesense.ai
GNU General Public License v3.0
3.17k stars 525 forks source link

Uploading multiple files with identical filenames breaks some export options. #351

Open heyodai opened 9 months ago

heyodai commented 9 months ago

Describe the bug Uploading multiple files with identical filenames breaks some export options.

I uploaded a dataset where some files had the same filename. For example:

For exporting rectangle annotations, this breaks the YOLO and VOC XML options. They will only export once per filename (i.e. in the above example, only the first image1.jpg will export). CSV still exports all files as expected.

Here are some example exports to reference:

I believe issue #330 is describing the same problem.

To Reproduce Steps to reproduce the behavior:

  1. Upload a few files with identical filenames
  2. Export as YOLO or VOC XML
  3. Note the incorrect number of annotations in the export

Expected behavior I would expect the YOLO and VOC XML exports to export every file that I uploaded.

Desktop (please complete the following information):

github-actions[bot] commented 9 months ago

👋 Hello @heyodai, thank you for your interest in make-sense - free to use online tool for labelling photos! 🏷️

🐞 Bug reports

If you noticed that make-sense is not working properly, please provide us with as much information as possible. To make your life easier, we have prepared a bug report template containing all the relevant details. We know, we ask for a lot... However, please believe that knowing all that extra information - like the type of browser you use or the version of node you have installed - really helps us to solve your problems faster and more efficiently. 😉

💬 Get in touch

If you've been trying to contact us but for some reason we haven't responded to your issue yet, don't hesitate to get back to us on Gitter or Twitter.

💻 Local setup

# clone repository
git clone https://github.com/SkalskiP/make-sense.git

# navigate to main dir
cd make-sense

# install dependencies
npm install

# serve with hot reload at localhost:3000
npm start

To ensure proper functionality of the application locally, an npm 8.x.x and node.js v16.x.x versions are required. More information about this problem is available in the #16 issue.

heyodai commented 9 months ago

For anyone else coming across this issue, I made a Jupyter notebook to convert the CSV export file to YOLO format. This should work for your own dataset with a few tweaks: https://gist.github.com/heyodai/43548d252b6ad03f54e400dbdfc93778