Skip to content
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

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

Open
heyodai opened this issue Jan 28, 2024 · 2 comments

Comments

@heyodai
Copy link

heyodai commented Jan 28, 2024

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:

  • ./dataset/class-1/image1.jpg
  • ./dataset/class-2/image1.jpg

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):

  • OS: macOS 14.2.1
  • Browser: Chrome 121.0.6167.85
Copy link

👋 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
Copy link
Author

heyodai commented Jan 28, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant