-
Notifications
You must be signed in to change notification settings - Fork 3
New market_description #26
base: master
Are you sure you want to change the base?
Conversation
I see that the description of market is remarkably similar to that of portfolio, and that both have the alias "equity". I also see that both datasets have the same column names: setdiff(
names(r2dii.analysis::portfolio),
names(r2dii.analysis::market)
)
#> character(0)
setdiff(
names(r2dii.analysis::market),
names(r2dii.analysis::portfolio)
)
#> character(0) Created on 2019-12-18 by the reprex package (v0.3.0.9001) Should we document both datasets together? If so, how would you adapt the description to clarify the difference between the two datasets? |
The only difference between the portfolio and market is that the market is used as a benchmark. So, in theory, you could upload any two portfolios and use one of them as the benchmark. We still probably want two portfolios because the values will be different between the two. @2diiKlaus maybe has some suggestions as well. |
#' Description of market datasets @maurolepore I just adjusted the first part and not the part with @name etc. I also integrated the type, market type (as there are several markets) and status to reflect to position in the code process. |
@2diiKlaus is clear to me! |
I see lot's of good information and I see it's clear to @vintented. @vintented, can you help me extract actionable steps? Here are my questions: . #25 adds the new datasets
. @2diiKlaus said that "Sample data needs to exist ... at several stages of the code [and] for different classes of datasets ... We thus have a matrix of datasets ... " (#26 (comment)):
. #26 (comment) contains what seems valuable information.
. #26 (comment) documents 4.1. Does this confirm that 4.2. Does #26 (comment) replace what @vintented shared before (currently at R/portfolio_description.R in #25 )? |
yes please
yes sounds like a very good idea!
I would be fine with this. Probably needs a bit of polishing, but can form the basis!
not sure about it, as users might get confused (especially non 2dii users but also 2dii internal users). I would be fine with it as long as the descriptions is very clear about the differences. But I think it might be better to keep them separate - not sure as said. maybe @vintented can reflect on the fact how "newer" employees see it and if the difference is clear to them even if we just have a single documentation?! |
Relates to https://github.com/2DegreesInvesting/r2dii.analysis/pull/10#issuecomment-566047068
Closes #24
Thanks @vintented