Skip to content

40 beginning pipeline simulation#45

Draft
DanielRhee wants to merge 2 commits intomainfrom
40-beginning-pipeline-simulation
Draft

40 beginning pipeline simulation#45
DanielRhee wants to merge 2 commits intomainfrom
40-beginning-pipeline-simulation

Conversation

@DanielRhee
Copy link
Collaborator

this should be going to dev but it's broken and i'll fix it later idk

sample sim using random data... draft 1
3-D and Top Down cone representation; Sim-Data v.s Time Graphs
@DanielRhee DanielRhee linked an issue Feb 18, 2026 that may be closed by this pull request
Copy link
Collaborator Author

@DanielRhee DanielRhee left a comment

Choose a reason for hiding this comment

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

i'm unsure why you aren't using the sims-data codebase for the simulations

import numpy as np
import matplotlib.pyplot as plt

# Veichel state form state.py
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fyi there's a large update coming to the format of this @goob10000 knows more

Copy link
Member

Choose a reason for hiding this comment

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

It is already on main if y'all wanna use it

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

oh yeah definitely move to it then.

[18, 10]
])

# finding velcoity from heading
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

why aren't you using the code in the sims for this?



## loading the json file
with open("/Users/atvyas/Desktop/coding/FS/sim_int/cones.json", "r") as f:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

use a relative path that works on anyone's computer

Comment on lines +3 to +4
from state_test import (
position, speed, heading,
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

very confused why you're importing this instead of an actual vehicle state

acceleration, throttle, brakes, lastCurrent
)

# sim time set up
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

these shouldn't be necessary if you use the actual sim

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

file shouldn't be necessary, use the Sims-Data repo

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@vyasatharv07 are the units in meters? what are the units here

Copy link
Member

Choose a reason for hiding this comment

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

You can make it a json5 file and use that library to add comments to the json. I did the same for the main sim so the params could have units

@DanielRhee DanielRhee requested a review from goob10000 February 18, 2026 18:08
@@ -0,0 +1,56 @@
import numpy as np
import matplotlib.pyplot as plt
from state_test import (
Copy link
Member

Choose a reason for hiding this comment

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

Similar to Daniel's comment, if you use Sims-Data you should be able to import * from ParamLoader and that will have all the parameters and the references to build an array and schema for your state array

Copy link
Member

Choose a reason for hiding this comment

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

We can chat more about how it works if you want

Choose a reason for hiding this comment

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

ok thanks

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.

Beginning Pipeline Simulation

3 participants