Closed slundqui closed 3 months ago
Attention: Patch coverage is 90.47619%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 81.41%. Comparing base (
84d0837
) to head (6efccdc
).:exclamation: Current head 6efccdc differs from pull request most recent head e864186
Please upload reports for the commit e864186 to get more accurate results.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This PR allows for only calculating pnl when calling
get_positions
for speed. To utilize this, setcalc_pnl = False
in theChain.Config
. If this configuration is set to False,get_positions
accepts acalc_pnl
argument, which fills the pnl values in the resulting dataframe.Includes a bugfix to
get_positions
when getting positions from multiple pools.