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

Fix negative bypass being ignored #6992

Merged
merged 2 commits into from
Dec 11, 2023

Conversation

mortentc
Copy link
Contributor

@mortentc mortentc commented Dec 11, 2023

Fixes #6990.

Description of the problem being solved:

Negative bypass was rounded up to 0, ignoring some instances of "not bypass"

Steps taken to verify a working solution:

  • Allocating the es mastery and achieving 70% chaos bypass

Link to a build that showcases this PR:

eNrlHNly4zbyefQVKFVly1PxwUOXvXJSsiUfiQ9Fsj2ZvKRgEpKYAQkNCNqjpPLv2wBIiZJFmpSUfdhNqjwS0XejG90N2u0fv_kUvRAeeiw4rZqHRhWRwGGuF4xPq48PFwet6o8_VNp9LCb3o7PIo3Llh8qHtvqMKHkhFPCsKhKYj4l4SkjZvwOpKQ7EhLDgFv_B-CVzT6t3LCBV9IwD1xPJN4fiMLzDPjmtDh1AriIcOiRwzxfPY8AJ5tgRhN9Itp1IsFvmwqrgEaz62AuGzPlCxCVn0RSkqqIXj7xqmOvb_v3gISWSF6RFApU-tPsUzwgfCixQCD9Oqx2wDB6TLvbhJ1DDNAJSxmHdtFqWkfysHuVin0U8FBuSGE4JcedY5qGVBdjnpDcaEUd4L-Sce-J8ggMnxS8LryzsbUSFN6Ue4Smp6lkYV2-INzNhH5jAtNsfpmzUMOt2qxb_tPIRmXhfgU-emJxRsOgmbCTy9TjwBNkUu8-8kAXbqLiEm-mmiFII0kKwAxIS_oKFtyJXJm3mP3vBZga8xQE-Z2EBP0nIPuGQA0QphCFxGKSNsjxKYt54I1IcspQeMUJZaTbTozcsClea8GYCDSBDFoMcsogWhBSLRGVlQnXJtwJQ14EoROuFCXWKvSebiureVX9BtF4_tOu23bRazYZVy8z1k1noOZje4m-eH_mQZB_wFxKkyNSyd8t4IgLIDlm4ZiszQV94nGSiNVrZKYO6m6BNMAs3wJPxsIA7tnJOVudEAl8HTrEoewy4ypbpAzmbvsQYwOaXNcAzJUVRFkziGFogGkY-rzEJYoazYgrdEOJMLqFqGmBBimXMhTRmI9eyEriQZSXgOstm01_GKGEmibjeTLlnVkkz9QLCx7PhxCN0oU72qZgGT-Q6x9MCmMrOafRC9l7mV2rPpFFLGuUT5m6xA6CsTC84TCfbd7ZlDF5sZxIoFQHBJUUL5D5nf8gSnJZD63CfRbzAWSY10MCFFEjOCd17DIgbOcUOpjMKbVRR6UEqSkthdITAzpcuc8ekFJNSGPNuSKEOo-kUMob0_QqBVt5xB-Wxl6oyDrIbtQX0PWzcdAAbeQdjcQYL6MIM5of9Khe7WRCluC7ytC6hzAK8MIu5Q28hNfiQ8lVvDM39IvYznQMdUaH2RgEWbLP67BUkn8ipR1gOGqqaOYKdKQonwZ-zwvSXwAsx6AVuxGUoFOaxirGOzYPnQ9oMwy4WGLlxKfyEuYcDYamRTEgwdyY34PoLTOkzZILTavqp-qbGOBceFYR34ZlkKgVbpWgmTm8fqXmU_HTtTxkXiHyT__QxF7PT6gjTkGhA9QTohMILVN8L-YjSKhpO2GvHfZGcHhijYYKE8HRKAneJxgMnBOEkuziyflDKyy_IxyFIPdPbFcj8ZbasmrFvHddbjb-XRlvXrtIpYCCNrLwlWK1uNuv7DcsGDKvWtPdNo3Zs7NdbLdver9ePjca-Xatb-7Zdax7vW40m4NTrNgDaLaPW2m_Yx82GNKdsuzCfdZbZBR6oKkD81JzOsuIRnJZHavKh_Ti4UR8-TISYhidHR6-vr4dTLCZsRL7BAXfoMP9oCkhgg4Pwi0fpgSR71IH_zsadTrfndsIr5_MTHf30c5_jz2OzNQ5d5t5N7ujP3XoDwJz6T1e_dE5PFcOjhGNbz_BCzT7-powk5ZOWqSJPEF9-iyMesNNIbZlTuAdW1XvySPplVw5SrtFO0Q76R11j_wOuOTfczuXFZ3YArrH867vR9c0Vvnr4_PDbcHxAAODX--HA6PzX3NI-kvEkP9wxQRSsfJh8aQ-lCiHkBC4uiR-ezSA9X8gadGUcFQekhB4SoXNNGieZE7tkhCMqn_8SYerJ_GCkn97omXbAuD_vsYEU5AdZRWiKD7Op3BKdmxu90qEiJibZJclC77lYIOS5i2yldZIfzzF1lM7t62AagUXVvNv3Quf352g0ksNrYCG4Gsj3Li565w_XT70426ZRlKd_DyL_Wc5m9b-LM3FIVO2Hwug51B9Pq08eeVWCdInAHoUTwGGU4mlI5ulOCR1rQAEvh5qCgk49mXSvp7UAyKbU-0Y4ZOcxNA0O90imXPP1d4TSDGVDIU-nLGpygpxNSNes55A6dMOTYSk1rs-mIifomerIxRxcONcwzeQcr75jCSG3LUSdN_IcebDnu1xucg2VYxfHgZrAmeX4Oy64s2mokXwWAb2YjayH6lnY8WqOVdVEP9OqejUbvUscnKm7XsxGnvfQLAAzZVGZQ-VQumOB2uQQNB2PyuI407M9SuYg2QTvxYTw-ITMonQLOSoByQ0c7j1HIjuMUxA5tlJjvQwLybVsVD23ytBBruVkoqVZToZB0zDZpPQEJDOR5aHqRinTfnHbleOCeL6QYX69mmOEZMSSoX-8nBMkKv92Xpjn6tY7I1xWwPISBpQV25NR84TtyawOGLaneAEl35dMf8er2eiPwpOVyBoqugAqREQG1XYUZGxtR2GwWkgscAf5JYTKX10yImDo3AQ2h8nxr4iCLhxyIse3BUkpsdYHwkK1UrR0Kl-raWmKeoPGNy15e1iDvEMIzqKrnGqnGKX50OeKYCpv0BndjuCbG6Wt9GQixIHblSPrLRWVE-9oCsQSye7XlasLl65SbR8ljYSaTsjSPp6pDAWXI5U_GfM_n1ZrjeZh0z4-bphGq6Wfxv3OcdzjQMHX9cDsXG2XhKcE_PW0emDVrMNm0zBapqUnp-1raPrCuPmSn5PeKwqJvvz9RPCUBepxqi2SoOhFj3HkACVuj1SnOIDHYnaCHu-uf3nsVS4p4x6LQvSEoRybVZIZE_qJvBJauSF4HJETdEPGYMpKPBo6QV3GZ4CA9s4Z59FUEBfJe-OPC4hfvWf8NcJor-u9QBuJLoDqJLX-G-HTCRDoCcIDTNFnFonJR7TX5wTZh6Z5aKSAO5MIEh7au_Z96BEBugO9qXTux4runYD_HNiqKKsPyFf4bFQG2PWiEFSQc7XKjed7ElqwE2RWoLKnngORdIKMyl-xxU7Mv__iOBiTE-Ow_vcZNP8A7wUI_kXPlDEXsRHaMw3joGUYxkcUYuiUoOhfQMmmUQLFVlpQtnZFObbugrK9K8raLwvCtV0R1j6s9PXwJJTLXLkGYXD5OQu-RhAXLnqeKbQnjGnlCoov2J-OGnFAuz6QkiAlj3rPq5pq_PMgrHch7Hchau9C1N-FSC7QjmSErglVOxF3XagOJ_iFsyAg_4KYD_8dok8cT6dwFISVe-gUqQBjJUtPJBTy8KroChrpEvoE2U2jki6qz3CYvOHiUSJDYh5GMhAhCg-N5Uf28qPziHPJ520Y2pV4aqOiMB5WnaCzg_n_qThtWOlYNCupPbdnH5jGx-9gvzicgLiQaWRRiHSWzojaPQt2pFVfxluyRUZQ7pk1QDQKIdrLiMDRzOVoGt-hkRqQyqE_FzIqliAQnFRq_J_W__s92zioSbqCofkdEtKdiSx1KgMyksYPkSlBQOsUmLaSfK6uXJGetRAeVtQ9UbLuMojIgAmIPjndXBH8fyv6ciBa78SnlYi7Lj57_pRwxuch6I09qvwzFIRQ9DOcO4lBdV8IG70F8ai_vI3E1dA9tt4J3eSwvsIcEqxYjVvzuFjgWjmBmwrZWmPl-EzHQuPAsmDH6vGU3H2qrSwUubYht_tyHGkTISjhsoMY4s8oh2OuxC-E7xJXX9ezSLZraaxzHMhQ0Sq9TsD6aMYiBOkZTju2wi0JQFlNohFnvsYD8rKCR46mtTbs3qI-Bs8x8kQiY_qKZ2GMTFbRL-XukzkAGL6G_z9B3NwwzOU9QnZhna6h20MKPpM3Mv3nx8GNbAH0PYHGQubiusZIhMlCOSNUoM7zLAyhrI1vfqwS-AnLVRr18jSsQjTU0rULBFZuqDQt1TegxZ1VwqqoGsNXPF2Vo7EDezR2YI8yNM4YNLCrBOwSBK4I9cmbrbHt1trIDOt8Ut-BT8wd0KiVNMgmLNfpv4sYLUNDDfRKWX3tDrS23D92aZHLOOiSMtkabrFN1kdNozSF0mqaW6tZxrcdP6JE7GAL2jtIi_bWqtdKm7vMPh7IIsjaNPGVVm7bEK2XNsb2x4q1NYXtZahta7jarnK7XUoSd5b0GjvPW_VdndbmbjSytnVR6Qpq69TS2I3m9a0FsctmLHNbY5s7yO07i6n6bvzQ2LJ-2b4a2baC2lEo2rtKDTsjtElrsZktdrUnd-SKjWNky520s9PB2kGi2AUNc1cK7SxrvSWkh0ZDIvQr-Ry7ZKjugj8ROXQP9YWxurhVr6CyYOSN37xQ6kShYP4tc8PFC6jfNwxE9AQvTL_x1KfYIRNGXcJjbIDyZ_GfL0heRm2mfzdvHXz6rxEkSPV8lOVfclqgWQVYreLY7-DEv0jFKFUzu7RimYj-_A8myD8BIG8Ph-pNUhYFYkjoKPWq7juaJnklgT9u5sPP39uaq2eamRihvAm4H6lfUwEB1e-DLL9D3D5K9kn7aPXPgfwHN32RFg==

@mortentc mortentc changed the title changed bypass override Fixed negative bypass being ignored Dec 11, 2023
@Wires77 Wires77 changed the title Fixed negative bypass being ignored Fix negative bypass being ignored Dec 11, 2023
@Wires77 Wires77 added the bug: calculation Numerical differences label Dec 11, 2023
@Wires77 Wires77 merged commit a9959c0 into PathOfBuildingCommunity:dev Dec 11, 2023
deathbeam added a commit to deathbeam/PathOfBuilding-1 that referenced this pull request Dec 12, 2023
* upstream-dev: (31 commits)
  Release 2.37.0 (PathOfBuildingCommunity#7019)
  Add support for new Uniques + fix parsing for changed mod names (PathOfBuildingCommunity#7016)
  Fix ES from Tricksters Escape Artist when using Oath of the Maji (PathOfBuildingCommunity#7018)
  Fix Necromancer Offering charm not working (PathOfBuildingCommunity#7014)
  Update Query mods (PathOfBuildingCommunity#7011)
  Fix projectile count being 1 higher on all skills (PathOfBuildingCommunity#7006)
  Fix Herald of Agony quality not working (PathOfBuildingCommunity#7017)
  Fix Pyroclast Mine Aura Effect scaling Maximum Added Flat Damage (PathOfBuildingCommunity#7005)
  Fix Ascendant nodes counting towards allocated passive skill total (PathOfBuildingCommunity#7002)
  Change Manastorm config option to not overrun options box (PathOfBuildingCommunity#7008)
  Release 2.36.1
  Release 2.36.1 (PathOfBuildingCommunity#6996)
  Fix pathing and mastery nodes (PathOfBuildingCommunity#6989)
  Export from game files
  Fix Crash when opening Timeless Jewel search (PathOfBuildingCommunity#6995)
  Fix negative bypass being ignored (PathOfBuildingCommunity#6992)
  Release 2.36.0
  Release 2.36.0 (PathOfBuildingCommunity#6988)
  Add support for Tincture Implicits
  Fix ES Bypass not overriding sources of negative bypass (PathOfBuildingCommunity#6965)
  ...
@mortentc mortentc deleted the es_bypass branch January 26, 2024 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: calculation Numerical differences
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect ES bypass
2 participants