Skip to content

Av 1076 fix monte python#41

Open
parthshrotri wants to merge 3 commits intomainfrom
AV-1076-Fix-monte-python
Open

Av 1076 fix monte python#41
parthshrotri wants to merge 3 commits intomainfrom
AV-1076-Fix-monte-python

Conversation

@parthshrotri
Copy link
Member

Restructured code into Configurations, Simulation core, executables, and output.
Cleaned up executing files.
Pysim can now be called by monte-python for Monte Carlo analysis
Monte-python now outputs to csv files

@EvanY14 EvanY14 requested review from EvanY14 and rbhog November 3, 2023 00:34
Copy link
Member Author

@parthshrotri parthshrotri left a comment

Choose a reason for hiding this comment

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

Monte Carlo plots are still not showing data from multiple runs and 3sigma, DO NOT MERGE UNTIL FIXED

Copy link
Member

@rbhog rbhog left a comment

Choose a reason for hiding this comment

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

mainly code styling changes, will do another pass for functionality

enable_direction_variance=atmo_args.get("enable_direction_variance", False),
enable_magnitude_variance=atmo_args.get("enable_magnitude_variance", False),
nominal_wind_magnitude=atmo_args.get("nominal_wind_magnitude", 0.0),
nominal_wind_direction=atmo_args.get("nominal_wind_direction", np.array([0, 0, 0])))
Copy link
Member

Choose a reason for hiding this comment

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

probably better for the sake of standardization and legibility to have each of these fields explicitly defined in the dictionary before being passed in


# Indices of each state variable in the output array
# kf and rkf values are arrays containing position, velocity and acceleration in each axis
indices = {"time":0,
Copy link
Member

Choose a reason for hiding this comment

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

better as an enum, see enum package

Copy link
Member

Choose a reason for hiding this comment

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

also looking back, probably better to just have a global dataframe or smth to hold sim data because then we can access fields by property name and it's alr sorted by timestamps

"kalman_raccel_z":60}

default_output_folder = 'Output'
def rmse(predictions, targets):
Copy link
Member

Choose a reason for hiding this comment

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

type hints needed

"launch_angle_mean": .00,
"launch_angle_stddev": 0.00}

monte(dt, kf_dt, samples, output_folder, target_size, **sim_params) No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

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

nit: newlines at end of files

"launch_angle_mean": .00,
"launch_angle_stddev": 0.00}

monte(dt, kf_dt, samples, output_folder, target_size, **sim_params) No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

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

idk if out of scope but idt this code is thread safe so if we wanna ramp up number of runs in parallel we need to do that

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.

2 participants