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

ImportError: cannot import name 'line_optim' from 'line_refinement' (unknown location) #28

Open
2jjjd opened this issue Oct 20, 2023 · 9 comments

Comments

@2jjjd
Copy link

2jjjd commented Oct 20, 2023

there is no such a file named 'line_optim' in the file'line_refinement',
how to import line_optim from line_refinement in the file deeplsd/models/deeplsd.py?

@rpautrat
Copy link
Member

Hi, did you correctly installed deeplsd with the full install, as described in the ReadMe?

@ogencoglu
Copy link

Hi-jacking this. Does line refinement mean e.g. double lines will be merged into one with the 'optimize': True config or something else?

@rpautrat
Copy link
Member

No, the line refinement is only meant to slightly move a bit the lines to make them perfectly fit the predicted distance and angle fields. So you would keep the same number of lines, including double lines.

@houyiliang
Copy link

Hello, it also appears after I complete the configuration according to full install.
ImportError: cannot import name 'line_optim' from 'line_refinement' (unknown location)
how did you solve it? thank you.

@rpautrat
Copy link
Member

Hi, can you send here the full output that you get after running the full install command?

@reai000
Copy link

reai000 commented Dec 30, 2023

Hi, can you send here the full output that you get after running the full install command?

0

Hi, I also encountered the same problem. Where is the definition of line_optim? Thank you

@rpautrat
Copy link
Member

rpautrat commented Jan 2, 2024

line_optim is defined in the line_refinement folder, with Python bindings. It is installed during the full install described in the Readme.

Note that you only need it to refine lines, but if you just want to use the base version of DeepLSD for inference, you don't need it. The Readme explains how to do this quickstart install.

@reai000
Copy link

reai000 commented Jan 3, 2024

I solved it, thank you very much。

@whw9483
Copy link

whw9483 commented Jan 26, 2024

I solved it, thank you very much。

How was it resolved?

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

6 participants