Lydorn / mapalignment

Aligning and Updating Cadaster Maps with Remote Sensing Images
Other
73 stars 13 forks source link

AttributeError: module 'math_utils' has no attribute 'to_homogeneous' #6

Closed MathisMohand closed 5 years ago

MathisMohand commented 5 years ago

Hi, I've been trying to try and use your code to infer some displacements. However, I've been confronted to this issue (title) and seeing the code in detail, the geo_utils.py calls this function but it doesn't exists.

Is it because last time the geo_utils.py file was modified was (to this date) 19 hours ago while the other files like math_utils were modified 9 months ago ?

Lydorn commented 5 years ago

Hi, Thank you for letting me know, yes indeed I mistakenly did not update the other utils. I pushed a commit to update them just now. Does it work for you now?

MathisMohand commented 5 years ago

It is working for math_utils (it has been updated). However I now have a new error : root@c997235f530f /w/p/m/mapalign_multires# python main.py Traceback (most recent call last): File "main.py", line 14, in <module> import test File "/workspace/projects/mapalign/mapalign_multires/test.py", line 7, in <module> import visualization File "../utils/visualization.py", line 9, in <module> import polygon_utils File "/workspace/projects/mapalign/utils/../../utils/polygon_utils.py", line 15, in <module> import smallestenclosingcircle ModuleNotFoundError: No module named 'smallestenclosingcircle' I don't really know where it comes from, I haven't looked all new files yet.

Lydorn commented 5 years ago

Actually smallestenclosingcircle is not used in the code so I removed from polygon_utils.py. The new commit should remove that error.

MathisMohand commented 5 years ago

It is working now thanks. However I had to modify few lines to make it work :

Lydorn commented 5 years ago

Good you managed to make it work! And thank you for telling me the changes you had to made, I just committed them. I also couldn't make the download script work just now... don't know what's wring with it!

Best,

Nicolas

On Wed, Oct 23, 2019 at 4:58 AM Mathis MOHAND notifications@github.com wrote:

It is working now thanks. However I had to modify few lines to make it work :

-

In model.py there is import polygonization_utils which has to be changed to polygon_utils

On the quickstart I had to download with the zipfile to get the pretrained models as the download_pretrained.py script just created an empty folder ds_fac8... in runs.igarss2019 folder

Had to add allow_pickle=True parameter in the np.load function call in file main.py line 111.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Lydorn/mapalignment/issues/6?email_source=notifications&email_token=AFCGEVGURNHVNRC5TFIMP33QQAGZPA5CNFSM4JDQ5GSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECAUL6Y#issuecomment-545342971, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFCGEVHNFAIJGEVHHW2X4MTQQAGZPANCNFSM4JDQ5GSA .