Skip to content
This repository was archived by the owner on Jan 1, 2026. It is now read-only.

[Do not merge] Use gsplat > 1.0#24

Open
una-dinosauria wants to merge 2 commits intomainfrom
julieta/gsplat1.0
Open

[Do not merge] Use gsplat > 1.0#24
una-dinosauria wants to merge 2 commits intomainfrom
julieta/gsplat1.0

Conversation

@una-dinosauria
Copy link
Copy Markdown
Contributor

@una-dinosauria una-dinosauria commented Dec 2, 2024

Using migration guide from https://docs.gsplat.studio/main/migration/migration_legacy.html

DO NOT MERGE: this works for inference but I haven't checked that training still works.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Dec 2, 2024
@una-dinosauria
Copy link
Copy Markdown
Contributor Author

Heads up, training doesn't seem to converge with this upgrade. I'll be running everything with the old env in the meantime. This is probably related to the way SH are handled: https://docs.gsplat.studio/main/migration/migration_legacy.html#color-as-spherical-harmonics

Not sure when I'll have time to look at this more closely.

@una-dinosauria una-dinosauria changed the title Use gsplat > 1.0 [Do not merge] Use gsplat > 1.0 Dec 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants