filter out short positions

[jdhoffa]

Broadly in-line with what was attempted here:
https://github.com/RMI-PACTA/PACTA_analysis/pull/533/files

Presumably it should happen here:

pacta.portfolio.audit/R/process_raw_portfolio.R

Line 46 in 61f1472

@NAYRA-HERRERA has agreed that short positions should never be in the analysis at any point and should not appear in any of the outputs, so removing it as early on in the process as possible seems to be desirable.

[cjyetman]

Was this intentionally closed? @jdhoffa

[jdhoffa]

Yes, I realized we need way more precise scoping before this is ready to be picked up.

Acceptance criteria scoping discussion is here:
https://dev.azure.com/RMI-PACTA/2DegreesInvesting/_workitems/edit/12306

Once we have a super clear idea of what needs to be done, I'll open a GH tracking issue