-
-
Notifications
You must be signed in to change notification settings - Fork 508
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
Docker fails to build #289
Comments
👋 Hello @aurelien-m, thank you for your interest in make-sense - free to use online tool for labelling photos! 🏷️ 🐞 Bug reportsIf 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 touchIf 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 |
Building works when updating the Dockerfile with I have also tried:
I have the following versions:
|
Hi, @aurelien-m 👋 It looks like our Dockerfile and instructions are heavily outdated. 😱 Let me quickly fix that. |
@aurelien-m sorry it took me so long. My changes are right now in the |
@SkalskiP Thanks! No worries, it was not urgent for me, it works perfectly now. I might have missed it but it doesn't look like Make Sense has an API or SDK of some sort? It would be nice to have one to be able to create projects and pre-annotate images in our custom pipelines. I also have a suggestion, it's a bit slow for our (quite strange) use case. We're labeling very tiny objects (~ 50 px in size) in very big images (5k images). So we need to be able to zoom in more than what's possible and zoom in where the cursor is pointing. The scrolling is a bit slow when I tried it, we can gain more time and have less headaches if it's smooth 😄 |
@aurelien-m I'm glad that the problem was solved relatively quickly!
You can use YOLOv5 models to annotate images in Make Sense. Would that be helpful?
We can increase max zoom if you want to.
That is weird. Could you share with me an image example so that I could test it out on my machine? |
I can't share any of our images, but this one is a pretty good example (6000 x 4000 pixels) of what we have: Here's a comparison between Make Sense and Label Studio (even though the best would be to try it out yourself): Screencast.from.10-27-2022.10.50.53.AM.webmScreencast.from.10-27-2022.10.49.32.AM.webmThe experience is much smoother in Label Studio, and the directed scrolling with the mouse is a really great feature! Please let me know if you have the same experience. Regarding YOLOv5, it could help us, but we would need to be able to use our own model trained on our data. We have data pipelines that process incoming data and automatically build annotation projects from them. Meaning that you can log into the app, and quickly begin annotating and verifying the automatic annotations. That's why I'm asking. |
This part of the editor was written quite a long time ago, and indeed it leaves much to be desired. For a long time I have been thinking of rewriting this functionality, but so far the community rarely reported problems, so I focused on other tasks. One thing in particular that annoys me is that the zoom does not take place in the direction of the cursor, but rather towards the center of the photo. Well, and of course, I can see the lag, which makes it difficult to work with the image at high zoom. 😔 Could you file a bug report on this? It would help me to keep track of this topic. Well, and hopefully I can find time to work on it.
You can upload your own pre train model. You just drag and drop model files into editor and use it to infer on images. Taka a look at this video for more details: 1-10-0-alpha.mov |
@SkalskiP Thanks for the video explanation and taking the time to reply! I'll open an issue today. I've also got a long 4-days week-end coming, I might take a look at the code 😄 |
@aurelien-m sure! I'm always excited when people try to use makesense.ai. :D I know it is not perfect, but I'm surely trying to make it as good as possible. |
Hi, I am trying to install make-sense locally however after running
I see that @aurelien-m had similar issues but it seems it shouldn't happen again. My npm is |
@domagojhack, is that happening in our docker container? |
no docker installation instructions are not working for me at all. I know this is a docker related issue but #289 (comment) suggested to install it using npm, and I am not keen about using docker on my setup. |
What's the setup? |
Freshlly installed Ubuntu 22.04 LTS. All requirements are already installed. |
Can I ask why? |
Describe the bug
The Docker image fails to build.
To Reproduce
docker build -t make_sense docker/
Expected behavior
It builds without any errors 😄
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: