rebane2001 / matterport-dl

A downloader for matterport virtual tours
The Unlicense
320 stars 79 forks source link

NEW FILES #73

Closed sbirkmann closed 2 months ago

sbirkmann commented 2 years ago

JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 207.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 260.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 300.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 309.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 385.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 393.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 521.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 564.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 58.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 633.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 769.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 794.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 934.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 976.js, will download for you though:) JS FILE EXTRACTED BUT not known, please file a github issue and tell us to add: 995.js, will download for you though:)

gene1wood commented 2 years ago

Also

330.js
39.js
399.js
438.js
62.js
76.js
926.js
933.js
redstreet commented 1 year ago

+1 to the above. They need to be added to this list.

EDIT: this is inadequate, see below.

sbirkmann commented 1 year ago

There are additional changes necessary as some functions in this file will load Real matterport urls instead of local ones, which results in an 401 unauthorised error and the model cannot be loaded when it's served locally

Red S @.***> schrieb am Mo., 10. Okt. 2022, 22:50:

+1 to the above. They need to added to this list https://github.com/mu-ramadan/matterport-dl/blob/21c51e03e9e14eecd0953be5c544f3426335b754/matterport-dl.py#L159 .

— Reply to this email directly, view it on GitHub https://github.com/rebane2001/matterport-dl/issues/73#issuecomment-1273807254, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABCFQ534F4N6RV2XGOKZH33WCR6PRANCNFSM6AAAAAAQXXIBUE . You are receiving this because you authored the thread.Message ID: @.***>

redstreet commented 1 year ago

Ah, good to know. Edited my comment.

sbirkmann commented 1 year ago

Do you have a fix for this?

mu-ramadan commented 1 year ago

@sbirkmann #66

mitchcapper commented 2 months ago

This should be fixed now, please test again on master and let us know if not