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

Add support for circles: click & drag to form a circle with a given radius #386

Open
jidkano opened this issue Dec 21, 2024 · 3 comments
Open

Comments

@jidkano
Copy link

jidkano commented Dec 21, 2024

Is your feature request related to a problem? Please describe.
No 'problem' just an additional feature

Describe the solution you'd like
Click and drag to draw a circle with a given radius

Describe alternatives you've considered
Sticking with points and encoding the radius into the label -> nightmare

Additional context
Just an additional type 'circle' besides rect point line & polygon
I can implement if assurance to get deployed live in short delay after PR is merged

Copy link

👋 Hello @jidkano, 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.

@jidkano
Copy link
Author

jidkano commented Dec 21, 2024

Would implement but I can see lots of unmerged PRs so not sure if project is still maintained

@SkalskiP
Copy link
Owner

SkalskiP commented Jan 6, 2025

Hi @jidkano 👋🏻 thanks a lot for your interest in make-sense. To be honest, due to time constraints, make-sense has not been actively developed for over a year. How experienced are you with JS? If you were able to work on this feature independently without a lot of guidance from me, maybe we could add support for circles.

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

2 participants