Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fixed 403 #575

Merged
merged 13 commits into from
Jan 27, 2023
Merged

fixed 403 #575

merged 13 commits into from
Jan 27, 2023

Conversation

Birkenpapier
Copy link
Contributor

@Birkenpapier Birkenpapier commented Dec 15, 2022

there are old for the pretrained weights which are leading to an 403 error


This change is Reviewable

ssheorey and others added 13 commits February 15, 2022 08:48
Update weights link to address this issue - isl-org#458.
[matterport_objects.py](https://github.com/isl-org/Open3D-ML/blob/13c847b50f4766cd73f942bd2c12bb86c202c088/ml3d/datasets/matterport_objects.py) calls `logging.basicConfig()`. `basicConfig` is intended for quickly setting up logging in scripts, not for library code. `matterport_objects` is imported by `import open3d` and sets the root log level and format which interferes with logging in the importing code. `matterport_objects` only logs once on initialization and doesn't seem like it should be opinionated about whether that log shows or how it's formatted.

As an example, Bottle's `run(..., quiet=True)` assumes that the root logger's log level stays at the default `WARNING` and `import open3d` sets it to `INFO` via `matterport_objects` which adds a lot of log noise.
* Get rid of torch._six

* more refactoring

* Fix styling
Python to 3.10 to match main repo CI.
Tensorflow, PyTorch, PyTest
Fix wheel, setuptools, pip versions.
Skip OpenVino in CI since it conflicts with TF 2.8.2
@ssheorey ssheorey self-requested a review January 21, 2023 00:18
@ssheorey ssheorey added this to the v0.17 milestone Jan 21, 2023
@ssheorey ssheorey merged commit f7c4302 into isl-org:dev Jan 27, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants