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

Proof of Concept Rhs scud missile #3178

Draft
wants to merge 5 commits into
base: unstable
Choose a base branch
from

Conversation

ante185
Copy link

@ante185 ante185 commented Mar 19, 2024

What type of PR is this.

  1. Bug
  2. Change
  3. Enhancement

What have you changed and why?

Information:
Adds compatibility for the 9M79 Scarab/Tochka
It is added to the regular artillery function just as proof that it'll work
It'll be more suitable to move into a different/new support class.

Please specify which Issue this PR Resolves.

closes #XXXX

Please verify the following and ensure all checks are completed.

  1. Have you loaded the mission in LAN host?
  2. Have you loaded the mission on a dedicated server?

Is further testing or are further changes required?

  1. No
  2. Yes (Please provide further detail below.)

How can the changes be tested?

Steps:
Set invader template to RHS SCUD TEST.
Move your player character somewhere at least 1 100m but within at least 40 000m from the invader's airbase.
Enter either Zeus or Spectator mode.
Run the following command and use Zeus or Spectator mode to observe the 9M79 in action

player allowDamage false;
["ARTILLERY", Invaders, "defence", 200, vehicle player, getPosATL player, 1, 1] spawn A3A_fnc_createSupport;

Observe the consequences.


Notes:

@ante185 ante185 added Change A change to existing functionality Don't merge This needs to be touched before being merged Proof of concept RFC Request for comment. Hopefully A Dev member will answer. labels Mar 19, 2024
@targetingsnake
Copy link

With the RHS Scud there is the issue that you maybe could adjust the pylons with an ammo-truck, ace loaded and logistics settings for ammo selected where you load not the full vehicle but only an amount of the ammo.

There might be the solution to make an own config entry for the vehicle with overwritten ammo data but i can't tell how that can be done

@ante185
Copy link
Author

ante185 commented Mar 28, 2024

With the RHS Scud there is the issue that you maybe could adjust the pylons with an ammo-truck, ace loaded and logistics settings for ammo selected where you load not the full vehicle but only an amount of the ammo.

There might be the solution to make an own config entry for the vehicle with overwritten ammo data but i can't tell how that can be done

With ACE Logistics Rearm settings set to "Entire Magazine" I am unable to select a different 'magazine'/missile for the vehicle.

@ante185 ante185 requested a review from jaj22 March 30, 2024 19:51
@Bob-Murphy Bob-Murphy added this to the Not organized yet milestone Apr 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Change A change to existing functionality Don't merge This needs to be touched before being merged Proof of concept RFC Request for comment. Hopefully A Dev member will answer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants