Skip to content

Added RCS propellants to propellant library and made migration function#271

Open
mihirshevade wants to merge 1 commit into
reilleya:stagingfrom
mihirshevade:add-rcs-propellants
Open

Added RCS propellants to propellant library and made migration function#271
mihirshevade wants to merge 1 commit into
reilleya:stagingfrom
mihirshevade:add-rcs-propellants

Conversation

@mihirshevade

Copy link
Copy Markdown

Added propellant data for all AeroTech propellants from the RCS website and created a migration function to add the new propellants to users' libraries when upgrading from 0.6.1 to 0.6.2

@JackPoehlman

Copy link
Copy Markdown

@mihirshevade nice work!

@reilleya reilleya left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks great overall, thanks for doing this! Just need to deduplicate some of them and fix a few issues, then this will be good to go.

Comment thread uilib/defaults.py
}

DEFAULT_PROPELLANTS = [CL_PROPS, OW_PROPS, KNDX_PROPS, KNSB_PROPS, KNSU_PROPS, WL_PROPS, BT_PROPS]
CL_PROPS = {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks like this is a duplicate of the other CL definition above?

Comment thread uilib/defaults.py
]
}

WL_PROPS = {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Duplicated?

Comment thread uilib/defaults.py
]
}

BT_PROPS = {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Duplicated?

Comment thread uilib/defaults.py
'minPressure': 0,
'maxPressure': 10342500,
'a': 0.00002969528,
'n': 0.36600000,

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I think this is the N for Redline?

Comment thread uilib/defaults.py
'minPressure': 0,
'maxPressure': 10342500,
'a': 1.23560472e-03,
'n': 0.056,

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

...and this is the one for Black Jack?

Comment thread uilib/defaults.py
'minPressure': 0,
'maxPressure': 10342500,
'a': 1.23560472e-03,
'n': 0.056,

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This is the black jack one again, Black Max is actually 0.398 according to RCS

Comment thread uilib/defaults.py
'minPressure': 0,
'maxPressure': 10342500,
'a': 0.00001507051,
'n': 0.398,

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

W9 looks to be 0.298?

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.

3 participants