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

Added HQ world #10

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Added HQ world #10

wants to merge 14 commits into from

Conversation

IDavGal
Copy link
Collaborator

@IDavGal IDavGal commented Dec 5, 2023

Adds the Ekumen HQ map, created during the Ekuthon.

@IDavGal IDavGal requested a review from JCarosella December 5, 2023 16:17
@IDavGal IDavGal self-assigned this Dec 5, 2023
@JCarosella
Copy link
Collaborator

Tested this, loaded correctly (with some missing links to assets, probably dataset related). Good enough for me, approving

image

@francocipollone
Copy link
Collaborator

Tested this, loaded correctly (with some missing links to assets, probably dataset related). Good enough for me, approving

image

What about that? How can we bring those missing assets?

@JCarosella
Copy link
Collaborator

Tested this, loaded correctly (with some missing links to assets, probably dataset related). Good enough for me, approving
image

What about that? How can we bring those missing assets?

I think it has to do with nucleus. I can give it a shot later

@BarceloChristian
Copy link
Collaborator

Tested this, loaded correctly (with some missing links to assets, probably dataset related). Good enough for me, approving
image

What about that? How can we bring those missing assets?

I think it has to do with nucleus. I can give it a shot later

Restart nucleus, collect the USD, open it, save it flattened. That should do the trick

@JCarosella
Copy link
Collaborator

Tested this, loaded correctly (with some missing links to assets, probably dataset related). Good enough for me, approving
image

What about that? How can we bring those missing assets?

I think it has to do with nucleus. I can give it a shot later

Restart nucleus, collect the USD, open it, save it flattened. That should do the trick

Done and pushed. Is that "saved as flattened" a new feature? I remember not having that option

image

@BarceloChristian
Copy link
Collaborator

BarceloChristian commented Dec 6, 2023

Restart nucleus, collect the USD, open it, save it flattened. That should do the trick

It's always been there, not always works as expected. Have you tried turning Nucleus off and testing the USD again to check it has no other reference?

@JCarosella
Copy link
Collaborator

Restart nucleus, collect the USD, open it, save it flattened. That should do the trick

It's always been there, not always works as expected. Have you tried turning Nucleus off and testing the USD again to check it has no other reference?

Turning off nucleus brings problems with the textures, everything's red now. The textures are still pointing to the nucleus path, so that's the problem. I'll see what I can do

@JCarosella
Copy link
Collaborator

Restart nucleus, collect the USD, open it, save it flattened. That should do the trick

It's always been there, not always works as expected. Have you tried turning Nucleus off and testing the USD again to check it has no other reference?

Turning off nucleus brings problems with the textures, everything's red now. The textures are still pointing to the nucleus path, so that's the problem. I'll see what I can do

I was able to download the materials and not depend on nucleus. I'm not convinced though on this approach, assets and materials take up a lot of space

@francocipollone
Copy link
Collaborator

Mjm, indeed it adds a lot of MBs to the repository. It might be ok though to have this this way for educational purposes: You directly have access to materials, etc. Besides you don't have to deal with nucleus service working correctly.

I will refer the decision to @BarceloChristian

Base automatically changed from launch_script to main December 6, 2023 14:58
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

Successfully merging this pull request may close these issues.

5 participants