Skip to content

Comments

Allow 3d Model overrides#1440

Merged
patrickelectric merged 4 commits intobluerobotics:masterfrom
Williangalvani:model_overrides
Feb 13, 2023
Merged

Allow 3d Model overrides#1440
patrickelectric merged 4 commits intobluerobotics:masterfrom
Williangalvani:model_overrides

Conversation

@Williangalvani
Copy link
Member

@Williangalvani Williangalvani commented Feb 4, 2023

Screenshot 2023-02-04 at 02 49 08

as a bonus commit, this fixed #1455

@Williangalvani Williangalvani changed the title Model overrides Allow 3d Model overrides Feb 4, 2023
@Williangalvani Williangalvani force-pushed the model_overrides branch 2 times, most recently from 1753e45 to 1c9a057 Compare February 4, 2023 05:50
@Williangalvani
Copy link
Member Author

onte interesting issue here. we cant make 404 fail quietly. so even if it works as intended, there's a bunch of red lines in the terminal. I tries using the HEAD request to check if things exist, but it also 404s. so maybe we could add something in nginx to make it fail in a nicer way.

@Williangalvani Williangalvani force-pushed the model_overrides branch 5 times, most recently from 5bb214d to 5853437 Compare February 9, 2023 15:57
@Williangalvani
Copy link
Member Author

for testing a custom model, set frame to VECTORED_6DOF and unzip this package at /usr/blueos/userdata:
userdata.zip

@Williangalvani
Copy link
Member Author

you should see something like this
Screenshot 2023-02-09 at 13 11 52

@Williangalvani Williangalvani marked this pull request as ready for review February 9, 2023 16:46
@Williangalvani Williangalvani requested review from ES-Alexander and patrickelectric and removed request for patrickelectric February 9, 2023 16:46
Copy link
Collaborator

@ES-Alexander ES-Alexander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok to me, but I haven't extensively reviewed the code - I just want the feature :-)

@Williangalvani Williangalvani deleted the model_overrides branch March 28, 2023 01:09
@ES-Alexander ES-Alexander added the docs-needed Change needs to be documented label Dec 4, 2023
@ES-Alexander ES-Alexander added docs-minimal Documentation exists but should be improved and removed docs-needed Change needs to be documented labels Oct 6, 2024
@ES-Alexander ES-Alexander added docs-in-progress Included in an open docs PR docs-complete Change documentation has been completed and removed docs-minimal Documentation exists but should be improved docs-in-progress Included in an open docs PR labels Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-complete Change documentation has been completed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vehicle setup: Enclosure is opaque when first loaded [chrome only]

3 participants