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

HALO #324

Merged
merged 18 commits into from
Jun 20, 2018
Merged

HALO #324

merged 18 commits into from
Jun 20, 2018

Conversation

Kexanone
Copy link
Member

@Kexanone Kexanone commented Jun 17, 2018

When merged this pull request will:

  • Fix backpack inventory duplicate in Achilles_fnc_chute.
  • Improve the timing and chute deploy height for Achilles_fnc_chute.
  • Improve the accuracy of paradrops based on fitted functions.
  • Set aircrafts to careless created by the reinforcement module (makes armed aircrafts mor reliable).
  • Add HALO option to the reinforcement module.
  • Add HALO option to the teleport player module (resolves Paradrop Module #90).

@Kexanone Kexanone added feature PR that adds a new feature to Achilles. priority/medium Issue or PR that impacts some portions of the user base and should be resolved with due time. labels Jun 17, 2018
@Kexanone Kexanone added this to the 1.0.3 milestone Jun 17, 2018
@Kexanone Kexanone self-assigned this Jun 17, 2018
@Kexanone Kexanone requested a review from CreepPork June 17, 2018 22:22
Copy link
Member

@CreepPork CreepPork left a comment

Choose a reason for hiding this comment

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

What about ACE reserve parachute?

@Kexanone Kexanone changed the title Halo HALO Jun 18, 2018
[_vehsGroup,_wp_pos,_radius] spawn
{
params ["_vehsGroup", "_wp_pos", "_radius"];
Copy link
Member

Choose a reason for hiding this comment

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

camelCase

_ctrl ctrlEnable false;
_ctrl ctrlCommit 0;
private _ctrl_wpType = _dialog displayCtrl _x;
_ctrl_wpType ctrlSetFade 0.8;
Copy link
Member

Choose a reason for hiding this comment

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

camelCase

private _crew = crew _firstVeh;
private _passengerCount = {(assignedVehicleRole _x) select 0 == "CARGO"} count _crew;
_radius = _radius + _passengerCount/2 * _speed/3.6;
// account for speed displacement
Copy link
Member

Choose a reason for hiding this comment

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

spacing

@@ -1,4 +1,4 @@
params[["_playersToTeleport", [objNull]], ["_teleportLocation", [0,0,0]], ["_showTeleportMessage", true], ["_includeVehicles", true]];
params[["_playersToTeleport", [objNull]], ["_teleportLocation", [0,0,0]], ["_showTeleportMessage", true], ["_additionalOption", 0]];
Copy link
Member

Choose a reason for hiding this comment

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

Make it into multiple lines so it can be read

if (_vehicle isKindOf "Air" and (_lzdz_type > 0)) then
{
_vehicleUnloadWp setWaypointType "SCRIPTED";
private _script = ["\achilles\functions_f_achilles\scripts\fn_wpParadrop.sqf", "\achilles\functions_f_achilles\scripts\fn_wpFastrope.sqf"] select (_lzdz_type isEqualTo 1);
Copy link
Member

Choose a reason for hiding this comment

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

Multiple lines

- Fix: Chute openning animation is smoother. #321
- Change: Paradroopers won't open their chute immediately at high altitude (>150 m) => HALO. #321
- Feature: AI accounts to a certain extend for paradrop displacment according to https://gyazo.com/32afcfefef24ba2cdc36eaa4c0467147 .
Copy link
Member

Choose a reason for hiding this comment

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

Why don't you make it as the markdown URL tags? Adding a dot at the end when you have a link is not good.

CreepPork
CreepPork previously approved these changes Jun 19, 2018
@Kexanone Kexanone merged commit 21f6577 into master Jun 20, 2018
@Kexanone Kexanone deleted the HALO branch June 20, 2018 00:10
Kexanone pushed a commit that referenced this pull request Jul 6, 2018
* Update readme paths

* update links in readme (part 2)

* Readme: Get rid of Heroku webhooks.

* Update README.md

* Small fixes (#293)

* Remove duplicate IDC

* Missing localization

* Remote control module still used vanilla function

* Rename stringtable key for delta distribution

* Hint module is now shown as Advanced Hint (#294)

* Hint module is now shown as Advanced Hint

* Swapped the category

* Add fr/de translations

* Reorganize and fix Adv Hint (#308)

* use ace_common_fnc_setName (#305)

* Fix for the Damage Building Module (#304)

* Use remoteExecCall instead of broadcasting the entire function

* Specify the target

* Fix undefined var error if ace is not installed. (#311)

* Added DeliciousJaffa (#317)

* Replace nonsensical default value (#320)

* Revised CAS modules (#319)

* Fix targeting and fire flares

* Address review comments + additional launch CM params

* Fix launch CM

* extraordinary important optimization

* Add change log for 1.0.3

* Small features (#321)

* Add artillery precision option

* Add artillery precision option (resolves #133)

* Revise paradrop: Drops below 150 => HALO (resolves #112)

* Add spaces for readability

* Correct change log (#322)

* Reorganize Tools (#323)

* move priv key folder to Achilles and ignore it

* Reorganize Tools

* New mintty-tmux session launcher

* Revised AddonBuilder batch script

* AchillesConnectToServer

* Update gitignore

* Rename Tools folder and add bat extension to AchillesAddonBuilder

* Fix travis tools path

* Reintroduce the faction filter (#326)

* Reintroduce the faction filter

* Update changlog.md

* Fix filter by uncommenting the required parts

* HALO (#324)

* Fix backpack inventory duplication

* Improve paradrop accuracy

* Add HALO option for reinforcements

* Improved the fitted function for HALO

* Fix error in function for HALO parameter

* More frequent checking time for height, since 1 sec can make a lot of difference in height

* Achilles_fnc_chute: Add default value for delay and reset chute deploy height to 100m

* Reinforcement: Set aircraft to carless, restrict HALO for planes only

* Add HALO option to teleport module

* Address point in review

* Address points in review (part 2)

* update change log

* Reinforcment: Fix CUP planes

* Achilles_fnc_chute: change chute deployment height to 120m

* Update changelog.md

* Use proper syntax for link in changlog

* Remove the extra space in the changelog

* Change 1.0.3 to 1.1.0 (#327)

* Fix locality issue for HALO option in the teleport module (#328)

* Broadcast chute function and exec on local machine

* Code optimization
Kexanone pushed a commit that referenced this pull request Aug 3, 2018
* Update readme paths

* update links in readme (part 2)

* Readme: Get rid of Heroku webhooks.

* Update README.md

* Small fixes (#293)

* Remove duplicate IDC

* Missing localization

* Remote control module still used vanilla function

* Rename stringtable key for delta distribution

* Hint module is now shown as Advanced Hint (#294)

* Hint module is now shown as Advanced Hint

* Swapped the category

* Add fr/de translations

* Reorganize and fix Adv Hint (#308)

* use ace_common_fnc_setName (#305)

* Fix for the Damage Building Module (#304)

* Use remoteExecCall instead of broadcasting the entire function

* Specify the target

* Fix undefined var error if ace is not installed. (#311)

* Added DeliciousJaffa (#317)

* Replace nonsensical default value (#320)

* Revised CAS modules (#319)

* Fix targeting and fire flares

* Address review comments + additional launch CM params

* Fix launch CM

* extraordinary important optimization

* Add change log for 1.0.3

* Small features (#321)

* Add artillery precision option

* Add artillery precision option (resolves #133)

* Revise paradrop: Drops below 150 => HALO (resolves #112)

* Add spaces for readability

* Correct change log (#322)

* Reorganize Tools (#323)

* move priv key folder to Achilles and ignore it

* Reorganize Tools

* New mintty-tmux session launcher

* Revised AddonBuilder batch script

* AchillesConnectToServer

* Update gitignore

* Rename Tools folder and add bat extension to AchillesAddonBuilder

* Fix travis tools path

* Reintroduce the faction filter (#326)

* Reintroduce the faction filter

* Update changlog.md

* Fix filter by uncommenting the required parts

* HALO (#324)

* Fix backpack inventory duplication

* Improve paradrop accuracy

* Add HALO option for reinforcements

* Improved the fitted function for HALO

* Fix error in function for HALO parameter

* More frequent checking time for height, since 1 sec can make a lot of difference in height

* Achilles_fnc_chute: Add default value for delay and reset chute deploy height to 100m

* Reinforcement: Set aircraft to carless, restrict HALO for planes only

* Add HALO option to teleport module

* Address point in review

* Address points in review (part 2)

* update change log

* Reinforcment: Fix CUP planes

* Achilles_fnc_chute: change chute deployment height to 120m

* Update changelog.md

* Use proper syntax for link in changlog

* Remove the extra space in the changelog

* Change 1.0.3 to 1.1.0 (#327)

* Fix locality issue for HALO option in the teleport module (#328)

* Broadcast chute function and exec on local machine

* Code optimization
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature PR that adds a new feature to Achilles. priority/medium Issue or PR that impacts some portions of the user base and should be resolved with due time.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Paradrop Module
2 participants